Chromium Code Reviews| Index: net/tools/quic/quic_client.cc |
| diff --git a/net/tools/quic/quic_client.cc b/net/tools/quic/quic_client.cc |
| index 15efa56502674166923b52a9501c35d689306971..4c8386a4dc630773f7e0d25e8fad5ba3a8f3a5ba 100644 |
| --- a/net/tools/quic/quic_client.cc |
| +++ b/net/tools/quic/quic_client.cc |
| @@ -120,11 +120,10 @@ QuicPacketWriter* QuicClient::DummyPacketWriterFactory::Create( |
| bool QuicClient::CreateUDPSocket() { |
| int address_family = server_address_.GetSockAddrFamily(); |
| - fd_ = socket(address_family, SOCK_DGRAM | SOCK_NONBLOCK, IPPROTO_UDP); |
| - if (fd_ < 0) { |
| - LOG(ERROR) << "CreateSocket() failed: " << strerror(errno); |
| - return false; |
| - } |
| + fd_ = QuicSocketUtils::CreateNonBlockingSocket(address_family, SOCK_DGRAM, |
| + IPPROTO_UDP); |
| + if (fd_ < 0) |
| + return false; // failure already logged |
|
Ryan Hamilton
2015/03/12 03:34:38
nit: We use {}s in one-line ifs for code that's sh
dougk
2015/03/12 12:39:10
Done.
|
| int get_overflow = 1; |
| int rc = setsockopt(fd_, SOL_SOCKET, SO_RXQ_OVFL, &get_overflow, |