commit 158c197b668ad684f92829db6a31bee3aec794ba
author Caroline Tice <cmtice@google.com> Tue Jan 17 16:33:03 2017 -0800
committer Caroline Tice <cmtice@google.com> Wed Jan 18 08:29:01 2017 -0800
tree e1add36937aa2e685c569ba0e046639b61877c2e
parent a72ef28f8cc2b894119b470fe6c93d1abe566398

Fix 'potential memory leak' compiler warning.

This CL fixes the following compiler warning:

frameworks/av/media/libstagefright/SampleTable.cpp:569:9: warning:
 Memory allocated by 'new[]' should be deallocated by 'delete[]', not
 'delete'.

Bug: 33137046
Test: Compiled with change; no warning generated.

Change-Id: I29abd90e02bf482fa840d1f7206ebbdacf7dfa37

media/libstagefright/SampleTable.cpp

diff --git a/media/libstagefright/SampleTable.cpp b/media/libstagefright/SampleTable.cpp
index 5ce2b76..de5ea9c 100644
--- a/media/libstagefright/SampleTable.cpp
+++ b/media/libstagefright/SampleTable.cpp
@@ -566,7 +566,7 @@
 
     if (mDataSource->readAt(data_offset + 8, mSyncSamples,
             (size_t)allocSize) != (ssize_t)allocSize) {
-        delete mSyncSamples;
+        delete[] mSyncSamples;
         mSyncSamples = NULL;
         return ERROR_IO;
     }
@@ -992,4 +992,3 @@
 }
 
 }  // namespace android
-