Closed Bug 1243482 (CVE-2016-2792) Opened 9 years ago Closed 9 years ago

graphite2: heap-buffer-overflow read in [@graphite2::Slot::getAttr] Slot.cpp:232

Categories

(Core :: Graphics: Text, defect)

Product:
Core
Component:
Graphics: Text
Type:
defect
Priority:
Not set
Severity:
critical

Tracking

()

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
firefox45 --- fixed
firefox46 --- fixed
firefox47 --- fixed
firefox-esr38 --- fixed
firefox-esr45 45+ ---

People

(Reporter: tsmith, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: csectype-bounds, sec-high, testcase, Whiteboard: [adv-main45+][adv-esr38.7+])

Attachments

(3 files, 2 obsolete files)

test_string.txt
548 bytes, text/plain
Details
test_case.ttf
55.94 KB, application/x-font-ttf
Details
call_stack.txt
6.08 KB, text/plain
Details
Attached file call_stack.txt (obsolete) —
This was found while fuzzing graphite2 1.3.5 (and is in 1.3.4)
Attached file test_case.ttf (obsolete) —
What is the test string for this: "Whereas" seems to render correctly.
Flags: needinfo?(twsmith)
Attached file test_string.txt —
Flags: needinfo?(twsmith)
Keywords: sec-high
fixed upstream in a8b3ac2aed0eb132cd80efe7de88f8153e73c829
This issue is reproducible in latest revision (e569e28d83491fedb31b9220493f3c07f6ec6d80)
Flags: needinfo?(martin_hosken)
Attached file test_case.ttf —
Attachment #8712796 - Attachment is obsolete: true
Attached file call_stack.txt —
Attachment #8712795 - Attachment is obsolete: true
fixed upstream in a8f9210571f5339d55c7fd4524acb64ce5ca8fd8
Flags: needinfo?(martin_hosken)
This issue is reproducible in the latest revision (df41ce06dda5962b9ff1c8c3175af00005d5fc0f)
Flags: needinfo?(martin_hosken)
Sorry. Didn't notice the new test_case.ttf that was introduced before my last claim to have fixed it. Is this new test_case effectively a new bug? Anyway fixed? in latest upstream and I think in 234c9634b76d31a597e048aa8596996dbdc85165
Flags: needinfo?(martin_hosken)
Summary: graphite2: heap-buffer-overflow read in [@graphite2::Slot::getAttr] → graphite2: heap-buffer-overflow read in [@graphite2::Slot::getAttr] Slot.cpp:232
See Also: → CVE-2016-2800
Verified existing test cases with aed0effc27edfb9da441dce3c77f5a1a3fd9f7db. I found a crash with a matching stack trace and logged it a bug 1249338.
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox45: --- → fixed
status-firefox46: --- → fixed
status-firefox47: --- → fixed
status-firefox-esr38: --- → fixed
Depends on: 1252311
Resolution: --- → FIXED
tracking-firefox-esr45: --- → 45+
Whiteboard: [adv-main45+][adv-esr38.7+]
Group: gfx-core-security → core-security-release
Alias: CVE-2016-2792
Group: core-security-release
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: