Bug 6765

Summary: Leak in JBIG2Stream.cc::readHalftoneRegionSeg
Product: poppler Reporter: Kjartan Maraas <kmaraas>
Component: generalAssignee: poppler-bugs <poppler-bugs>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: high    
Version: unspecified   
Hardware: x86 (IA32)   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:
Attachments: proposed patch

Description Kjartan Maraas 2006-04-27 18:10:36 UTC
Coverity report #2074.
Comment 1 Kjartan Maraas 2006-04-27 18:10:55 UTC
Created attachment 5497 [details] [review]
proposed patch
Comment 2 Albert Astals Cid 2006-04-29 03:33:25 UTC
Interesting i don't see poppler at http://scan.coverity.com/ do you have any 
insider info?

BTW i'll have a look at the patches if nobody has done, tomorrow.
Comment 3 Kjartan Maraas 2006-04-29 04:11:32 UTC
It's in the GNOME runs if you don't have access to that "product" you should
just mail and ask for it.
Comment 4 Albert Astals Cid 2006-04-29 04:17:14 UTC
Well, i'm not a Gnome developer in any way, so i'm not sure i should be allowed 
to see all your mistakes ;-)
Comment 5 Kjartan Maraas 2006-04-29 05:53:32 UTC
Well, they're all available for your viewing pleasure in GNOME CVS so I don't
think it's that controversial :-)
Comment 6 Albert Astals Cid 2006-04-30 01:22:29 UTC
The patch is wrong, one has to move the delete down near gfree(grayImg) as 
skipBitmap is used on the if (!(enableSkip && skipBitmap->getPixel(n, m))) line 
2452

Commited to cvs

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.