New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 796426 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Email to this user bounced
Closed: Dec 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug



Sign in to add a comment

Null-dereference READ in sh::TParseContext::addStructDeclaratorList

Project Member Reported by ClusterFuzz, Dec 20 2017

Issue description

Detailed report: https://clusterfuzz.com/testcase?key=5147581956227072

Fuzzer: libFuzzer_angle_translator_fuzzer
Job Type: libfuzzer_chrome_ubsan
Platform Id: linux

Crash Type: Null-dereference READ
Crash Address: 0x000000000000
Crash State:
  sh::TParseContext::addStructDeclaratorList
  sh::TParseContext::addStructDeclaratorListWithQualifiers
  yyparse
  
Sanitizer: undefined (UBSAN)

Regressed: https://clusterfuzz.com/revisions?job=libfuzzer_chrome_ubsan&range=525039:525066

Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=5147581956227072

Issue filed automatically.

See https://chromium.googlesource.com/chromium/src/+/master/testing/libfuzzer/reference.md for more information.
 
Project Member

Comment 1 by ClusterFuzz, Dec 20 2017

Labels: Test-Predator-Auto-Owner
Owner: oetu...@nvidia.com
Status: Assigned (was: Untriaged)
Automatically assigning owner based on suspected regression changelist https://chromium.googlesource.com/angle/angle/+/ae4dbf32ba112c767d6e67d6e5734fc81d8c2d8c (Don't allocate name strings for empty symbols).

If this is incorrect, please remove the owner and apply the Test-Predator-Wrong-CLs label.
Project Member

Comment 2 by ClusterFuzz, Dec 20 2017

ClusterFuzz has detected this issue as fixed in range 525188:525204.

Detailed report: https://clusterfuzz.com/testcase?key=5147581956227072

Fuzzer: libFuzzer_angle_translator_fuzzer
Job Type: libfuzzer_chrome_ubsan
Platform Id: linux

Crash Type: Null-dereference READ
Crash Address: 0x000000000000
Crash State:
  sh::TParseContext::addStructDeclaratorList
  sh::TParseContext::addStructDeclaratorListWithQualifiers
  yyparse
  
Sanitizer: undefined (UBSAN)

Regressed: https://clusterfuzz.com/revisions?job=libfuzzer_chrome_ubsan&range=525039:525066
Fixed: https://clusterfuzz.com/revisions?job=libfuzzer_chrome_ubsan&range=525188:525204

Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=5147581956227072

See https://chromium.googlesource.com/chromium/src/+/master/testing/libfuzzer/reference.md for more information.

If you suspect that the result above is incorrect, try re-doing that job on the test case report page.
Project Member

Comment 3 by ClusterFuzz, Dec 20 2017

Labels: ClusterFuzz-Verified
Status: Verified (was: Assigned)
ClusterFuzz testcase 5147581956227072 is verified as fixed, so closing issue as verified.

If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue.

Comment 4 by oetu...@nvidia.com, Dec 20 2017

Cc: jmad...@chromium.org
Jamie, any idea what might have happened with this bug? When I first saw this I was worried I could have introduced a null pointer dereference with the "don't allocate name strings for empty symbols" patch, but I didn't find any issues looking through the code. Could this have been just the tool malfunctioning?
It's not out of the question, but I suspect if the crash was reproducible it would be real. Maybe you fixed the crash with an unrelated change, or the code was reorganized so the crash would show up in a different place?

Sign in to add a comment