CHG: compiler warning about not using fscanf return value.

This commit is contained in:
iceman1001 2017-03-06 19:24:50 +01:00
commit 6795b3da93

View file

@ -111,70 +111,76 @@ void ReceiveCommand(UsbCommand *c)
usb_dev_handle* findProxmark(int verbose, unsigned int *iface) usb_dev_handle* findProxmark(int verbose, unsigned int *iface)
{ {
struct usb_bus *busses, *bus; struct usb_bus *busses, *bus;
usb_dev_handle *handle = NULL; usb_dev_handle *handle = NULL;
struct prox_unit units[50]; struct prox_unit units[50];
int iUnit = 0; int iUnit = 0;
usb_find_busses(); usb_find_busses();
usb_find_devices(); usb_find_devices();
busses = usb_get_busses(); busses = usb_get_busses();
for (bus = busses; bus; bus = bus->next) { for (bus = busses; bus; bus = bus->next) {
struct usb_device *dev; struct usb_device *dev;
for (dev = bus->devices; dev; dev = dev->next) { for (dev = bus->devices; dev; dev = dev->next) {
struct usb_device_descriptor *desc = &(dev->descriptor); struct usb_device_descriptor *desc = &(dev->descriptor);
if ((desc->idProduct == 0x4b8f) && (desc->idVendor == 0x9ac4)) { if ((desc->idProduct == 0x4b8f) && (desc->idVendor == 0x9ac4)) {
handle = usb_open(dev); handle = usb_open(dev);
if (!handle) { if (!handle) {
if (verbose) if (verbose)
fprintf(stderr, "open fabiled: %s!\n", usb_strerror()); fprintf(stderr, "open fabiled: %s!\n", usb_strerror());
//return NULL; //return NULL;
continue; continue;
} }
*iface = dev->config[0].interface[0].altsetting[0].bInterfaceNumber; *iface = dev->config[0].interface[0].altsetting[0].bInterfaceNumber;
struct prox_unit unit = {handle, {0}}; struct prox_unit unit = {handle, {0}};
usb_get_string_simple(handle, desc->iSerialNumber, unit.serial_number, sizeof(unit.serial_number)); usb_get_string_simple(handle, desc->iSerialNumber, unit.serial_number, sizeof(unit.serial_number));
units[iUnit++] = unit; units[iUnit++] = unit;
//return handle; //return handle;
} }
} }
} }
if (iUnit > 0) { if (iUnit > 0) {
int iSelection = 0; int iSelection = 0;
fprintf(stdout, "\nConnected units:\n"); fprintf(stdout, "\nConnected units:\n");
for (int i = 0; i < iUnit; i++) { for (int i = 0; i < iUnit; i++) {
struct usb_device * dev = usb_device(units[i].handle); struct usb_device * dev = usb_device(units[i].handle);
fprintf(stdout, "\t%d. SN: %s [%s/%s]\n", i+1, units[i].serial_number, dev->bus->dirname, dev->filename); fprintf(stdout, "\t%d. SN: %s [%s/%s]\n", i+1, units[i].serial_number, dev->bus->dirname, dev->filename);
} }
if (iUnit > 1) { if (iUnit > 1) {
while (iSelection < 1 || iSelection > iUnit) { while (iSelection < 1 || iSelection > iUnit) {
fprintf(stdout, "Which unit do you want to connect to? "); fprintf(stdout, "Which unit do you want to connect to? ");
fscanf(stdin, "%d", &iSelection); int res = fscanf(stdin, "%d", &iSelection);
} if ( res != 1 ) {
} fprintf(stderr, "Input parse error");
else fflush(stderr);
iSelection = 1; abort();
iSelection --; }
}
}
else {
iSelection = 1;
}
for (int i = 0; i < iUnit; i++) { iSelection --;
if (iSelection == i) continue;
usb_close(units[i].handle);
units[i].handle = NULL;
}
return units[iSelection].handle; for (int i = 0; i < iUnit; i++) {
} if (iSelection == i) continue;
usb_close(units[i].handle);
units[i].handle = NULL;
}
return NULL; return units[iSelection].handle;
}
return NULL;
} }
usb_dev_handle* OpenProxmark(int verbose) usb_dev_handle* OpenProxmark(int verbose)