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

Issue 725885 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: May 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug



Sign in to add a comment

Use-of-uninitialized-value in libglib-2.0.so.0

Project Member Reported by ClusterFuzz, May 24 2017

Issue description

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

Fuzzer: libfuzzer_stylesheet_contents_fuzzer
Job Type: libfuzzer_chrome_msan
Platform Id: linux

Crash Type: Use-of-uninitialized-value
Crash Address: 
Crash State:
  libglib-2.0.so.0
  g_main_context_new
  g_main_context_default
  
Sanitizer: memory (MSAN)

Recommended Security Severity: Medium

Regressed: https://clusterfuzz.com/revisions?job=libfuzzer_chrome_msan&range=474161:474211

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


Issue filed automatically.

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

Comment 1 by sheriffbot@chromium.org, May 24 2017

Labels: M-60
Project Member

Comment 2 by sheriffbot@chromium.org, May 24 2017

Labels: ReleaseBlock-Beta
This issue is a security regression. If you are not able to fix this quickly, please revert the change that introduced it.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 3 by sheriffbot@chromium.org, May 24 2017

Labels: Pri-1

Comment 4 by kenrb@chromium.org, May 24 2017

Cc: mmoroz@chromium.org
mmoroz@: The repro case on this is a single byte, and it is flagging initialization of the message pump. Is this an issue with the fuzzing setup?

Comment 5 by ta...@google.com, May 24 2017

Cc: mbarbe...@chromium.org infe...@chromium.org och...@chromium.org
Labels: -ReleaseBlock-Beta -M-60
Owner: ta...@google.com
Status: Assigned (was: Untriaged)
We've figured out the root cause.

I uploaded the testcase manually, so that it would work with the reproduce tool (though only with the download build).

See: https://clusterfuzz.com/v2/testcase-detail/5820193367654400

`/google/data/ro/teams/clusterfuzz-tools/releases/clusterfuzz reproduce 5820193367654400 --build download` reproduces.

But `/google/data/ro/teams/clusterfuzz-tools/releases/clusterfuzz reproduce 5820193367654400` doesn't reproduce.


------

ochang@ found that, for the downloaded build, the prebuilt instrumented libs are not linked properly, e.g. `libglib-2.0.so.0 => /lib/x86_64-linux-gnu/libglib-2.0.so.0 (0x00007f55f7674000)`

But, on the locally built build, the libs seem to be linked correctly, e.g. `libglib-2.0.so.0 => /usr/local/google/home/tanin/projects/chromium/src/out/clusterfuzz_5820193367654400_d8955f1c68c20bc97db9d800be8a13bcc6631b38/instrumented_libraries_prebuilt/msan/lib/libglib-2.0.so.0 (0x00007f876edc8000)`

This means that we need to file a bug to the buildbot team to make sure that the folder `msan/lib` is included in the zip file.

Comment 6 by kenrb@chromium.org, May 24 2017

Labels: -Type-Bug-Security -Reproducible -Restrict-View-SecurityTeam -Security_Severity-Medium -Security_Impact-Head Type-Bug
Project Member

Comment 7 by ClusterFuzz, May 26 2017

ClusterFuzz has detected this issue as fixed in range 474419:474818.

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

Fuzzer: libfuzzer_stylesheet_contents_fuzzer
Job Type: libfuzzer_chrome_msan
Platform Id: linux

Crash Type: Use-of-uninitialized-value
Crash Address: 
Crash State:
  libglib-2.0.so.0
  g_main_context_new
  g_main_context_default
  
Sanitizer: memory (MSAN)

Recommended Security Severity: Medium

Regressed: https://clusterfuzz.com/revisions?job=libfuzzer_chrome_msan&range=474161:474211
Fixed: https://clusterfuzz.com/revisions?job=libfuzzer_chrome_msan&range=474419:474818

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


See https://chromium.googlesource.com/chromium/src/+/master/testing/libfuzzer/reproducing.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, May 26 2017

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

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

Comment 9 by kenrb@chromium.org, May 29 2017

 Issue 727096  has been merged into this issue.

Comment 10 by mmoroz@google.com, May 30 2017

Thanks for the investigation, folks!

Tanin, did you or Oliver file a bug as per c#5?

Sign in to add a comment