commit b383438b97643060e7ff5e737f0439418bd0ce3e
author Mathias Agopian <mathias@google.com> Wed Aug 21 11:56:48 2013 -0700
committer Android Git Automerger <android-git-automerger@android.com> Wed Aug 21 11:56:48 2013 -0700
tree 0a0f260eab88727ca41dfc743b1613f3984306b8
parent 5bc0c358a80f3c0e762b489e8bfb5888444159a6
parent a525483dc73df39a72c9b4865102a86704b3622c

am a525483d: Merge "Fix a bug where non-cropped layer could be scaled incorrectly" into klp-dev

* commit 'a525483dc73df39a72c9b4865102a86704b3622c':
  Fix a bug where non-cropped layer could be scaled incorrectly

libs/gui/IGraphicBufferProducer.cpp

diff --git a/libs/gui/IGraphicBufferProducer.cpp b/libs/gui/IGraphicBufferProducer.cpp
index 3080220..704fa50 100644
--- a/libs/gui/IGraphicBufferProducer.cpp
+++ b/libs/gui/IGraphicBufferProducer.cpp
@@ -61,7 +61,11 @@
         bool nonNull = reply.readInt32();
         if (nonNull) {
             *buf = new GraphicBuffer();
-            reply.read(**buf);
+            result = reply.read(**buf);
+            if(result != NO_ERROR) {
+                (*buf).clear();
+                return result;
+            }
         }
         result = reply.readInt32();
         return result;

libs/ui/GraphicBuffer.cpp

diff --git a/libs/ui/GraphicBuffer.cpp b/libs/ui/GraphicBuffer.cpp
index 0ecd3d9..96a7188 100644
--- a/libs/ui/GraphicBuffer.cpp
+++ b/libs/ui/GraphicBuffer.cpp
@@ -282,6 +282,8 @@
     if (handle != 0) {
         status_t err = mBufferMapper.registerBuffer(handle);
         if (err != NO_ERROR) {
+            width = height = stride = format = usage = 0;
+            handle = NULL;
             ALOGE("unflatten: registerBuffer failed: %s (%d)",
                     strerror(-err), err);
             return err;