commit fd1382895821a40310da9a22d7e0dce5bef4373b
author Sean Paul <seanpaul@chromium.org> Fri Jul 10 16:43:06 2015 -0400
committer Sean Paul <seanpaul@chromium.org> Mon Jul 13 11:57:56 2015 -0400
tree b6630d500affe070ecec80e01816e6339640e8a1
parent acb2a4494e79f0026f8615acc561257276a71062

drm_hwcomposer: Free failed composition in QueueComposition

Rather than maintaining ownership of the composition in the
caller, free the DrmComposition in QueueComposition on failure.

Change-Id: Id70e98ae1a8eef400bc29460bfc5a89c4707d617
Signed-off-by: Sean Paul <seanpaul@chromium.org>

drmcompositor.cpp

diff --git a/drmcompositor.cpp b/drmcompositor.cpp
index 4268ab5..9ade368 100644
--- a/drmcompositor.cpp
+++ b/drmcompositor.cpp
@@ -71,6 +71,7 @@
         drm_composition->TakeDisplayComposition(display));
     if (ret) {
       ALOGE("Failed to queue composition for display %d", display);
+      delete composition;
       return ret;
     }
   }