Merge pull request #111 from diamant3/fix-Wmaybe-uninitialized

fix resource leaks
This commit is contained in:
Wouter Wijsman
2023-01-20 09:44:07 +01:00
committed by GitHub

View File

@@ -245,7 +245,7 @@ int main(int argc, char *argv[])
{ {
u32 fd_size, start = 0, end = 0xffffffff, size = 0xffffffff; u32 fd_size, start = 0, end = 0xffffffff, size = 0xffffffff;
unsigned char * buffer; unsigned char * buffer;
FILE * source, * dest; FILE * source = NULL, * dest = NULL;
char * f_source = 0, * f_dest = 0, * f_label = 0; char * f_source = 0, * f_dest = 0, * f_label = 0;
int i; int i;
@@ -362,13 +362,15 @@ int main(int argc, char *argv[])
buffer = malloc(size); buffer = malloc(size);
if (buffer == NULL) { if (buffer == NULL) {
printf("Failed to allocate memory.\n"); printf("Failed to allocate memory.\n");
fclose(dest); fclose(source);
free(buffer);
return 1; return 1;
} }
if (fread(buffer, 1, size, source) != size) { if (fread(buffer, 1, size, source) != size) {
printf("Failed to read file.\n"); printf("Failed to read file.\n");
fclose(dest); fclose(source);
free(buffer);
return 1; return 1;
} }
fclose(source); fclose(source);