From 4bb17c66333da6ac8ed6c4757f8377d37f7f7e24 Mon Sep 17 00:00:00 2001
From: iceman1001 <iceman@iuse.se>
Date: Tue, 12 Jan 2016 22:47:48 +0100
Subject: [PATCH] FIX: Coverity , Argument can't be negative,  CID #121323,  
 ftell(f) can be negative, not allowed in malloc.

FIX: forgot to close the filehandle :(
---
 client/cmdhficlass.c | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/client/cmdhficlass.c b/client/cmdhficlass.c
index d4750735..86524cd6 100644
--- a/client/cmdhficlass.c
+++ b/client/cmdhficlass.c
@@ -286,10 +286,10 @@ int CmdHFiClassELoad(const char *Cmd) {
 
 	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);
@@ -1509,6 +1509,12 @@ static int loadKeys(char *filename) {
 	long fsize = ftell(f);
 	fseek(f, 0, SEEK_SET);
 
+	if ( fsize < 0 ) {
+		PrintAndLog("Error, when getting filesize");
+		fclose(f);
+		return 1;
+	}
+
 	uint8_t *dump = malloc(fsize);
 
 	size_t bytes_read = fread(dump, 1, fsize, f);
-- 
2.39.2