lua: fix coverity issue with out of scope variable

Fix usage of out-of-scope variables. Introduced with the hashing
and adding the guard of g_disable_hashing.

To fix, just remove the guard so all variables are in scope. Hashes
are not initialized here so there is no need for the guard.
pull/5727/head
Jason Ish 5 years ago committed by Victor Julien
parent 1c68f4aed6
commit 512b0350a0

@ -757,36 +757,35 @@ static int LuaCallbackFileInfoPushToStackFromFile(lua_State *luastate, const Fil
char *md5ptr = NULL; char *md5ptr = NULL;
char *sha1ptr = NULL; char *sha1ptr = NULL;
char *sha256ptr = NULL; char *sha256ptr = NULL;
if (!g_disable_hashing) {
char md5[33] = ""; char md5[33] = "";
md5ptr = md5; md5ptr = md5;
if (file->flags & FILE_MD5) { if (file->flags & FILE_MD5) {
size_t x; size_t x;
for (x = 0; x < sizeof(file->md5); x++) { for (x = 0; x < sizeof(file->md5); x++) {
char one[3] = ""; char one[3] = "";
snprintf(one, sizeof(one), "%02x", file->md5[x]); snprintf(one, sizeof(one), "%02x", file->md5[x]);
strlcat(md5, one, sizeof(md5)); strlcat(md5, one, sizeof(md5));
}
} }
char sha1[41] = ""; }
sha1ptr = sha1; char sha1[41] = "";
if (file->flags & FILE_SHA1) { sha1ptr = sha1;
size_t x; if (file->flags & FILE_SHA1) {
for (x = 0; x < sizeof(file->sha1); x++) { size_t x;
char one[3] = ""; for (x = 0; x < sizeof(file->sha1); x++) {
snprintf(one, sizeof(one), "%02x", file->sha1[x]); char one[3] = "";
strlcat(sha1, one, sizeof(sha1)); snprintf(one, sizeof(one), "%02x", file->sha1[x]);
} strlcat(sha1, one, sizeof(sha1));
} }
char sha256[65] = ""; }
sha256ptr = sha256; char sha256[65] = "";
if (file->flags & FILE_SHA256) { sha256ptr = sha256;
size_t x; if (file->flags & FILE_SHA256) {
for (x = 0; x < sizeof(file->sha256); x++) { size_t x;
char one[3] = ""; for (x = 0; x < sizeof(file->sha256); x++) {
snprintf(one, sizeof(one), "%02x", file->sha256[x]); char one[3] = "";
strlcat(sha256, one, sizeof(sha256)); snprintf(one, sizeof(one), "%02x", file->sha256[x]);
} strlcat(sha256, one, sizeof(sha256));
} }
} }

Loading…
Cancel
Save