summaryrefslogtreecommitdiffstats
path: root/fastboot/usb_windows.c
diff options
context:
space:
mode:
authorElliott Hughes <enh@google.com>2009-10-06 18:07:49 -0700
committerElliott Hughes <enh@google.com>2009-10-06 18:07:49 -0700
commitb4add9b74525210478bac702d27fdaf9cf7ab18f (patch)
treec25ecbe4025f917ffa2d40b2c8890555424a2eaf /fastboot/usb_windows.c
parent04b7f9d42e751c7b96b872075d0fca4f3249254a (diff)
downloadcore-b4add9b74525210478bac702d27fdaf9cf7ab18f.tar.gz
core-b4add9b74525210478bac702d27fdaf9cf7ab18f.tar.bz2
core-b4add9b74525210478bac702d27fdaf9cf7ab18f.zip
Make fastboot say "no permissions" for non-writable devices.
Without this patch, "adb devices" will say "no permissions" when it sees a device it can't write to, but "fastboot devices" will silently ignore it. This is confusing to n00bs, especially since it doesn't seem to be widely known that a device's USB id might be different in the bootloader (meaning two udev rules are needed). It can also be confusing if you're sshed in, when you can't access the device because you won't be in the "plugdev" group, but "fastboot devices" won't make this clear. I'm not sure about the Mac OS and Windows changes. AIUI, devices are always writable on those platforms, but I don't use either, so I can't test this. This patch shouldn't alter the behavior on either of those platforms.
Diffstat (limited to 'fastboot/usb_windows.c')
-rw-r--r--fastboot/usb_windows.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/fastboot/usb_windows.c b/fastboot/usb_windows.c
index 9c0a9cbf1..54008a423 100644
--- a/fastboot/usb_windows.c
+++ b/fastboot/usb_windows.c
@@ -301,6 +301,7 @@ int recognized_device(usb_handle* handle, ifc_match_func callback) {
info.ifc_class = interf_desc.bInterfaceClass;
info.ifc_subclass = interf_desc.bInterfaceSubClass;
info.ifc_protocol = interf_desc.bInterfaceProtocol;
+ info.writable = 1;
// read serial number (if there is one)
unsigned long serial_number_len = sizeof(info.serial_number);