This patch looks wrong to me.  The original behavior would only treat a
wireless interface as enabled if the file existed under /sys and had a
value of 0.  The new behavior is to treat a wireless interface as
enabled *unless* the file exists and does *not* have a value of 0.  This
results in different handling of the case where $DEVICE/wireless exists
as a directory, and neither $DEVICE/device/power/state nor
$DEVICE/device/rf_kill exists.  Do you have a reason to believe that
this behavior needs to be changed?  If not, I would recommend restoring
the previous default of assuming a device is disabled.

(BTW, the comments in the patch are wrong -- 0 means enabled in each
case here, and 1 means disabled.)

-- 
change isAnyWirelessPoweredOn() accordingly to the removal of the old status 
kernel interface
https://bugs.launchpad.net/bugs/180598
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to