commit 9d06df4ecfe39e7e46873f7cd75a71e32a01d4f7
author Daniel Lam <dalam@google.com> Mon Apr 16 22:47:07 2012 -0700
committer Daniel Lam <dalam@google.com> Mon Apr 16 22:47:07 2012 -0700
tree ef69b0f39d2992098afab3e530bdf2a51da83bb6
parent a98db787397633155fd71b06ecaa765a7ad20cf2

Fixed leaks in SurfaceMediaSource_test

mEglSurface was being reused which resulted in
EGLSurfaces being lost.

Change-Id: Ica07cd24b0e4f3d4b2156f55b7de8e83d5466136

media/libstagefright/tests/SurfaceMediaSource_test.cpp

diff --git a/media/libstagefright/tests/SurfaceMediaSource_test.cpp b/media/libstagefright/tests/SurfaceMediaSource_test.cpp
index 10713fe..466f521 100644
--- a/media/libstagefright/tests/SurfaceMediaSource_test.cpp
+++ b/media/libstagefright/tests/SurfaceMediaSource_test.cpp
@@ -486,6 +486,10 @@
     mSTC = new SurfaceTextureClient(iST);
     mANW = mSTC;
 
+    if (mEglSurface != EGL_NO_SURFACE) {
+        EXPECT_TRUE(eglDestroySurface(mEglDisplay, mEglSurface));
+        mEglSurface = EGL_NO_SURFACE;
+    }
     mEglSurface = eglCreateWindowSurface(mEglDisplay, mGlConfig,
                                 mANW.get(), NULL);
     ASSERT_EQ(EGL_SUCCESS, eglGetError());
@@ -783,6 +787,11 @@
     DummyRecorder writer(mSMS);
     writer.start();
 
+    if (mEglSurface != EGL_NO_SURFACE) {
+        EXPECT_TRUE(eglDestroySurface(mEglDisplay, mEglSurface));
+        mEglSurface = EGL_NO_SURFACE;
+    }
+
     mEglSurface = eglCreateWindowSurface(mEglDisplay, mGlConfig,
                                 mANW.get(), NULL);
     ASSERT_EQ(EGL_SUCCESS, eglGetError());