Merge lp:~aacid/mir/fix_mismatched_delete into lp:mir

[Albert Astals Cid]

Free Mismatched free() / delete / delete [] reported by valgrind

Comes from https://code.launchpad.net/~aacid/mir/fix_mismatched_delete/+merge/190653

[Daniel van Vugt (vanvugt)]

Still wrong target branch :)

review: Needs Resubmitting

[Albert Astals Cid (aacid)]

Now

[Daniel van Vugt (vanvugt)]

review: Approve

[PS Jenkins bot (ps-jenkins)]

FAILED: Autolanding.
More details in the following jenkins job:
http://jenkins.qa.ubuntu.com/job/mir-team-mir-development-branch-autolanding/48/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/mir-android-saucy-i386-build/2339/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-clang-saucy-amd64-build/2224
    FAILURE: http://jenkins.qa.ubuntu.com/job/mir-team-mir-development-branch-saucy-amd64-autolanding/47/console

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

review: Approve (continuous-integration)

[Kevin DuBois (kdub)]

some sort of ps-jenkins autolanding failure? looks okay to me

review: Approve

[Vincent Ladeuil (vila)]

http://10.97.0.26:8080/job/generic-land/18481/console fails because this proposed branch does not include the fix in debian/changelog (lp:mir revno 1099). You need to merge it in this proposed branch, it's not enough to have it in the target branch :-/

=== modified file 'src/server/graphics/android/hwc_layerlist.cpp'
--- src/server/graphics/android/hwc_layerlist.cpp	2013-09-24 22:20:17 +0000
+++ src/server/graphics/android/hwc_layerlist.cpp	2013-10-14 08:12:36 +0000
@@ -70,7 +70,7 @@
 {
     if (self.visibleRegionScreen.rects)
     {
-        delete self.visibleRegionScreen.rects;
+        delete[] self.visibleRegionScreen.rects;
     }
 }