New issue
Advanced search Search tips

Issue 877888 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Aug 28
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug



Sign in to add a comment

ASSERT: cv_record_elf->cv_signature == MD_CVINFOELF_SIGNATURE

Project Member Reported by ClusterFuzz, Aug 27

Issue description

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

Fuzzer: libFuzzer_minidump_fuzzer
Job Type: libfuzzer_chrome_asan_debug
Platform Id: linux

Crash Type: ASSERT
Crash Address: 
Crash State:
  cv_record_elf->cv_signature == MD_CVINFOELF_SIGNATURE
  google_breakpad::MinidumpModule::code_identifier
  google_breakpad::MinidumpModuleList::Read
  
Sanitizer: address (ASAN)

Regressed: https://clusterfuzz.com/revisions?job=libfuzzer_chrome_asan_debug&range=571005:571014

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

Issue filed automatically.

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

Comment 1 by ClusterFuzz, Aug 27

Components: Internals>CrashReporting
Labels: Test-Predator-Auto-Components
Automatically applying components based on crash stacktrace and information from OWNERS files.

If this is incorrect, please apply the Test-Predator-Wrong-Components label.
Project Member

Comment 2 by ClusterFuzz, Aug 27

Cc: thestig@chromium.org
Labels: Test-Predator-Auto-CC
Automatically adding ccs based on suspected regression changelists:

Only do Android-specific adjustments for Android minidumps. by thestig@chromium.org - https://chromium.googlesource.com/breakpad/breakpad/+/fac0e886c8dee6418fcd303c1186518cc51e064d

Refactor code out of MinidumpModuleList::Read(). by thestig@chromium.org - https://chromium.googlesource.com/breakpad/breakpad/+/d531e1b2ba94f3f06b3706eb1f245b329c1bf9d2

Ignore duplicate module list entries. by thestig@chromium.org - https://chromium.googlesource.com/breakpad/breakpad/+/79ba6a494fb2097b39f76fe6a4b4b4f407e32a02

If this is incorrect, please let us know why and apply the Test-Predator-Wrong-CLs label.
Cc: -thestig@chromium.org kkaluri@chromium.org
Labels: M-69
Owner: thestig@chromium.org
Status: Assigned (was: Untriaged)
Predator has provided 3 possible suspects which has same author, hence assigning it to thestig@

1. Only do Android-specific adjustments for Android minidumps. by thestig@chromium.org
2. Refactor code out of MinidumpModuleList::Read(). by thestig@chromium.org
3. Ignore duplicate module list entries. by thestig@chromium.org

thestig@ Could you please look into it.

Project Member

Comment 4 by bugdroid1@chromium.org, Aug 28

The following revision refers to this bug:
  https://chromium.googlesource.com/breakpad/breakpad/+/54fa71efbe50fb2b58096d871575b59e12edba6d

commit 54fa71efbe50fb2b58096d871575b59e12edba6d
Author: Lei Zhang <thestig@chromium.org>
Date: Tue Aug 28 17:41:06 2018

Swap MDCVInfoELF::cv_signature if needed.

BUG= chromium:877888 

Change-Id: Ie4437228dfc32619ce5feb1769ba1644a4ea8ca2
Reviewed-on: https://chromium-review.googlesource.com/1192963
Reviewed-by: Joshua Peraza <jperaza@chromium.org>

[modify] https://crrev.com/54fa71efbe50fb2b58096d871575b59e12edba6d/src/processor/minidump.cc

Project Member

Comment 5 by bugdroid1@chromium.org, Aug 28

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d4bc0b15b90288c0fc1b80d08eaf32d6a13e00d7

commit d4bc0b15b90288c0fc1b80d08eaf32d6a13e00d7
Author: Lei Zhang <thestig@chromium.org>
Date: Tue Aug 28 21:59:50 2018

Roll src/third_party/breakpad/breakpad/ 1459e5df7..54fa71efb (3 commits)

https://chromium.googlesource.com/breakpad/breakpad.git/+log/1459e5df74dd..54fa71efbe50

$ git log 1459e5df7..54fa71efb --date=short --no-merges --format='%ad %ae %s'
2018-08-28 thestig Swap MDCVInfoELF::cv_signature if needed.
2018-08-27 jperaza win: fix formatting after 21c98747
2018-08-27 jperaza win: replace a while loop with a for loop

Created with:
  roll-dep src/third_party/breakpad/breakpad

BUG= 877888 
TBR=mark@chromium.org

Change-Id: If8fef0541eb58d8c512c2ef924eac0573f54f81d
Reviewed-on: https://chromium-review.googlesource.com/1194734
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#586875}
[modify] https://crrev.com/d4bc0b15b90288c0fc1b80d08eaf32d6a13e00d7/DEPS

Status: Fixed (was: Assigned)
Project Member

Comment 7 by ClusterFuzz, Aug 29

ClusterFuzz has detected this issue as fixed in range 586874:586876.

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

Fuzzer: libFuzzer_minidump_fuzzer
Job Type: libfuzzer_chrome_asan_debug
Platform Id: linux

Crash Type: ASSERT
Crash Address: 
Crash State:
  cv_record_elf->cv_signature == MD_CVINFOELF_SIGNATURE
  google_breakpad::MinidumpModule::code_identifier
  google_breakpad::MinidumpModuleList::Read
  
Sanitizer: address (ASAN)

Regressed: https://clusterfuzz.com/revisions?job=libfuzzer_chrome_asan_debug&range=571005:571014
Fixed: https://clusterfuzz.com/revisions?job=libfuzzer_chrome_asan_debug&range=586874:586876

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

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 8 by ClusterFuzz, Aug 29

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

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

Sign in to add a comment