commit 1d4c54cbc40ac57139d336df4b80561df3238932
author Sean Paul <seanpaul@chromium.org> Fri Jul 10 16:43:06 2015 -0400
committer Sean Paul <seanpaul@chromium.org> Fri Jul 10 22:30:29 2015 -0400
tree b6630d500affe070ecec80e01816e6339640e8a1
parent 1abf5c3fd61c1698e1ef7f9794cac5e22609a58c

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>

compositor.h

diff --git a/compositor.h b/compositor.h
index e148416..b424b37 100644
--- a/compositor.h
+++ b/compositor.h
@@ -90,6 +90,8 @@
   // Starts a fresh composition.
   virtual Composition *CreateComposition(Importer *importer) = 0;
 
+  // Transfers ownership of composition to the Compositor (whether or not this
+  // call returns success) for compositing.
   // On success returns a syncpoint fd that will be signaled when composition is
   // complete or -1 if compositing was completed by this method's return. On
   // error returns an integer less than -1. The composition is invalid after

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;
     }
   }

hwcomposer.cpp

diff --git a/hwcomposer.cpp b/hwcomposer.cpp
index 02c13af..b4fb340 100644
--- a/hwcomposer.cpp
+++ b/hwcomposer.cpp
@@ -407,9 +407,10 @@
   }
 
   ret = ctx->drm.compositor()->QueueComposition(composition);
+  composition = NULL;
   if (ret) {
     ALOGE("Failed to queue the composition");
-    hwc_set_cleanup(num_displays, display_contents, composition);
+    hwc_set_cleanup(num_displays, display_contents, NULL);
     return ret;
   }
   hwc_set_cleanup(num_displays, display_contents, NULL);