mirror of
https://github.com/RfidResearchGroup/proxmark3.git
synced 2025-03-10 23:15:22 +08:00
BUGS: more Coverity scan fixes.
This commit is contained in:
parent
2696349f16
commit
664bb5ae72
6 changed files with 10 additions and 13 deletions
|
@ -1202,7 +1202,7 @@ int CmdHF14AMfELoad(const char *Cmd)
|
|||
|
||||
if (len > FILE_PATH_SIZE) len = FILE_PATH_SIZE;
|
||||
|
||||
fnameptr += len;
|
||||
fnameptr += len-4;
|
||||
|
||||
sprintf(fnameptr, ".eml");
|
||||
|
||||
|
@ -1593,6 +1593,7 @@ int CmdHF14AMfCLoad(const char *Cmd)
|
|||
|
||||
if (fgets(buf, sizeof(buf), f) == NULL) {
|
||||
PrintAndLog("File reading error.");
|
||||
fclose(f);
|
||||
return 2;
|
||||
}
|
||||
|
||||
|
@ -1600,6 +1601,7 @@ int CmdHF14AMfCLoad(const char *Cmd)
|
|||
if(strlen(buf) && feof(f))
|
||||
break;
|
||||
PrintAndLog("File content error. Block data must include 32 HEX symbols");
|
||||
fclose(f);
|
||||
return 2;
|
||||
}
|
||||
for (i = 0; i < 32; i += 2)
|
||||
|
|
|
@ -83,7 +83,7 @@ int CmdList(const char *Cmd)
|
|||
{
|
||||
while ((ep = readdir (dp)) != NULL)
|
||||
{
|
||||
if(ep->d_name != NULL && str_ends_with(ep->d_name, ".lua"))
|
||||
if( str_ends_with(ep->d_name, ".lua"))
|
||||
PrintAndLog("%-16s %s", ep->d_name, "A script file");
|
||||
}
|
||||
(void) closedir (dp);
|
||||
|
|
|
@ -356,10 +356,7 @@ int loadTraceCard(uint8_t *tuid) {
|
|||
FillFileNameByUID(traceFileName, tuid, ".eml", 7);
|
||||
|
||||
f = fopen(traceFileName, "r");
|
||||
if (!f) {
|
||||
fclose(f);
|
||||
return 1;
|
||||
}
|
||||
if (!f) return 1;
|
||||
|
||||
blockNum = 0;
|
||||
|
||||
|
@ -396,10 +393,7 @@ int saveTraceCard(void) {
|
|||
if ((!strlen(traceFileName)) || (isTraceCardEmpty())) return 0;
|
||||
|
||||
f = fopen(traceFileName, "w+");
|
||||
if ( !f ) {
|
||||
fclose(f);
|
||||
return 1;
|
||||
}
|
||||
if ( !f ) return 1;
|
||||
|
||||
for (int i = 0; i < 64; i++) { // blocks
|
||||
for (int j = 0; j < 16; j++) // bytes
|
||||
|
|
|
@ -341,6 +341,7 @@ int setLuaPath( lua_State* L, const char* path )
|
|||
lua_pushstring( L, buf ); // push the new one
|
||||
lua_setfield( L, -2, "path" ); // set the field "path" in table at -2 with value at top of stack
|
||||
lua_pop( L, 1 ); // get rid of package table from top of stack
|
||||
free(buf);
|
||||
return 0; // all done!
|
||||
}
|
||||
|
||||
|
|
|
@ -124,7 +124,7 @@ char *sprint_hex(const uint8_t *data, const size_t len) {
|
|||
|
||||
char *sprint_bin_break(const uint8_t *data, const size_t len, const uint8_t breaks) {
|
||||
|
||||
int maxLen = ( len > 1024) ? 1024 : len;
|
||||
int maxLen = ( len > 1020) ? 1020 : len;
|
||||
static char buf[1024];
|
||||
memset(buf, 0x00, 1024);
|
||||
char *tmp = buf;
|
||||
|
|
Loading…
Reference in a new issue