Bug 7924 - CairoOutputDev::currentFont used before initialization.
Summary: CairoOutputDev::currentFont used before initialization.
Status: RESOLVED FIXED
Alias: None
Product: poppler
Classification: Unclassified
Component: cairo backend (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-08-19 14:06 UTC by Pascal Terjan
Modified: 2006-09-04 18:22 UTC (History)
0 users

See Also:
i915 platform:
i915 features:

Attachments
A PDF provoking crash (45.38 KB, application/pdf)
2006-08-19 14:06 UTC, Pascal Terjan 		Details
Initialize currentFont to NULL (502 bytes, patch)
2006-08-19 14:07 UTC, Pascal Terjan 		Details | Splinter Review
View All

Description Pascal Terjan 2006-08-19 14:06:03 UTC 
Some methods (drawChar and endString) test that currentFont is not NULL but the
constructor does not set it to NULL. 
valgrind reports it to be read by endString before initialization on the
attached PDF on which I get a crash.
Comment 1 Pascal Terjan 2006-08-19 14:06:46 UTC 
Created attachment 6615 [details]
A PDF provoking crash
Comment 2 Pascal Terjan 2006-08-19 14:07:53 UTC 
Created attachment 6616 [details] [review]
Initialize currentFont to NULL
Comment 3 Pascal Terjan 2006-08-19 14:35:52 UTC 
This patch fixes the crash for me on this PDF and 5 others.
Comment 4 Jeff Muizelaar 2006-09-04 18:22:57 UTC 
Commited. Thanks.

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.