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

Issue 967723002: Cast: Fix crash in udp_transport (Closed)

Created:
5 years, 9 months ago by hubbe
Modified:
5 years, 7 months ago
Reviewers:
miu
CC:
chromium-reviews, hclam+watch_chromium.org, cbentzel+watch_chromium.org, imcheng+watch_chromium.org, hguihot+watch_chromium.org, jasonroberts+watch_google.com, avayvod+watch_chromium.org, pwestin+watch_google.com, feature-media-reviews_chromium.org, miu+watch_chromium.org, hubbe+watch_chromium.org, mikhal+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Cast: Fix crash in udp_transport When connecting to a port on localhost which is not yet open, connect fails and udp_socket crashes when trying to send packets. This fixes that. Committed: https://crrev.com/f90e67e8844bf3c40c1091bd843a3ee11c9bc1c3 Cr-Commit-Position: refs/heads/master@{#330442}

Patch Set 1 #

Patch Set 2 : missed one thing... #

Total comments: 2

Patch Set 3 : comments addressed #

Patch Set 4 : typ0 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -1 line) Patch
M media/cast/net/udp_transport.h View 1 2 1 chunk +1 line, -1 line 0 comments Download
M media/cast/net/udp_transport.cc View 1 2 3 6 chunks +13 lines, -0 lines 0 comments Download

Messages

Total messages: 9 (3 generated)
hubbe
5 years, 7 months ago (2015-05-07 20:04:56 UTC) #2
miu
lgtm % one consideration: https://codereview.chromium.org/967723002/diff/10002/media/cast/net/udp_transport.cc File media/cast/net/udp_transport.cc (right): https://codereview.chromium.org/967723002/diff/10002/media/cast/net/udp_transport.cc#newcode70 media/cast/net/udp_transport.cc:70: Should there be a guard ...
5 years, 7 months ago (2015-05-08 06:49:05 UTC) #3
hubbe
https://codereview.chromium.org/967723002/diff/10002/media/cast/net/udp_transport.cc File media/cast/net/udp_transport.cc (right): https://codereview.chromium.org/967723002/diff/10002/media/cast/net/udp_transport.cc#newcode70 media/cast/net/udp_transport.cc:70: On 2015/05/08 06:49:05, miu wrote: > Should there be ...
5 years, 7 months ago (2015-05-18 20:41:06 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/967723002/50001
5 years, 7 months ago (2015-05-18 20:44:25 UTC) #7
commit-bot: I haz the power
Committed patchset #4 (id:50001)
5 years, 7 months ago (2015-05-18 22:29:18 UTC) #8
commit-bot: I haz the power
5 years, 7 months ago (2015-05-18 22:44:51 UTC) #9
Message was sent while issue was closed.
Patchset 4 (id:??) landed as
https://crrev.com/f90e67e8844bf3c40c1091bd843a3ee11c9bc1c3
Cr-Commit-Position: refs/heads/master@{#330442}

Powered by Google App Engine
This is Rietveld 408576698