commit | 662484ae8b26971216f817caf3932869b564a24c | [log] [tgz] |
---|---|---|
author | Manoj Gupta <manojgupta@google.com> | Fri Aug 24 10:57:06 2018 -0700 |
committer | Manoj Gupta <manojgupta@chromium.org> | Sat Aug 25 14:52:17 2018 +0000 |
tree | a15406270f684ee1783225d5f4734020e8c2a2cb | |
parent | e8f26fb85a963c387bf00e399f444bae63ec8b0a [diff] |
bzip2: Fix an issue with shift caught by ubsan. Apply the patch already sent/applied in Gentoo. https://github.com/gentoo/gentoo/commit/66f614c51f017b0693f5aaeb5897db28ef3aff6c Patch details: Use unsigned 1 for shifting instead of signed 1. This fixed an issue with shift caught by undefined behavior sanitizer in clang. bzip2-1.0.6/blocksort.c:255:7 runtime error: left shift of 1 by 31 places cannot be represented in type 'int' BUG=chromium:877236 TEST=bzip2 builds. Change-Id: If9de5b5d5b1dd2f676b8921fe19cec4a6e8c4430 Reviewed-on: https://chromium-review.googlesource.com/1188822 Tested-by: Manoj Gupta <manojgupta@chromium.org> Reviewed-by: Mike Frysinger <vapier@chromium.org> Commit-Queue: Manoj Gupta <manojgupta@chromium.org>