Issue metadata
Sign in to add a comment
|
Deadlock in libFuzzer |
||||||||||||||||||||||||
Issue descriptionDavid noticed a deadlock in libFuzzer in Chrome OS. The deadlock seems to have been fixed in upstream. If this is the case, then the compiler-rt version should be rolled or the fixes backported. Otherwise this deadlock should be fixed. Attatched is the file that causes the deadlock on virglrenderer_fuzzer. To repro you need to run the fuzzer in a loop with this input, since it does not deterministically reproduce.
,
Jul 23
Yunlian, Cqan you cherry-pick r331825 to llvm and compiler-rt?
,
Jul 23
This looks like a UBSan+OOM variant of https://github.com/google/sanitizers/issues/788. I the cherry-pick doesn't fix this, we probably need to fix this in UBSan with something similar to https://reviews.llvm.org/D46277.
,
Jul 24
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/5ce408c2f75c79e6fb9dba767f6bf67e91b6651b commit 5ce408c2f75c79e6fb9dba767f6bf67e91b6651b Author: Yunlian Jiang <yunlian@google.com> Date: Tue Jul 24 11:11:25 2018 llvm: fix deadlock in libFuzzer. This backports an upstream compiler-rt patch to fix the deadlock in libFuzzer. BUG= chromium:866546 TEST=sudo emerge llvm Change-Id: Icb79b89da16c99ba2c5cfa464e8062d35a1ee98b Reviewed-on: https://chromium-review.googlesource.com/1147840 Commit-Ready: Yunlian Jiang <yunlian@chromium.org> Tested-by: Yunlian Jiang <yunlian@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org> [add] https://crrev.com/5ce408c2f75c79e6fb9dba767f6bf67e91b6651b/sys-devel/llvm/files/cherry/1b2e8fb0ce651da2d47b98c33b29c0f42a080f13.patch [add] https://crrev.com/5ce408c2f75c79e6fb9dba767f6bf67e91b6651b/sys-libs/compiler-rt/files/cherry/1b2e8fb0ce651da2d47b98c33b29c0f42a080f13.patch [rename] https://crrev.com/5ce408c2f75c79e6fb9dba767f6bf67e91b6651b/sys-devel/llvm/llvm-7.0_pre331547_p20180529-r8.ebuild [rename] https://crrev.com/5ce408c2f75c79e6fb9dba767f6bf67e91b6651b/sys-libs/compiler-rt/compiler-rt-7.0_pre331547-r4.ebuild
,
Jul 30
,
Jul 30
This should have been fixed by the cherry-pick + llvm update in https://chromium-review.googlesource.com/1147840. Please re-open if that is not the case.
,
Jul 30
Looks fixed from my tests. |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by davidri...@chromium.org
, Jul 23