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

Issue 590610 link

Starred by 0 users

Issue metadata

Status: Fixed
Owner:
Buried. Ping if important.
Closed: Feb 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug-Security



Sign in to add a comment

Bad-cast to const blink::WebPasswordCredential from blink::WebCredential;credential_manager_content_utils.cc:26:9

Project Member Reported by ClusterFuzz, Feb 28 2016

Issue description

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

Fuzzer: inferno_twister
Job Type: linux_ubsan_vptr_chrome
Platform Id: linux

Crash Type: Bad-cast
Crash Address: 0x7fffa85848c8
Crash State:
  Bad-cast to const blink::WebPasswordCredential from blink::WebCredential
  credential_manager_content_utils.cc:26:9
  

Minimized Testcase (0.10 Kb):
Download: https://cluster-fuzz.appspot.com/download/AMIfv94qpq9vvcxNGGgx3038U9GG9XAgZSn3hnLl1sVG-XHGuN7gcpIAImQv-UqZ_kECus_ShFXSH0uBD3AVXwXUwk7u3dZx5DuBIlI2cBt5QtNi5sbsx1odQW3J9G3tKyIP8EosNIqYfjv80o9Qhhv1Ycsac7XPdg
<script>
var local = new PasswordCredential({
});
    navigator.credentials.store(local);
</script>


Additional requirements: Requires HTTP

Filer: inferno

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
 
Owner: mkwst@chromium.org
Status: Assigned (was: Available)

Comment 2 by mkwst@chromium.org, Feb 29 2016

https://codereview.chromium.org/1745963002 up for review to tighten up the checks for the constructor.

Comment 4 by och...@chromium.org, Feb 29 2016

Components: UI>Browser>Passwords
Status: Fixed (was: Assigned)
Marking as fixed based on #3. 

mkwst, mind updating the impact (stable/beta/head) on this bug? The crash was marked as Unreproducible on CF, so the "Head" impact may not be entirely accurate.

Comment 5 by mkwst@chromium.org, Mar 1 2016

Labels: -Unreproducible Merge-Request-50 M-50 Security_Impact-Beta
I think this was just barely in the beta we just cut. Requesting merge.

Comment 6 by tin...@google.com, Mar 1 2016

Labels: -Merge-Request-50 Merge-Approved-50 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M50 (branch: 2661)
Project Member

Comment 8 by ClusterFuzz, Mar 10 2016

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

Comment 9 by sheriffbot@chromium.org, Jun 7 2016

Labels: -Restrict-View-SecurityNotify
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
Project Member

Comment 10 by sheriffbot@chromium.org, Oct 1 2016

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

Comment 11 by sheriffbot@chromium.org, Oct 2 2016

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
Labels: allpublic
Project Member

Comment 13 by sheriffbot@chromium.org, Jul 28

Labels: Pri-1

Sign in to add a comment