New issue
Advanced search Search tips

Issue 636544 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Aug 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

QUIC packets should be marked DF

Project Member Reported by rch@chromium.org, Aug 10 2016

Issue description

Since QUIC manages MTU explicitly it should mark packets with the DF bit to ensure that they are not fragmented.
 
Components: Internals>Network>QUIC
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 25 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ff006a1aab6168613e272a763d34c3cd34b58cc0

commit ff006a1aab6168613e272a763d34c3cd34b58cc0
Author: rch <rch@chromium.org>
Date: Wed Aug 24 23:56:31 2016

Add a SetDoNotFragment() method to DatagramSocket, and call this for
QUIC sockets.

BUG= 636544 

Review-Url: https://codereview.chromium.org/2235973002
Cr-Commit-Position: refs/heads/master@{#414206}

[modify] https://crrev.com/ff006a1aab6168613e272a763d34c3cd34b58cc0/content/browser/renderer_host/p2p/socket_host_udp_unittest.cc
[modify] https://crrev.com/ff006a1aab6168613e272a763d34c3cd34b58cc0/net/dns/address_sorter_posix_unittest.cc
[modify] https://crrev.com/ff006a1aab6168613e272a763d34c3cd34b58cc0/net/dns/mock_mdns_socket_factory.h
[modify] https://crrev.com/ff006a1aab6168613e272a763d34c3cd34b58cc0/net/quic/chromium/quic_stream_factory.cc
[modify] https://crrev.com/ff006a1aab6168613e272a763d34c3cd34b58cc0/net/socket/socket_test_util.cc
[modify] https://crrev.com/ff006a1aab6168613e272a763d34c3cd34b58cc0/net/socket/socket_test_util.h
[modify] https://crrev.com/ff006a1aab6168613e272a763d34c3cd34b58cc0/net/udp/datagram_socket.h
[modify] https://crrev.com/ff006a1aab6168613e272a763d34c3cd34b58cc0/net/udp/fuzzed_datagram_client_socket.cc
[modify] https://crrev.com/ff006a1aab6168613e272a763d34c3cd34b58cc0/net/udp/fuzzed_datagram_client_socket.h
[modify] https://crrev.com/ff006a1aab6168613e272a763d34c3cd34b58cc0/net/udp/udp_client_socket.cc
[modify] https://crrev.com/ff006a1aab6168613e272a763d34c3cd34b58cc0/net/udp/udp_client_socket.h
[modify] https://crrev.com/ff006a1aab6168613e272a763d34c3cd34b58cc0/net/udp/udp_server_socket.cc
[modify] https://crrev.com/ff006a1aab6168613e272a763d34c3cd34b58cc0/net/udp/udp_server_socket.h
[modify] https://crrev.com/ff006a1aab6168613e272a763d34c3cd34b58cc0/net/udp/udp_socket_posix.cc
[modify] https://crrev.com/ff006a1aab6168613e272a763d34c3cd34b58cc0/net/udp/udp_socket_posix.h
[modify] https://crrev.com/ff006a1aab6168613e272a763d34c3cd34b58cc0/net/udp/udp_socket_unittest.cc
[modify] https://crrev.com/ff006a1aab6168613e272a763d34c3cd34b58cc0/net/udp/udp_socket_win.cc
[modify] https://crrev.com/ff006a1aab6168613e272a763d34c3cd34b58cc0/net/udp/udp_socket_win.h

Comment 3 by rch@chromium.org, Aug 25 2016

Status: Fixed (was: Untriaged)

Sign in to add a comment