lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 06 Mar 2014 14:11:52 -0700 From: Shuah Khan <shuah.kh@...sung.com> To: Valentina Manea <valentina.manea.m@...il.com>, gregkh@...uxfoundation.org Cc: tobias.polzer@....de, dominik.paulus@....de, ly80toro@....cs.fau.de, ihadzic@...earch.bell-labs.com, linux-kernel@...r.kernel.org, linux-usb@...r.kernel.org, devel@...verdev.osuosl.org, firefly@...ts.rosedu.org, Shuah Khan <shuah.kh@...sung.com> Subject: Re: [PATCH 05/12] staging: usbip: userspace: re-add interface information listing On 03/04/2014 12:10 PM, Valentina Manea wrote: > This was deleted in the driver conversion patch. It didn't need > to be deleted; showing more information is ok. > > Signed-off-by: Valentina Manea <valentina.manea.m@...il.com> > --- > drivers/staging/usbip/userspace/src/usbip_list.c | 20 +++++++++++++++++++- > drivers/staging/usbip/userspace/src/usbipd.c | 16 +++++++++++++++- > 2 files changed, 34 insertions(+), 2 deletions(-) > > diff --git a/drivers/staging/usbip/userspace/src/usbip_list.c b/drivers/staging/usbip/userspace/src/usbip_list.c > index 2255107..7e1820a 100644 > --- a/drivers/staging/usbip/userspace/src/usbip_list.c > +++ b/drivers/staging/usbip/userspace/src/usbip_list.c > @@ -52,8 +52,9 @@ static int get_exported_devices(char *host, int sockfd) > struct op_devlist_reply reply; > uint16_t code = OP_REP_DEVLIST; > struct usbip_usb_device udev; > + struct usbip_usb_interface uintf; > unsigned int i; > - int rc; > + int rc, j; > > rc = usbip_net_send_op_common(sockfd, OP_REQ_DEVLIST, 0); > if (rc < 0) { > @@ -103,6 +104,23 @@ static int get_exported_devices(char *host, int sockfd) > printf("%11s: %s\n", "", udev.path); > printf("%11s: %s\n", "", class_name); > > + for (j = 0; j < udev.bNumInterfaces; j++) { > + rc = usbip_net_recv(sockfd, &uintf, sizeof(uintf)); > + if (rc < 0) { > + dbg("usbip_net_recv failed: usbip_usb_intf[%d]", > + j); Please use err() instead of dbg() so this error is reported when it happens, not just when usbip_use_debug is enabled. > + > + return -1; > + } > + usbip_net_pack_usb_interface(0, &uintf); > + > + usbip_names_get_class(class_name, sizeof(class_name), > + uintf.bInterfaceClass, > + uintf.bInterfaceSubClass, > + uintf.bInterfaceProtocol); > + printf("%11s: %2d - %s\n", "", j, class_name); > + } > + > printf("\n"); > } > > diff --git a/drivers/staging/usbip/userspace/src/usbipd.c b/drivers/staging/usbip/userspace/src/usbipd.c > index c2b3ced..b2230f7 100644 > --- a/drivers/staging/usbip/userspace/src/usbipd.c > +++ b/drivers/staging/usbip/userspace/src/usbipd.c > @@ -159,8 +159,9 @@ static int send_reply_devlist(int connfd) > { > struct usbip_exported_device *edev; > struct usbip_usb_device pdu_udev; > + struct usbip_usb_interface pdu_uinf; > struct op_devlist_reply reply; > - int rc; > + int rc, i; > > reply.ndev = 0; > /* number of exported devices */ > @@ -194,6 +195,19 @@ static int send_reply_devlist(int connfd) > dbg("usbip_net_send failed: pdu_udev"); Same comment as above. > return -1; > } > + > + for (i = 0; i < edev->udev.bNumInterfaces; i++) { > + dump_usb_interface(&edev->uinf[i]); > + memcpy(&pdu_uinf, &edev->uinf[i], sizeof(pdu_uinf)); > + usbip_net_pack_usb_interface(1, &pdu_uinf); > + > + rc = usbip_net_send(connfd, &pdu_uinf, > + sizeof(pdu_uinf)); > + if (rc < 0) { > + dbg("usbip_net_send failed: pdu_uinf"); Same comment as above. > + return -1; > + } > + } > } > > return 0; > You have my Reviewed-by after making the recommended changes. Reviewed-by: Shuah Khan <shuah.kh@...sung.com> -- Shuah -- Shuah Khan Senior Linux Kernel Developer - Open Source Group Samsung Research America(Silicon Valley) shuah.kh@...sung.com | (970) 672-0658 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists