From c92fead4333683f7b13fc730203c67834aa478ef Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Morten=20Johan=20S=C3=B8rvig?= <morten.sorvig@digia.com>
Date: Mon, 7 Apr 2014 09:26:48 +0200
Subject: [PATCH] Cocoa: Fix possible printing memory leak.

The created page format must always be released,
even if one of the subsequent PM* functions fails.

Change-Id: If42aaeccd6bdb51ba53444f491ca2878783d0678
Reviewed-by: John Layt <jlayt@kde.org>
---
 src/plugins/platforms/cocoa/qcocoaprintdevice.mm | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/plugins/platforms/cocoa/qcocoaprintdevice.mm b/src/plugins/platforms/cocoa/qcocoaprintdevice.mm
index 281b140e15..57d5c800e0 100644
--- a/src/plugins/platforms/cocoa/qcocoaprintdevice.mm
+++ b/src/plugins/platforms/cocoa/qcocoaprintdevice.mm
@@ -219,10 +219,11 @@ QPageSize QCocoaPrintDevice::defaultPageSize() const
     QPageSize pageSize;
     PMPageFormat pageFormat;
     PMPaper paper;
-    if (PMCreatePageFormat(&pageFormat) == noErr
-        && PMSessionDefaultPageFormat(m_session, pageFormat) == noErr
-        && PMGetPageFormatPaper(pageFormat, &paper) == noErr) {
-        pageSize = createPageSize(paper);
+    if (PMCreatePageFormat(&pageFormat) == noErr) {
+        if (PMSessionDefaultPageFormat(m_session, pageFormat) == noErr
+            && PMGetPageFormatPaper(pageFormat, &paper) == noErr) {
+            pageSize = createPageSize(paper);
+        }
         PMRelease(pageFormat);
     }
     return pageSize;