Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#17796 closed bug (no change required)

NetworkStatus keeps showing interfaces that aren't present anymore

Reported by: fatigatti Owned by: nobody
Priority: normal Milestone: Unscheduled
Component: Applications/NetworkStatus Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description

Steps to reproduce: 1) Connect a network interface (Android phone tethering, USB wireless card, etc). 2) Disconnect it. 3) NetworkStatus will still show it as present on the menu that appears when clicked.

I have tested this using an Android phone tethering via usb_rndis. The wireless USB card that I have in hand is unfortunately not supported so I can't try that, but I suspect same thing would happen.

Change History (4)

comment:1 by pulkomandy, 2 years ago

Resolution: no change required
Status: newclosed

This is intentional for USB interfaces. The idea is that unplugging it on the USB side is the same as unplugging it on the "ethernet" side. If you reconnect the device, the network traffic will resume.

NetworkStatus will show it as present, but link down.

comment:2 by fatigatti, 2 years ago

I get the idea, but I personally think it's not the right approach. You click it and nothing happens (or an empty info box opens, I don't remember) and besides, when you open the Network preflet (or ifconfig, for that matter) the interface is not shown. So while NetworkStatus says one thing, the Network preflet says otherwise. At the very least this should be consistent between this two apps. Then it's a matter of deciding which way to go.

I can only speak for myself, but when I first saw this the feeling was "this is bugged, it is not updating the list". Then for some reason every once in a while the usb_rndis driver takes a different interface number (/dev/net/usb_rndis/1 instead of 0) and I'm left seeing two different interfaces in NetworkStatus, one of them is there, the other isn't, when I open the preflet I just see one... I don't know, it just doesn't feel right.

A moment after I opened this I tought maybe this wasn't so difficult to fix so I went ahead and finished a patch just before seeing your message (I'm taking babysteps learning to code). It's working for me here if you want to review it.

Thanks for the effort in reviewing all of this :)

comment:3 by pulkomandy, 2 years ago

Well, please submit the patch then, at least let's have things consistent.

comment:4 by fatigatti, 2 years ago

Sorry about possible coding style mistakes, bad practices, etc, first time here https://review.haiku-os.org/c/haiku/+/5378

Note: See TracTickets for help on using tickets.