Issue metadata
Sign in to add a comment
|
Heap-use-after-free in base::ObserverListBase<content::RenderThreadObserver>::RemoveObserver |
||||||||||||||||||||||||
Issue descriptionDetailed report: https://cluster-fuzz.appspot.com/testcase?key=5765792664911872 Fuzzer: attekett_dom_fuzzer Job Type: linux_asan_chrome_chromeos Platform Id: linux Crash Type: Heap-use-after-free READ 8 Crash Address: 0x6110000796c0 Crash State: base::ObserverListBase<content::RenderThreadObserver>::RemoveObserver content::ImageDownloaderImpl::~ImageDownloaderImpl content::ImageDownloaderImpl::~ImageDownloaderImpl Recommended Security Severity: Medium Unminimized Testcase: https://cluster-fuzz.appspot.com/download/AMIfv97XBdSQa4ic-T1XTMXXX8QDiUSjMLC_vXsEyzeyVwVYtTWfPilc8OLzP4QZxnasuXqSzCF7UY7YCq4jbMAKZ3cRF1BYgbnmSnJ4GaDbpcXqiUqp5QE74TPss__s5ztCqQT6ZQhGndhJXBVKfSfZpvxERID-OA?testcase_id=5765792664911872 Additional requirements: Requires Gestures Issue filed automatically. See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
,
Aug 29 2016
I think https://chromium.googlesource.com/chromium/src.git/+/d2ac649bc631233f4355860a011d3d30f985bb67 should fix it, than CF will auto-close this. If not closed within a day, we should look more into this, might be another bug.
,
Aug 29 2016
,
Aug 30 2016
,
Aug 30 2016
,
Sep 6 2016
Hi, I suppose this bug has been fixed. Would you please help to confirm whether this crash can still be observed? If not, I'd like to close this. Thanks.
,
Sep 6 2016
ClusterFuzz has detected this issue as fixed in range 416466:416526. Detailed report: https://cluster-fuzz.appspot.com/testcase?key=5765792664911872 Fuzzer: attekett_dom_fuzzer Job Type: linux_asan_chrome_chromeos Platform Id: linux Crash Type: Heap-use-after-free READ 8 Crash Address: 0x6110000796c0 Crash State: base::ObserverListBase<content::RenderThreadObserver>::RemoveObserver content::ImageDownloaderImpl::~ImageDownloaderImpl content::ImageDownloaderImpl::~ImageDownloaderImpl Recommended Security Severity: Medium Fixed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_chrome_chromeos&range=416466:416526 Unminimized Testcase: https://cluster-fuzz.appspot.com/download/AMIfv97XBdSQa4ic-T1XTMXXX8QDiUSjMLC_vXsEyzeyVwVYtTWfPilc8OLzP4QZxnasuXqSzCF7UY7YCq4jbMAKZ3cRF1BYgbnmSnJ4GaDbpcXqiUqp5QE74TPss__s5ztCqQT6ZQhGndhJXBVKfSfZpvxERID-OA?testcase_id=5765792664911872 Additional requirements: Requires Gestures 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.
,
Sep 6 2016
ClusterFuzz testcase is verified as fixed, closing issue. If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue.
,
Sep 7 2016
,
Sep 9 2016
,
Sep 10 2016
Your change meets the bar and is auto-approved for M54 (branch: 2840)
,
Sep 12 2016
Will pick commit d2ac649bc631233f4355860a011d3d30f985bb67 into M54 branch soon.
,
Sep 13 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/a64d2294bbe77cf839715bfefa407d61147e79d6 commit a64d2294bbe77cf839715bfefa407d61147e79d6 Author: Vaclav Brozek <vabr@chromium.org> Date: Tue Sep 13 10:15:30 2016 Forbid ImageDownloaderImpl from accessing RenderThread instance after message loop shutdown. This CL fixes the crash caused by following possible scenario: - the message loop is shutdown. - during its destructor it calls connection error on ImageDownloaderImpl. - the strong binding of ImageDownloaderImpl causes it to be destructed. - the destructor calls RenderThread::Get()->RemoveObserver(this), which is not valid any more. BUG= 640999 TBR=jochen@chromium.org Review-Url: https://codereview.chromium.org/2286603002 Cr-Commit-Position: refs/heads/master@{#414985} (cherry picked from commit d2ac649bc631233f4355860a011d3d30f985bb67) Review URL: https://codereview.chromium.org/2335173002 . Cr-Commit-Position: refs/branch-heads/2840@{#324} Cr-Branched-From: 1ae106dbab4bddd85132d5b75c670794311f4c57-refs/heads/master@{#414607} [modify] https://crrev.com/a64d2294bbe77cf839715bfefa407d61147e79d6/content/renderer/image_downloader/image_downloader_impl.cc
,
Sep 16 2016
,
Oct 10 2016
,
Oct 11 2016
,
Oct 12 2016
,
Oct 12 2016
,
Dec 14 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
,
Apr 25 2018
|
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by infe...@chromium.org
, Aug 29 2016Status: Assigned (was: Untriaged)