Skip to content

Commit

Permalink
xcf: don't double-free a pointer on bogus data.
Browse files Browse the repository at this point in the history
free_xcf_hierarchy() frees the same pointer that this code manually frees.
  • Loading branch information
icculus committed Jan 28, 2018
1 parent 0ea4c21 commit 5c6aaba
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions IMG_xcf.c
Expand Up @@ -651,9 +651,6 @@ do_layer_surface(SDL_Surface * surface, SDL_RWops * src, xcf_header * head, xcf_
default:
SDL_Log("Unknown Gimp image type (%d)\n", head->image_type);
if (hierarchy) {
if (hierarchy->level_file_offsets)
SDL_free(hierarchy->level_file_offsets);

free_xcf_hierarchy(hierarchy);
}
if (level)
Expand Down

0 comments on commit 5c6aaba

Please sign in to comment.