Bug 1243482 (CVE-2016-2792)

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

RESOLVED FIXED

Status

()

defect
--
critical
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: tsmith, Unassigned)

Tracking

(Blocks 1 bug, {csectype-bounds, sec-high, testcase})

unspecified
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox45 fixed, firefox46 fixed, firefox47 fixed, firefox-esr38 fixed, firefox-esr4545+)

Details

(Whiteboard: [adv-main45+][adv-esr38.7+])

Attachments

(3 attachments, 2 obsolete attachments)

(Reporter)

Description

3 years ago
Posted file call_stack.txt (obsolete) —
This was found while fuzzing graphite2 1.3.5 (and is in 1.3.4)
(Reporter)

Comment 1

3 years ago
Posted file test_case.ttf (obsolete) —

Comment 2

3 years ago
What is the test string for this: "Whereas" seems to render correctly.
Flags: needinfo?(twsmith)
(Reporter)

Comment 3

3 years ago
Posted file test_string.txt
Flags: needinfo?(twsmith)
Keywords: sec-high

Comment 4

3 years ago
fixed upstream in a8b3ac2aed0eb132cd80efe7de88f8153e73c829
(Reporter)

Comment 5

3 years ago
This issue is reproducible in latest revision (e569e28d83491fedb31b9220493f3c07f6ec6d80)
Flags: needinfo?(martin_hosken)
(Reporter)

Comment 6

3 years ago
Posted file test_case.ttf
Attachment #8712796 - Attachment is obsolete: true
(Reporter)

Comment 7

3 years ago
Posted file call_stack.txt
Attachment #8712795 - Attachment is obsolete: true

Comment 8

3 years ago
fixed upstream in a8f9210571f5339d55c7fd4524acb64ce5ca8fd8

Updated

3 years ago
Flags: needinfo?(martin_hosken)
(Reporter)

Comment 9

3 years ago
This issue is reproducible in the latest revision (df41ce06dda5962b9ff1c8c3175af00005d5fc0f)
Flags: needinfo?(martin_hosken)

Comment 10

3 years ago
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

Updated

3 years ago
Flags: needinfo?(martin_hosken)
(Reporter)

Updated

3 years ago
Summary: graphite2: heap-buffer-overflow read in [@graphite2::Slot::getAttr] → graphite2: heap-buffer-overflow read in [@graphite2::Slot::getAttr] Slot.cpp:232
(Reporter)

Updated

3 years ago
See Also: → CVE-2016-2800
(Reporter)

Comment 11

3 years ago
Verified existing test cases with aed0effc27edfb9da441dce3c77f5a1a3fd9f7db.

I found a crash with a matching stack trace and logged it a bug 1249338.
(Reporter)

Updated

3 years ago
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Depends on: 1252311
Resolution: --- → FIXED
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.