New issue
Advanced search Search tips

Issue 698502 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 697191
Owner:
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug-Security



Sign in to add a comment

Use-of-uninitialized-value in v8::internal::wasm::LEBHelper::write_i32v

Project Member Reported by ClusterFuzz, Mar 4 2017

Issue description

Detailed report: https://cluster-fuzz.appspot.com/testcase?key=5472428240404480

Fuzzer: libfuzzer_v8_wasm_compile_fuzzer
Job Type: libfuzzer_chrome_msan
Platform Id: linux

Crash Type: Use-of-uninitialized-value
Crash Address: 
Crash State:
  v8::internal::wasm::LEBHelper::write_i32v
  v8::internal::wasm::WasmFunctionBuilder::EmitVarInt
  v8::internal::wasm::WasmFunctionBuilder::EmitWithVarInt
  
Sanitizer: memory (MSAN)

Recommended Security Severity: Medium

Regressed: https://cluster-fuzz.appspot.com/revisions?job=libfuzzer_chrome_msan&range=454233:454247

Reproducer Testcase: https://cluster-fuzz.appspot.com/download/AMIfv94bnyBM6c15KcrsUl0wrEmEQ_mqDxZxGUXSVhkI_PleBOhlDhxvmo3FQDv4KDG2UU6vRGHPZjqenxB0LZ2apw1556zxL3j9Vl00pD0ICT9xbiUA3iPmhy-r_2nfmFQFE-RyhIFwg40eTojJBv9DN1LeUhvODeDM6QoLJxznmH5Um1SzMePbtw4cTrTFd-m8tzeJTWQqiWV55WTpWz1oP7A1PdehBz0DBeN4yfCt1UoCd7yXY6lqazACQdYvi7oOreC8WyhvEanXCz_oFMWH68aUwcBgF0ouf0_HqoBoQ-QxCz9zTNpxGzJ7a_pMKZIk7kIFaYyt0PZhmfkTXb_mDPp96Xp7ulqIY_FJxD9wsMycA3POUAk?testcase_id=5472428240404480


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, Mar 4 2017

Labels: M-58
Project Member

Comment 2 by sheriffbot@chromium.org, Mar 4 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, Mar 4 2017

Labels: Pri-1

Comment 4 by vakh@chromium.org, Mar 5 2017

Cc: titzer@chromium.org
Components: Blink>JavaScript
Owner: hablich@chromium.org
Status: Available (was: Untriaged)
hablich@ -- please help triage this and find the right owner. Thanks.
Project Member

Comment 5 by sheriffbot@chromium.org, Mar 5 2017

Status: Assigned (was: Available)
Project Member

Comment 6 by ClusterFuzz, Mar 6 2017

ClusterFuzz has detected this issue as fixed in range 454791:454794.

Detailed report: https://cluster-fuzz.appspot.com/testcase?key=5472428240404480

Fuzzer: libfuzzer_v8_wasm_compile_fuzzer
Job Type: libfuzzer_chrome_msan
Platform Id: linux

Crash Type: Use-of-uninitialized-value
Crash Address: 
Crash State:
  v8::internal::wasm::LEBHelper::write_i32v
  v8::internal::wasm::WasmFunctionBuilder::EmitVarInt
  v8::internal::wasm::WasmFunctionBuilder::EmitWithVarInt
  
Sanitizer: memory (MSAN)

Recommended Security Severity: Medium

Regressed: https://cluster-fuzz.appspot.com/revisions?job=libfuzzer_chrome_msan&range=454233:454247
Fixed: https://cluster-fuzz.appspot.com/revisions?job=libfuzzer_chrome_msan&range=454791:454794

Reproducer Testcase: https://cluster-fuzz.appspot.com/download/AMIfv94bnyBM6c15KcrsUl0wrEmEQ_mqDxZxGUXSVhkI_PleBOhlDhxvmo3FQDv4KDG2UU6vRGHPZjqenxB0LZ2apw1556zxL3j9Vl00pD0ICT9xbiUA3iPmhy-r_2nfmFQFE-RyhIFwg40eTojJBv9DN1LeUhvODeDM6QoLJxznmH5Um1SzMePbtw4cTrTFd-m8tzeJTWQqiWV55WTpWz1oP7A1PdehBz0DBeN4yfCt1UoCd7yXY6lqazACQdYvi7oOreC8WyhvEanXCz_oFMWH68aUwcBgF0ouf0_HqoBoQ-QxCz9zTNpxGzJ7a_pMKZIk7kIFaYyt0PZhmfkTXb_mDPp96Xp7ulqIY_FJxD9wsMycA3POUAk?testcase_id=5472428240404480


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.
Cc: hablich@chromium.org
Labels: Merge-Request-58
Owner: ishell@chromium.org
Assigning to CF sheriff.

Please merge the fix to 58 when it is isolated.
Project Member

Comment 8 by ClusterFuzz, Mar 6 2017

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

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

Comment 9 by sheriffbot@chromium.org, Mar 6 2017

Labels: -Restrict-View-SecurityTeam Restrict-View-SecurityNotify
Project Member

Comment 10 by sheriffbot@chromium.org, Mar 7 2017

Labels: -Merge-Request-58 Hotlist-Merge-Approved Merge-Approved-58
Your change meets the bar and is auto-approved for M58. Please go ahead and merge the CL to branch 3029 manually. Please contact milestone owner if you have questions.
Owners: amineer@(clank), cmasso@(bling), bhthompson@(cros), govind@(desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Mergedinto: 697191
Status: Duplicate (was: Verified)
The fix seems to be fuzzer driver related: https://chromium-review.googlesource.com/448317

and it was not merged to M58 yet.
Project Member

Comment 12 by sheriffbot@chromium.org, Mar 10 2017

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Please merge your change to M58 branch 3029 before 5:00 PM PT, Monday (03/13/17) so we can take it in for next week dev release. Thank you!

Labels: -Merge-Approved-58 Merge-Merged
Merged in the original bug: https://bugs.chromium.org/p/chromium/issues/detail?id=697191#c20
Project Member

Comment 15 by sheriffbot@chromium.org, Jun 12 2017

Labels: -Restrict-View-SecurityNotify allpublic
This bug has been closed for more than 14 weeks. Removing security view restrictions.

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

Sign in to add a comment