Package: libbladerf1
Version: 0.2016.06-1
Tags: patch

Hi

While checking the udev rules in the Debian archive, I noticed a problem
with the entry for libbladerf1.  It fail to grant access to local users
originating for a remote user directory like LDAP, because it depend on
plugdev group membership.  The emerging way to to fix this is to use the
uaccess udev tag.  This is handled by /lib/udev/rules.d/70-uaccess.rules
for all rules setting the ENV{ID_SOFTWARE_RADIO}="1", which seem to be
the correct approach here.  This setting is used by other software
radios.  Also, the current matching rule will match PCI and other
subsystems.  I suspect you only want to match USB devices?

Here is a patch implementing this change:

--- /lib/udev/rules.d/60-libbladerf1.rules      2015-08-09 01:52:51.000000000 
+0000
+++ /tmp/60-libbladerf1.rules   2016-10-22 06:56:01.588313307 +0000
@@ -1,8 +1,8 @@
 # nuand bladeRF
-ATTR{idVendor}=="1d50", ATTR{idProduct}=="6066", MODE="660", GROUP="plugdev"
+SUBSYSTEM=="usb", ATTR{idVendor}=="1d50", ATTR{idProduct}=="6066", 
ENV{ID_SOFTWARE_RADIO}="1", MODE="660", GROUP="plugdev"
 
 # Reserved for future bladeRF-specific bootloader
-ATTR{idVendor}=="1d50", ATTR{idProduct}=="6081", MODE="660", GROUP="plugdev"
+SUBSYSTEM=="usb", ATTR{idVendor}=="1d50", ATTR{idProduct}=="6081", 
ENV{ID_SOFTWARE_RADIO}="1", MODE="660", GROUP="plugdev"
 
 # Cypress Bootloader
-ATTR{idVendor}=="04b4", ATTR{idProduct}=="00f3", MODE="660", GROUP="plugdev"
+SUBSYSTEM=="usb", ATTR{idVendor}=="04b4", ATTR{idProduct}=="00f3", 
ENV{ID_SOFTWARE_RADIO}="1", MODE="660", GROUP="plugdev"

Please include it in a future version of the package.

-- 
Happy hacking
Petter Reinholdtsen

Reply via email to