summaryrefslogtreecommitdiff
path: root/core/fxcodec
diff options
context:
space:
mode:
authordsinclair <dsinclair@chromium.org>2016-08-31 06:03:49 -0700
committerCommit bot <commit-bot@chromium.org>2016-08-31 06:03:49 -0700
commit87dffc0315477150c9c1964913b65bc97bdf654f (patch)
treef8a2574c163d1dbc951a8300006fb0ccf7874d23 /core/fxcodec
parent7e7ef3dd60a836521607d4b2ec41cbbe950183f9 (diff)
downloadpdfium-87dffc0315477150c9c1964913b65bc97bdf654f.tar.xz
Free the GifPlainText object when complete.
We allocate the GifPlainText object on line ~685 inside GIF_D_STATUS_EXT_PTE. We cleanup the internal pointers in the gif_destroy_decompress() but we failed to cleanup the pointer itself. This CL frees the allocated pointer once the data is cleaned up. BUG=chromium:638499 Review-Url: https://codereview.chromium.org/2291143003
Diffstat (limited to 'core/fxcodec')
-rw-r--r--core/fxcodec/lgif/fx_gif.cpp1
1 files changed, 1 insertions, 0 deletions
diff --git a/core/fxcodec/lgif/fx_gif.cpp b/core/fxcodec/lgif/fx_gif.cpp
index b94445ccf3..08b008ddb4 100644
--- a/core/fxcodec/lgif/fx_gif.cpp
+++ b/core/fxcodec/lgif/fx_gif.cpp
@@ -418,6 +418,7 @@ void gif_destroy_decompress(gif_decompress_struct_pp gif_ptr_ptr) {
FX_Free(p->gce_ptr);
FX_Free(p->pte_ptr);
delete p->string_ptr;
+ FX_Free(p);
}
gif_ptr->pt_ptr_arr_ptr->RemoveAll();
delete gif_ptr->pt_ptr_arr_ptr;