Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(334)

Issue 1727233005: Disable tests failing under UBSan to enable deployment to main waterfall. (Closed)

Created:
4 years, 10 months ago by kjellander_webrtc
Modified:
4 years, 10 months ago
CC:
webrtc-reviews_webrtc.org
Base URL:
https://chromium.googlesource.com/external/webrtc.git@master
Target Ref:
refs/pending/heads/master
Project:
webrtc
Visibility:
Public.

Description

Disable tests failing under UBSan to enable deployment to main waterfall. modules_unittests: https://build.chromium.org/p/client.webrtc.fyi/builders/Linux%20UBSan/builds/1138/steps/modules_unittests/logs/stdio [ RUN ] ByteIoTest.Test64SBitBigEndian ../../webrtc/modules/rtp_rtcp/source/byte_io_unittest.cc:34:33: runtime error: shift exponent 64 is too large for 64-bit type 'long' rtc_unittests: https://build.chromium.org/p/client.webrtc.fyi/builders/Linux%20UBSan/builds/1138/steps/rtc_unittests/logs/stdio [ RUN ] IPAddressTest.TestCountIPMaskBits ../../webrtc/base/ipaddress.cc:415:20: runtime error: negation of -2147483648 cannot be represented in type 'int32_t' (aka 'int'); cast to an unsigned type to negate this value to itself [ RUN ] BandwidthSmootherTest.TestSampleRollover ../../webrtc/base/rollingaccumulator.h:73:22: runtime error: signed integer overflow: 2147483647 * 2147483647 cannot be represented in type 'int' [ RUN ] RandomNumberGeneratorTest.UniformSignedInterval ../../webrtc/base/random_unittest.cc:121:50: runtime error: signed integer overflow: 2147483647 - -2147483648 cannot be represented in type 'int' rtc_media_unittests: https://build.chromium.org/p/client.webrtc.fyi/builders/Linux%20UBSan/builds/1138/steps/rtc_media_unittests/logs/stdio [ RUN ] VideoCommonTest.TestComputeScaleWithHighFps ../../webrtc/media/base/videocommon.cc:75:34: runtime error: signed integer overflow: 2621440 - -2147483648 cannot be represented in type 'int' BUG=webrtc:5487, webrtc:5490, webrtc:5491 NOTRY=True R=pbos@webrtc.org TBR=henrika@webrtc.org Committed: https://crrev.com/a2644c06ee28c97ab670d59c416e59b8325fc678 Cr-Commit-Position: refs/heads/master@{#11764}

Patch Set 1 #

Patch Set 2 : Adding src:*/webrtc/base/random.cc #

Patch Set 3 : Disabled only failing tests instead of blacklisting #

Unified diffs Side-by-side diffs Delta from patch set Stats (+56 lines, -8 lines) Patch
M webrtc/base/bandwidthsmoother_unittest.cc View 1 2 1 chunk +7 lines, -1 line 0 comments Download
M webrtc/base/ipaddress_unittest.cc View 1 2 1 chunk +7 lines, -1 line 0 comments Download
M webrtc/base/random_unittest.cc View 1 2 1 chunk +7 lines, -1 line 0 comments Download
M webrtc/media/base/videocommon_unittest.cc View 1 2 1 chunk +7 lines, -1 line 0 comments Download
M webrtc/modules/rtp_rtcp/source/byte_io_unittest.cc View 1 2 2 chunks +28 lines, -4 lines 0 comments Download

Messages

Total messages: 17 (9 generated)
kjellander_webrtc
It's green!
4 years, 10 months ago (2016-02-25 10:21:16 UTC) #3
pbos-webrtc
On 2016/02/25 10:21:16, kjellander (webrtc) wrote: > It's green! If these only trigger in single ...
4 years, 10 months ago (2016-02-25 11:49:19 UTC) #4
kjellander_webrtc
On 2016/02/25 11:49:19, pbos-webrtc wrote: > On 2016/02/25 10:21:16, kjellander (webrtc) wrote: > > It's ...
4 years, 10 months ago (2016-02-25 13:06:52 UTC) #8
pbos-webrtc
lgtm, thanks
4 years, 10 months ago (2016-02-25 13:08:52 UTC) #9
kjellander_webrtc
TBRing henrika for root owners
4 years, 10 months ago (2016-02-25 13:22:43 UTC) #11
kjellander_webrtc
Committed patchset #3 (id:60001) manually as a2644c06ee28c97ab670d59c416e59b8325fc678 (presubmit successful).
4 years, 10 months ago (2016-02-25 13:23:33 UTC) #14
commit-bot: I haz the power
Patchset 3 (id:??) landed as https://crrev.com/a2644c06ee28c97ab670d59c416e59b8325fc678 Cr-Commit-Position: refs/heads/master@{#11764}
4 years, 10 months ago (2016-02-25 13:23:35 UTC) #16
henrika_webrtc
4 years, 10 months ago (2016-02-25 13:30:19 UTC) #17
Message was sent while issue was closed.
LGTM

Powered by Google App Engine
This is Rietveld 408576698