diff options
| author | Eduardo Barretto <edusbarretto@gmail.com> | 2020-04-16 19:09:40 -0300 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-04-17 00:09:40 +0200 |
| commit | 040e142288e90c9c2d46d25d0a27f828f968bb93 (patch) | |
| tree | 48d747bb1bcdfa12c4b1cc3105e136e0fbfdb868 /src/bin/jp3d | |
| parent | 221a801a97a3ea968a311f7905c18a1eb7f034c4 (diff) | |
jp3d/jpwl/mj2/jpip: Fix resource leaks (#1226)
This issues were found by cppcheck and coverity.
Diffstat (limited to 'src/bin/jp3d')
| -rw-r--r-- | src/bin/jp3d/convert.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/src/bin/jp3d/convert.c b/src/bin/jp3d/convert.c index acad8f82..4e4bea9c 100644 --- a/src/bin/jp3d/convert.c +++ b/src/bin/jp3d/convert.c @@ -280,6 +280,7 @@ opj_volume_t* pgxtovolume(char *relpath, opj_cparameters_t *parameters) if (!sliceno) { fprintf(stdout, "[ERROR] No slices with this pattern founded !! Please check input volume name\n"); + closedir(dirp); return NULL; } /*if ( maxslice != sliceno) { @@ -317,6 +318,7 @@ opj_volume_t* pgxtovolume(char *relpath, opj_cparameters_t *parameters) } else { fprintf(stdout, "[ERROR] Bad pgx header, please check input file\n"); fclose(f); + closedir(dirp); return NULL; } @@ -396,9 +398,7 @@ opj_volume_t* pgxtovolume(char *relpath, opj_cparameters_t *parameters) fclose(f); } /* for s --> sliceno*/ comp->bpp = int_floorlog2(maxvalue) + 1; - if (sliceno != 1) { - closedir(dirp); - } + closedir(dirp); /*dump_volume(stdout, volume);*/ return volume; } @@ -581,6 +581,7 @@ opj_volume_t* bintovolume(char *filename, char *fileimg, f = fopen(filename, "rb"); if (!f) { fprintf(stdout, "[ERROR] Failed to open %s for reading !!\n", filename); + opj_free(volume); return 0; } @@ -918,7 +919,7 @@ opj_volume_t* imgtovolume(char *fileimg, opj_cparameters_t *parameters) f = fopen(filename, "rb"); if (!f) { fprintf(stderr, "[ERROR] Failed to open %s for reading !!\n", filename); - fclose(f); + opj_free(volume); return 0; } |
