Revision: 3563
          http://skim-app.svn.sourceforge.net/skim-app/?rev=3563&view=rev
Author:   amaxwell
Date:     2008-03-26 16:39:57 -0700 (Wed, 26 Mar 2008)

Log Message:
-----------
Check for NULL before calling CFRelease, since ImageIO doesn't wrap 
CFRetain/CFRelease.

Modified Paths:
--------------
    trunk/SKDocumentController.m

Modified: trunk/SKDocumentController.m
===================================================================
--- trunk/SKDocumentController.m        2008-03-26 20:19:52 UTC (rev 3562)
+++ trunk/SKDocumentController.m        2008-03-26 23:39:57 UTC (rev 3563)
@@ -198,7 +198,7 @@
 
     NSMutableData *pdfData = nil;
     
-    if (CGImageSourceGetCount(imsrc)) {
+    if (imsrc && CGImageSourceGetCount(imsrc)) {
         CGImageRef cgImage = CGImageSourceCreateImageAtIndex(imsrc, 0, NULL);
 
         pdfData = [NSMutableData dataWithCapacity:[tiffData length]];
@@ -219,7 +219,7 @@
         CGImageRelease(cgImage);
     }
     
-    CFRelease(imsrc);
+    if (imsrc) CFRelease(imsrc);
 
     return pdfData;
 }


This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.

-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
_______________________________________________
Skim-app-commit mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/skim-app-commit

Reply via email to