]> granicus.if.org Git - libvpx/commit
projects / libvpx / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 2d0f3b2) | patch
Don't collect stats if they won't be used
authorJorge E. Moreira <jemoreira@google.com>
Fri, 8 May 2020 22:14:48 +0000 (15:14 -0700)
committerJorge E. Moreira <jemoreira@google.com>
Fri, 8 May 2020 22:30:26 +0000 (15:30 -0700)
commite8b818b581b9ab7e9c8f7b1becd753dd4efbdf43
tree3259e5690fb323481541cf75df42c247ab16370ftree | snapshot
parent2d0f3b23a908bb54dbeddf6d1dcc666600470e2ecommit | diff
Don't collect stats if they won't be used

When the encoder is run continuously for a few minutes at 60 fps, the
total_target_vs_actual field overflows. Since this field is a signed
integer that's considered undefined behavior in C++, which causes an
abort when used in an android binary (those run with ubsan enabled)

Bug: b/154172422
Test: run cuttelfish with webrtc enabled for an hour
Change-Id: I8f7d9d0884311a6338bdcdec76348b8cc3ce8c69
vp8/encoder/onyx_if.c diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.