1252736 - graphite2: SEGV near NULL in [@graphite2::Pass::collisionShift]

Status: RESOLVED FIXED

(Core :: Graphics: Text, defect)

Description

[Tyson Smith [:tsmith]]

Attachment: test_case.ttf

Found in graphite2 revision bc5409c573aa9ecccacd18cf713021272998cd35

./gr2fonttest <test> -auto

==53955==ERROR: AddressSanitizer: SEGV on unknown address 0x00000000002c (pc 0x7f54939c8262 bp 0x7ffcc691ebb0 sp 0x7ffcc691e880 T0)
    #0 0x7f54939c8261 in graphite2::SlotCollision::flags() const /home/user/code/graphite/src/inc/Collider.h:83:5
    #1 0x7f54939c8261 in graphite2::Pass::collisionShift(graphite2::Segment*, int, graphite2::json*) const /home/user/code/graphite/src/Pass.cpp:751
    #2 0x7f54939c406c in graphite2::Pass::runGraphite(graphite2::vm::Machine&, graphite2::FiniteStateMachine&, bool) const /home/user/code/graphite/src/Pass.cpp:436:14
    #3 0x7f54939e354d in graphite2::Silf::runGraphite(graphite2::Segment*, unsigned char, unsigned char, int) const /home/user/code/graphite/src/Silf.cpp:423:21
    #4 0x7f5493998c26 in graphite2::Face::runGraphite(graphite2::Segment*, graphite2::Silf const*) const /home/user/code/graphite/src/Face.cpp:180:16
    #5 0x7f5493962172 in graphite2::Segment::runGraphite() /home/user/code/graphite/src/inc/Segment.h:97:45
    #6 0x7f5493962172 in (anonymous namespace)::makeAndInitialize(graphite2::Font const*, graphite2::Face const*, unsigned int, graphite2::FeatureVal const*, gr_encform, void const*, unsigned long, int) /home/user/code/graphite/src/gr_segment.cpp:46
    #7 0x7f5493962172 in gr_make_seg /home/user/code/graphite/src/gr_segment.cpp:105
    #8 0x4e7c87 in Parameters::testFileFont() const /home/user/code/graphite/gr2fonttest/gr2FontTest.cpp:684:20
    #9 0x4e8d09 in main /home/user/code/graphite/gr2fonttest/gr2FontTest.cpp:787:9
    #10 0x7f54935a4ec4 in __libc_start_main /build/eglibc-3GlaMS/eglibc-2.19/csu/libc-start.c:287
    #11 0x41a595 in _start (/home/user/Desktop/graphite/gr2fonttest+0x41a595)

AddressSanitizer can not provide additional info.
SUMMARY: AddressSanitizer: SEGV /home/user/code/graphite/src/inc/Collider.h:83:5 in graphite2::SlotCollision::flags() const

Comment 1

[martin_hosken]

fixed? upstream in 6ed4db227feb23cddecd2166806d632148dfa4fc

Comment 2

[Tyson Smith [:tsmith]]

Verified with graphite revision 520d76818052772d614e581dacea69499b912be6

Comment 3

[Daniel Veditz [:dveditz]]

Graphite2 has been updated on all affected branches including ESRs.