commit 449d3f08148ac73f5cf7e2464a3faec1eba42f6c
author Yunlian Jiang <yunlian@google.com> Tue Jun 27 15:51:41 2017 -0700
committer Yunlian Jiang <yunlian@google.com> Wed Jun 28 11:34:08 2017 -0700
tree adafd303faae43450a84ba2db749e4b0202987ba
parent 66688d8836830d742dd52dffabbf6c26bd16ad94

Fix memory leak warning.

This CL eliminates the following compiler warning:

MediaPlayerFactory.cpp:249:27: warning: Potential memory leak

Bug: None
Test: After this change the warning is gone.
Change-Id: I09e7ec5cc3bc84decfea2456550b532aac55d26f

media/libmediaplayerservice/MediaPlayerFactory.cpp

diff --git a/media/libmediaplayerservice/MediaPlayerFactory.cpp b/media/libmediaplayerservice/MediaPlayerFactory.cpp
index 605c710..f7c691d 100644
--- a/media/libmediaplayerservice/MediaPlayerFactory.cpp
+++ b/media/libmediaplayerservice/MediaPlayerFactory.cpp
@@ -246,8 +246,12 @@
     if (sInitComplete)
         return;
 
-    registerFactory_l(new NuPlayerFactory(), NU_PLAYER);
-    registerFactory_l(new TestPlayerFactory(), TEST_PLAYER);
+    IFactory* factory = new NuPlayerFactory();
+    if (registerFactory_l(factory, NU_PLAYER) != OK)
+        delete factory;
+    factory = new TestPlayerFactory();
+    if (registerFactory_l(factory, TEST_PLAYER) != OK)
+        delete factory;
 
     sInitComplete = true;
 }