New issue
Advanced search Search tips

Issue 876252 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Sep 8
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug-Security



Sign in to add a comment

Use-of-uninitialized-value in v8::internal::Factory::NewNumber

Project Member Reported by ClusterFuzz, Aug 21

Issue description

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

Fuzzer: inferno_layout_test_unmodified
Job Type: linux_msan_chrome
Platform Id: linux

Crash Type: Use-of-uninitialized-value
Crash Address: 
Crash State:
  v8::internal::Factory::NewNumber
  v8::internal::LookupIterator::FetchValue
  v8::internal::Object::GetProperty
  
Sanitizer: memory (MSAN)

Recommended Security Severity: Medium

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

Issue filed automatically.

See https://github.com/google/clusterfuzz-tools for more information.
 
Project Member

Comment 1 by sheriffbot@chromium.org, Aug 21

Labels: Target-70 M-70
Project Member

Comment 2 by sheriffbot@chromium.org, Aug 21

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

If this doesn't affect a release branch, or has not been properly classified for severity, please update the Security_Impact or Security_Severity labels, and remove the ReleaseBlock label. To disable this altogether, apply ReleaseBlock-NA.

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

Comment 3 by sheriffbot@chromium.org, Aug 21

Labels: Pri-1
Components: Blink>JavaScript
Owner: yangguo@chromium.org
Status: Assigned (was: Untriaged)
yangguo -- if you are not the right person to own this, can you please help triage it better? Thanks.
Cc: ishell@chromium.org
Owner: mstarzinger@chromium.org
Assigning to this week's sheriff.
Project Member

Comment 6 by sheriffbot@chromium.org, Sep 4

mstarzinger: Uh oh! This issue still open and hasn't been updated in the last 14 days. This is a serious vulnerability, and we want to ensure that there's progress. Could you please leave an update with the current status and any potential blockers?

If you're not the right owner for this issue, could you please remove yourself as soon as possible or help us find the right one?

If the issue is fixed or you can't reproduce it, please close the bug. If you've started working on a fix, please set the status to Started.

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
Project Member

Comment 7 by sheriffbot@chromium.org, Sep 5

Labels: -Security_Impact-Head Security_Impact-Beta
Cc: mstarzinger@chromium.org
Owner: titzer@chromium.org
Reassigning to this week's sheriff.
Project Member

Comment 9 by ClusterFuzz, Sep 8

ClusterFuzz has detected this issue as fixed in range 589772:589773.

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

Fuzzer: inferno_layout_test_unmodified
Job Type: linux_msan_chrome
Platform Id: linux

Crash Type: Use-of-uninitialized-value
Crash Address: 
Crash State:
  v8::internal::Factory::NewNumber
  v8::internal::LookupIterator::FetchValue
  v8::internal::Object::GetProperty
  
Sanitizer: memory (MSAN)

Recommended Security Severity: Medium

Fixed: https://clusterfuzz.com/revisions?job=linux_msan_chrome&range=589772:589773

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

See https://github.com/google/clusterfuzz-tools 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 10 by ClusterFuzz, Sep 8

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

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

Comment 11 by sheriffbot@chromium.org, Sep 8

Labels: -Restrict-View-SecurityTeam Restrict-View-SecurityNotify
Looks like this was fixed by the chromite roll in https://chromium.googlesource.com/chromium/src/+/c62924a209ede743e5156632e3194d548df3ad92 - might we want to take this for M70?
Labels: Merge-Request-70
Project Member

Comment 14 by sheriffbot@chromium.org, Oct 2

Labels: -Merge-Request-70 Merge-Review-70 Hotlist-Merge-Review
This bug requires manual review: We are only 13 days from stable.
Please contact the milestone owner if you have questions.
Owners: benmason@(Android), kariahda@(iOS), geohsu@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
How safe is this merge overall?
Labels: -Target-70 -M-70 -Merge-Review-70 Target-71 M-71
Given the stage in the release, tracking for 71.
Labels: -ReleaseBlock-Stable
Project Member

Comment 18 by sheriffbot@chromium.org, Dec 15

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