New issue
Advanced search Search tips

Issue 818320 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Fuchsia
Pri: 3
Type: Bug



Sign in to add a comment

[Fuchsia] ioctl_netc_get_if_info() fails

Project Member Reported by sergeyu@chromium.org, Mar 2 2018

Issue description

NetworkInterfacesTest.GetNetworkList and UDPSocketTest.JoinMulticastGroup 
fail on Fuchsia with the following error:
[3:1980715861:0302/184620.443305:18526588:ERROR:network_interfaces_fuchsia.cc(26)] ioctl_netc_get_if_info: Invalid argument (22)

This started happening after the last SDK roll.
 
Summary: [Fuchsia] ioctl_netc_get_if_info() fails (was: Fuchsia ioctl_netc_get_if_info() fails)
Looks like ioctl_netc_get_if_info()  was replaced with two new ioctls, see
https://fuchsia.googlesource.com/garnet/+/40f9bfa87e84f3941295886dec83920fad535bd2

So we just need to update the code to use the new ioctls.
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 9 2018

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

commit 61bdd884234836de0af39829d55eb1a84b7ce0fe
Author: Sergey Ulanov <sergeyu@chromium.org>
Date: Fri Mar 09 21:44:33 2018

[Fuchsia] Remove dependency on deprecated ioctl_netc_get_if_info()

ioctl_netc_get_if_info() has been replaced with ioctl_netc_get_num_ifs()
and ioctl_netc_get_num_ifs(). Update the corresponding code in //net.

Bug:  818320 
Change-Id: I42746f97616000f8839bcb0433c42e1d7feca129
Reviewed-on: https://chromium-review.googlesource.com/956613
Commit-Queue: Sergey Ulanov <sergeyu@chromium.org>
Reviewed-by: Matt Menke <mmenke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#542236}
[modify] https://crrev.com/61bdd884234836de0af39829d55eb1a84b7ce0fe/net/base/network_interfaces_fuchsia.cc
[modify] https://crrev.com/61bdd884234836de0af39829d55eb1a84b7ce0fe/net/socket/udp_socket_posix.cc

Status: Fixed (was: Started)

Sign in to add a comment