New issue
Advanced search Search tips

Issue 677820 link

Starred by 1 user

Issue metadata

Status: Verified
Owner: ----
Closed: Jan 2017
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 3
Type: Bug



Sign in to add a comment

Heap-use-after-free in base::ObserverListBase<gpu::CommandBufferProxyImpl::DeletionObserver>::begin

Project Member Reported by ClusterFuzz, Jan 2 2017

Issue description

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

Fuzzer: inferno_twister
Job Type: windows_syzyasan_content_shell
Platform Id: windows

Crash Type: Heap-use-after-free READ 4
Crash Address: 0x2c078497
Crash State:
  base::ObserverListBase<gpu::CommandBufferProxyImpl::DeletionObserver>::begin
  device::MockBluetoothGattNotifySession::DoNotify
  base::internal::Invoker<base::internal::BindState<void
  
Recommended Security Severity: High

Regressed: https://cluster-fuzz.appspot.com/revisions?job=windows_syzyasan_content_shell&range=441048:441049

Minimized Testcase (0.65 Kb):
Download: https://cluster-fuzz.appspot.com/download/AMIfv97Lw6hdsFXT1VRadLFyEXwM0lEBBRxfdAda73V8JPLsGWJipra1RH-IqLZOL7yHe37IGVP-vaCn6xul1aYl3P2CvqSZ-UduZoNeX8m2MeFAflHrBxlM_t4eFwzjZE03yj9LaIYiN-oEgtXWg_FcKLvHoRexDgubaVeWIK9VLxPh9o-_YmAHKWLQHfE12NCr94jeU45MOxIdRWVEjGHVxSpTB1eBaL1B7tVz65sKhADZ7G_IO3f67UxWYcOSCWtxPZJcXTzKaoESnTz2X6zK1Uo1_yMb5Zt1jN_IDsb-cwKjPldlYM2DZpG9Zpm6KzOP1MSvFnKn_4WT78e01ahkPi3WwwePbUfGN6Gh1RsQWAbzJ20D6ew?testcase_id=5702006426828800
  &#x95320;&#x821;<script src=../../resources/testharness.js></script>
<script src=../../resources/testharnessreport.js></script>
<script src=../../resources/bluetooth/bluetooth-helpers.js></script>
<script>
promise_test(() => {
  return setBluetoothFakeAdapter('HeartRateAdapter')
    .then(() => requestDeviceWithKeyDown({
      filters: [{services: ['heart_rate']}]}))
    .then(device => device.gatt.connect())
    .then(gattServer => gattServer.getPrimaryService('heart_rate'))
    .then(service => service.getCharacteristic('heart_rate_measurement'))
    .then(characteristic => {
      return characteristic.startNotifications()
    });
});
</script>


Additional requirements: Requires HTTP

Issue filed automatically.

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
 

Comment 1 by kenrb@chromium.org, Jan 2 2017

Cc: mbarbe...@chromium.org
mbarbella@: There are test-only classes in the call stack. I'm unclear about the details of the build being tested, but is it safe to close this bug?
Project Member

Comment 2 by sheriffbot@chromium.org, Jan 3 2017

Labels: M-57
Project Member

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

Labels: Pri-1
Cc: -mbarbe...@chromium.org
Labels: -Type-Bug-Security -Pri-1 -ReleaseBlock-Beta -Restrict-View-SecurityTeam -Security_Impact-Head -Security_Severity-High -M-57 Pri-3 Type-Bug
Might as well keep it open since it would be nice if this didn't crash, but certainly not a security bug (or above Pri-3).
Project Member

Comment 6 by ClusterFuzz, Jan 16 2017

ClusterFuzz has detected this issue as fixed in range 443594:443650.

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

Fuzzer: inferno_twister
Job Type: windows_syzyasan_content_shell
Platform Id: windows

Crash Type: Heap-use-after-free READ 4
Crash Address: 0x2c078497
Crash State:
  base::ObserverListBase<gpu::CommandBufferProxyImpl::DeletionObserver>::begin
  device::MockBluetoothGattNotifySession::DoNotify
  base::internal::Invoker<base::internal::BindState<void
  
Memory Tool: SYZYASAN

Recommended Security Severity: High

Regressed: https://cluster-fuzz.appspot.com/revisions?job=windows_syzyasan_content_shell&range=441048:441049
Fixed: https://cluster-fuzz.appspot.com/revisions?job=windows_syzyasan_content_shell&range=443594:443650

Minimized Testcase (0.65 Kb):
Download: https://cluster-fuzz.appspot.com/download/AMIfv97Lw6hdsFXT1VRadLFyEXwM0lEBBRxfdAda73V8JPLsGWJipra1RH-IqLZOL7yHe37IGVP-vaCn6xul1aYl3P2CvqSZ-UduZoNeX8m2MeFAflHrBxlM_t4eFwzjZE03yj9LaIYiN-oEgtXWg_FcKLvHoRexDgubaVeWIK9VLxPh9o-_YmAHKWLQHfE12NCr94jeU45MOxIdRWVEjGHVxSpTB1eBaL1B7tVz65sKhADZ7G_IO3f67UxWYcOSCWtxPZJcXTzKaoESnTz2X6zK1Uo1_yMb5Zt1jN_IDsb-cwKjPldlYM2DZpG9Zpm6KzOP1MSvFnKn_4WT78e01ahkPi3WwwePbUfGN6Gh1RsQWAbzJ20D6ew?testcase_id=5702006426828800
  &#x95320;&#x821;<script src=../../resources/testharness.js></script>
<script src=../../resources/testharnessreport.js></script>
<script src=../../resources/bluetooth/bluetooth-helpers.js></script>
<script>
promise_test(() => {
  return setBluetoothFakeAdapter('HeartRateAdapter')
    .then(() => requestDeviceWithKeyDown({
      filters: [{services: ['heart_rate']}]}))
    .then(device => device.gatt.connect())
    .then(gattServer => gattServer.getPrimaryService('heart_rate'))
    .then(service => service.getCharacteristic('heart_rate_measurement'))
    .then(characteristic => {
      return characteristic.startNotifications()
    });
});
</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 7 by ClusterFuzz, Jan 16 2017

Labels: ClusterFuzz-Verified
Status: Verified (was: Untriaged)
ClusterFuzz testcase 5702006426828800 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