New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 818893 link

Starred by 1 user
Status: Assigned
Owner:
█ wangyix@chromium.org
Last visit > 30 days ago
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 488402



Sign in to add a comment

UDPSocket should support sending MSG_CONFIRM

Project Member Reported by rch@chromium.org, Mar 6 2018 
When sending packets using sendto() the caller can specify MSG_CONFIRM

       MSG_CONFIRM (since Linux 2.3.15)
              Tell the link layer that forward progress happened: you got a successful reply from the other side.  If the link layer doesn't get this it will regularly reprobe the neighbor (e.g.,  via  a
              unicast ARP).  Valid only on SOCK_DGRAM and SOCK_RAW sockets and currently implemented only for IPv4 and IPv6.  See arp(7) for details.

UDPSocket should support this, probably by adding a SetMsgConfirm(bool confirm) method.
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 15 2018 

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

commit 3ae76ca8a924921d7639955adf50970ea734382f
Author: Yixin Wang <wangyix@chromium.org>
Date: Thu Mar 15 17:18:52 2018

Add SetMsgConfirm(bool) to udp_socket_posix and udp_socket_win

Bug: 818893
Change-Id: I58d04134464770368736cfb513a515d0debfead5
Reviewed-on: https://chromium-review.googlesource.com/956929
Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
Reviewed-by: Ryan Hamilton <rch@chromium.org>
Commit-Queue: Yixin Wang <wangyix@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543416}
[modify] https://crrev.com/3ae76ca8a924921d7639955adf50970ea734382f/content/browser/renderer_host/p2p/socket_host_udp_unittest.cc
[modify] https://crrev.com/3ae76ca8a924921d7639955adf50970ea734382f/net/dns/address_sorter_posix_unittest.cc
[modify] https://crrev.com/3ae76ca8a924921d7639955adf50970ea734382f/net/dns/mock_mdns_socket_factory.h
[modify] https://crrev.com/3ae76ca8a924921d7639955adf50970ea734382f/net/socket/datagram_socket.h
[modify] https://crrev.com/3ae76ca8a924921d7639955adf50970ea734382f/net/socket/fuzzed_datagram_client_socket.h
[modify] https://crrev.com/3ae76ca8a924921d7639955adf50970ea734382f/net/socket/socket_test_util.h
[modify] https://crrev.com/3ae76ca8a924921d7639955adf50970ea734382f/net/socket/udp_client_socket.cc
[modify] https://crrev.com/3ae76ca8a924921d7639955adf50970ea734382f/net/socket/udp_client_socket.h
[modify] https://crrev.com/3ae76ca8a924921d7639955adf50970ea734382f/net/socket/udp_server_socket.cc
[modify] https://crrev.com/3ae76ca8a924921d7639955adf50970ea734382f/net/socket/udp_server_socket.h
[modify] https://crrev.com/3ae76ca8a924921d7639955adf50970ea734382f/net/socket/udp_socket_posix.cc
[modify] https://crrev.com/3ae76ca8a924921d7639955adf50970ea734382f/net/socket/udp_socket_posix.h
[modify] https://crrev.com/3ae76ca8a924921d7639955adf50970ea734382f/net/socket/udp_socket_win.cc
[modify] https://crrev.com/3ae76ca8a924921d7639955adf50970ea734382f/net/socket/udp_socket_win.h

Comment 2 by benhenry@chromium.org, Aug 3

Status: Assigned (was: Untriaged)
This bug has an owner, thus, it's been triaged. Changing status to "assigned".
►

Sign in to add a comment