Comment 3 for bug 180598

Revision history for this message
Steve Langasek (vorlon) wrote :

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.)