Bug 6188 - mem leak in poppler_page_get_link_mapping
Summary: mem leak in poppler_page_get_link_mapping
Status: RESOLVED DUPLICATE of bug 6947
Alias: None
Product: poppler
Classification: Unclassified
Component: glib frontend (show other bugs)
Version: unspecified
Hardware: x86 (IA32) Linux (All)
: high normal
Assignee: poppler-bugs
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-03-09 07:50 UTC by Christian Persch (GNOME)
Modified: 2006-05-19 15:42 UTC (History)
0 users

See Also:
i915 platform:
i915 features:

Attachments
Proposed patch (259 bytes, patch)
2006-05-08 08:46 UTC, Henning Norén 		Details | Splinter Review
View All

Description Christian Persch (GNOME) 2006-03-09 07:50:55 UTC 
==16259== 96 bytes in 12 blocks are definitely lost in loss record 107 of 305
==16259==    at 0x401B7F3: operator new(unsigned) (vg_replace_malloc.c:164)
==16259==    by 0x42C6459: poppler_page_get_link_mapping (poppler-page.cc:808)
==16259==    by 0x8097EA8: pdf_document_get_links(_EvDocument*, int)
(ev-poppler.cc:292)
==16259==    by 0x808BEA8: ev_document_get_links (ev-document.c:198)
==16259==    by 0x8062000: ev_job_render_run (ev-jobs.c:300)

in glib/poppler-page.c:poppler_page_get_link_mapping 

        links = new Links (page->page->getAnnots (&obj),
                           page->document->doc->getCatalog ()->getBaseURI ());

but |links| is never deleted.
Comment 1 Henning Norén 2006-05-08 08:46:50 UTC 
Created attachment 5569 [details] [review]
Proposed patch

Proposed patch with naive fix. Seems to fix the specific memory leak though.
Please verify it's correctness.
Comment 2 Kristian Høgsberg 2006-05-20 08:42:28 UTC 

*** This bug has been marked as a duplicate of 6947 ***

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.