Chromium Code Reviews
|
|
Chromium Code Reviews
Issue
1883223002:
Fix valgrind complaint on uninitialized value (Closed)
|
Created:
4 years, 8 months ago by Irfan Modified:
4 years, 8 months ago Reviewers:
stefan-webrtc CC:
webrtc-reviews_webrtc.org, tterriberry_mozilla.com, zhengzhonghou_agora.io, video-team_agora.io, mflodman Base URL:
https://chromium.googlesource.com/external/webrtc.git@master Target Ref:
refs/pending/heads/master Project:
webrtc Visibility:
Public. |
DescriptionFix valgrind complaint on uninitialized value
BUG=
Committed: https://crrev.com/4d7bc240b960f42d3de415fd33eb4f0af07c41bb
Cr-Commit-Position: refs/heads/master@{#12460}
Patch Set 1 #
Messages
Total messages: 21 (9 generated)
Description was changed from ========== Fix valgrind complaint on uninitialized value BUG= ========== to ========== Fix valgrind complaint on uninitialized value BUG= ==========
isheriff@chromium.org changed reviewers: + stefan@webrtc.org
When was this bug introduced? I wonder if this affects Chrome M51?
lgtm
The CQ bit was checked by isheriff@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1883223002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1883223002/1
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: android_arm64_rel on tryserver.webrtc (JOB_TIMED_OUT, no build URL)
The CQ bit was checked by isheriff@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1883223002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1883223002/1
The bug was introduced when I made that change like a month ago or something. It does not actually have any impact on the functionality.
On 2016/04/19 18:56:00, Irfan wrote: > The bug was introduced when I made that change like a month ago or something. It > does not actually have any impact on the functionality. Good to know, thanks!
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: android_arm64_rel on tryserver.webrtc (JOB_TIMED_OUT, no build URL)
The CQ bit was checked by isheriff@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1883223002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1883223002/1
Message was sent while issue was closed.
Description was changed from ========== Fix valgrind complaint on uninitialized value BUG= ========== to ========== Fix valgrind complaint on uninitialized value BUG= ==========
Message was sent while issue was closed.
Committed patchset #1 (id:1)
Message was sent while issue was closed.
Description was changed from ========== Fix valgrind complaint on uninitialized value BUG= ========== to ========== Fix valgrind complaint on uninitialized value BUG= Committed: https://crrev.com/4d7bc240b960f42d3de415fd33eb4f0af07c41bb Cr-Commit-Position: refs/heads/master@{#12460} ==========
Message was sent while issue was closed.
Patchset 1 (id:??) landed as https://crrev.com/4d7bc240b960f42d3de415fd33eb4f0af07c41bb Cr-Commit-Position: refs/heads/master@{#12460} |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
