From: iceman1001 Date: Wed, 13 Jan 2016 12:22:25 +0000 (+0100) Subject: FIX: Coverity, resource leak, and variable can't be negative, X-Git-Url: https://git.zerfleddert.de/cgi-bin/gitweb.cgi/proxmark3-svn/commitdiff_plain/6b1111d7b88efe0c22d793e08c501d37a9635e9c?hp=089f91dc9da296f3dfcd95d01553e5ad7fcff99f FIX: Coverity, resource leak, and variable can't be negative, --- diff --git a/client/loclass/elite_crack.c b/client/loclass/elite_crack.c index c824eaa1..2329e597 100644 --- a/client/loclass/elite_crack.c +++ b/client/loclass/elite_crack.c @@ -563,6 +563,12 @@ int bruteforceFile(const char *filename, uint16_t keytable[]) long fsize = ftell(f); fseek(f, 0, SEEK_SET); + if (fsize < 0) { + prnlog("Error, when getting filesize"); + fclose(f); + return 1; + } + uint8_t *dump = malloc(fsize); size_t bytes_read = fread(dump, 1, fsize, f); @@ -571,7 +577,10 @@ int bruteforceFile(const char *filename, uint16_t keytable[]) { prnlog("Error, could only read %d bytes (should be %d)",bytes_read, fsize ); } - return bruteforceDump(dump,fsize,keytable); + + uint8_t res = bruteforceDump(dump,fsize,keytable); + free(dump); + return res; } /** *