summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTom Taylor <tomtaylor@google.com>2017-01-04 09:42:37 -0800
committerTom Taylor <tomtaylor@google.com>2017-01-04 09:42:37 -0800
commit837474a376a4599d57f791966080f93ca7afaf67 (patch)
tree0827f59b8f495a5e1acfb8698e1498311691922c
parent3f6e2e2b07a4b2a7f1e0780ce3607eab6c45b737 (diff)
downloadandroid_packages_apps_Messaging-837474a376a4599d57f791966080f93ca7afaf67.tar.gz
android_packages_apps_Messaging-837474a376a4599d57f791966080f93ca7afaf67.tar.bz2
android_packages_apps_Messaging-837474a376a4599d57f791966080f93ca7afaf67.zip
33388925 Mismatched new vs delete in framesequence library
* The array allocations neglected to include [] so delete, instead of delete [] would get called. * Test Manual - tested sending a large gif that would invoke the GifTranscoder library to make the gif smaller. Bug: 33388925 Change-Id: I1e200e470d66ae615ffe9340ff9c049eaa73f63c
-rw-r--r--jni/GifTranscoder.cpp4
1 files changed, 2 insertions, 2 deletions
diff --git a/jni/GifTranscoder.cpp b/jni/GifTranscoder.cpp
index 0d50770..81f3f75 100644
--- a/jni/GifTranscoder.cpp
+++ b/jni/GifTranscoder.cpp
@@ -144,10 +144,10 @@ bool GifTranscoder::resizeBoxFilter(GifFileType* gifIn, GifFileType* gifOut) {
std::vector<GifByteType> srcBuffer(gifIn->SWidth * gifIn->SHeight);
// Buffer for rendering images from the input GIF.
- std::unique_ptr<ColorARGB> renderBuffer(new ColorARGB[gifIn->SWidth * gifIn->SHeight]);
+ std::unique_ptr<ColorARGB[]> renderBuffer(new ColorARGB[gifIn->SWidth * gifIn->SHeight]);
// Buffer for writing new images to output GIF (one row at a time).
- std::unique_ptr<GifByteType> dstRowBuffer(new GifByteType[gifOut->SWidth]);
+ std::unique_ptr<GifByteType[]> dstRowBuffer(new GifByteType[gifOut->SWidth]);
// Many GIFs use DISPOSE_DO_NOT to make images draw on top of previous images. They can also
// use DISPOSE_BACKGROUND to clear the last image region before drawing the next one. We need