Bug 6766 - Overrun local variable in PageLabelInfo::toRoman
Summary: Overrun local variable in PageLabelInfo::toRoman
Status: RESOLVED INVALID
Alias: None
Product: poppler
Classification: Unclassified
Component: general (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-04-27 18:35 UTC by Kjartan Maraas
Modified: 2006-04-29 08:58 UTC (History)
0 users

See Also:
i915 platform:
i915 features:

Attachments
possible patch (3.53 KB, patch)
2006-04-27 18:36 UTC, Kjartan Maraas 		Details | Splinter Review
View All

Description Kjartan Maraas 2006-04-27 18:35:25 UTC 
Coverity report #902
Comment 1 Kjartan Maraas 2006-04-27 18:36:09 UTC 
Created attachment 5498 [details] [review]
possible patch

Could this be the right fix?
Comment 2 Kjartan Maraas 2006-04-29 04:11:47 UTC 
s/runs/reports :-)
Comment 3 Albert Astals Cid 2006-04-30 01:58:12 UTC 
Your patch is wrong, i've checked and the code works up to 3999, accessing the 
7th element of the uppercaseNumerals lowercaseNumerals arrays is NOT wrong 
because there's a \0 there.

I'm closign the bug because AFAIK 4000 can not be represented with the roman 
notation i learnt in school.

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.