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

Issue 676275 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

Float-cast-overflow in blink::formatChromiumMediaControlsTime

Project Member Reported by ClusterFuzz, Dec 21 2016

Issue description

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

Fuzzer: inferno_twister
Job Type: linux_ubsan_chrome
Platform Id: linux

Crash Type: Float-cast-overflow
Crash Address: 
Crash State:
  blink::formatChromiumMediaControlsTime
  blink::LayoutTheme::formatMediaControlsCurrentTime
  blink::MediaControls::updateCurrentTimeDisplay
  
Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_ubsan_chrome&range=435261:438085

Minimized Testcase (0.27 Kb):
Download: https://cluster-fuzz.appspot.com/download/AMIfv97aQPvnDa6RJQkOvT7LXmPMTaKP39WzZ2-nlpzNnJEJewqXoWup7RpEdWrhtNRXoJJbSZ0AlZSIDayD5WfskGf3rqIdclH7Frim1H5Hj2xgkkC96ZPFCbBxPHiaFY01FkZbttpqlnVCdS8yEvAYQltZrHHLWQ?testcase_id=5111080297955328
<video id=video1><script>
var V = document.getElementById('video1');
var scriptStrs = ['V.currentTime += 16362579531048844855',
'V.play()'];
var index = 0; function execute() {; try { eval(scriptStrs[index++]); } catch(e) {}}
setTimeout('setInterval(execute, 17)');</script>


Additional requirements: Requires HTTP

Issue filed automatically.

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
 
Cc: msrchandra@chromium.org
Components: Blink>HTML
Labels: Test-Predator-Correct-CLs
Owner: mlamouri@chromium.org
Status: Assigned (was: Untriaged)
Assigning to the concern owner from find it results --
The result is a list of CLs that change the crashed files. 

Author: mlamouri
Project: chromium
Changelist: https://chromium.googlesource.com/chromium/src/+/a069837e9fe584243f357ca535ca0bebd0b7de85
Time: Mon Dec 05 16:43:56 2016
Files HTMLMediaElement.cpp, MediaControls.cpp are changed in this cl (and is part of stack frame #4, "blink::HTMLMediaElement::invokeResourceSelectionAlgorithm"; frame #5, "blink::HTMLMediaElement::playInternal"; frame #6, "blink::HTMLMediaElement::play")
Minimum distance from crash line to modified line: 3. (file: MediaControls.cpp, crashed on: 345, modified: 342). 

Author: zqzhang
Project: chromium
Changelist: https://chromium.googlesource.com/chromium/src/+/d054c278da7a25db1c4646bd7e61dcb5f051bceb
Time: Fri Dec 09 23:18:59 2016
File HTMLMediaElement.cpp is changed in this cl (and is part of stack frame #4, "blink::HTMLMediaElement::invokeResourceSelectionAlgorithm"; frame #5, "blink::HTMLMediaElement::playInternal"; frame #6, "blink::HTMLMediaElement::play")
Minimum distance from crash line to modified line: 6. (file: HTMLMediaElement.cpp, crashed on: 2232, modified: 2226). 

Author: mlamouri
Project: chromium
Changelist: https://chromium.googlesource.com/chromium/src/+/dbe9b60586d307549e435135dceef0f4cf231dad
Time: Mon Dec 05 17:39:28 2016
File MediaControls.cpp is changed in this cl (and is part of stack frame #2, "blink::MediaControls::updateCurrentTimeDisplay"; frame #3, "blink::MediaControls::reset")
Minimum distance from crash line to modified line: 41. (file: MediaControls.cpp, crashed on: 491, modified: 532). 

Author: foolip
Project: chromium
Changelist: https://chromium.googlesource.com/chromium/src/+/e0f53cb2304b26339df19a65b40137651a3346b1
Time: Mon Dec 12 12:46:50 2016
Files HTMLMediaElement.cpp, MediaControls.cpp are changed in this cl (and is part of stack frame #4, "blink::HTMLMediaElement::invokeResourceSelectionAlgorithm"; frame #5, "blink::HTMLMediaElement::playInternal"; frame #6, "blink::HTMLMediaElement::play")
Minimum distance from crash line to modified line: 42. (file: MediaControls.cpp, crashed on: 491, modified: 533). 

Author: zqzhang
Project: chromium
Changelist: https://chromium.googlesource.com/chromium/src/+/87b9e3b5bcf706b2a3313bfc44606325614b9eac
Time: Wed Nov 30 21:33:53 2016
File HTMLMediaElement.cpp is changed in this cl (and is part of stack frame #4, "blink::HTMLMediaElement::invokeResourceSelectionAlgorithm"; frame #5, "blink::HTMLMediaElement::playInternal"; frame #6, "blink::HTMLMediaElement::play")
Minimum distance from crash line to modified line: 65. (file: HTMLMediaElement.cpp, crashed on: 2232, modified: 2167).

@mlamouri -- Could you please look into the issue, kindly re-assign if this is not related to your changes.
Thank You.
Project Member

Comment 2 by ClusterFuzz, Dec 22 2016

ClusterFuzz has detected this issue as fixed in range 440242:440280.

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

Fuzzer: inferno_twister
Job Type: linux_ubsan_chrome
Platform Id: linux

Crash Type: Float-cast-overflow
Crash Address: 
Crash State:
  blink::formatChromiumMediaControlsTime
  blink::LayoutTheme::formatMediaControlsCurrentTime
  blink::MediaControls::updateCurrentTimeDisplay
  
Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_ubsan_chrome&range=435261:438085
Fixed: https://cluster-fuzz.appspot.com/revisions?job=linux_ubsan_chrome&range=440242:440280

Minimized Testcase (0.27 Kb):
Download: https://cluster-fuzz.appspot.com/download/AMIfv97aQPvnDa6RJQkOvT7LXmPMTaKP39WzZ2-nlpzNnJEJewqXoWup7RpEdWrhtNRXoJJbSZ0AlZSIDayD5WfskGf3rqIdclH7Frim1H5Hj2xgkkC96ZPFCbBxPHiaFY01FkZbttpqlnVCdS8yEvAYQltZrHHLWQ?testcase_id=5111080297955328
<video id=video1><script>
var V = document.getElementById('video1');
var scriptStrs = ['V.currentTime += 16362579531048844855',
'V.play()'];
var index = 0; function execute() {; try { eval(scriptStrs[index++]); } catch(e) {}}
setTimeout('setInterval(execute, 17)');</script>


Additional requirements: Requires HTTP

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs 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 3 by ClusterFuzz, Dec 22 2016

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

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

Sign in to add a comment