Do not allocate space for the whole structure when we just want to store
authormpi <mpi@openbsd.org>
Sun, 10 Aug 2014 12:58:49 +0000 (12:58 +0000)
committermpi <mpi@openbsd.org>
Sun, 10 Aug 2014 12:58:49 +0000 (12:58 +0000)
a pointer, found by clang.

sys/dev/usb/usb_subr.c

index 75027b8..2a7a75d 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: usb_subr.c,v 1.107 2014/08/09 09:45:14 mpi Exp $ */
+/*     $OpenBSD: usb_subr.c,v 1.108 2014/08/10 12:58:49 mpi Exp $ */
 /*     $NetBSD: usb_subr.c,v 1.103 2003/01/10 11:19:13 augustss Exp $  */
 /*     $FreeBSD: src/sys/dev/usb/usb_subr.c,v 1.18 1999/11/17 22:33:47 n_hibma Exp $   */
 
@@ -909,8 +909,7 @@ usbd_probe_and_attach(struct device *parent, struct usbd_device *dev, int port,
                }
                nifaces = dev->cdesc->bNumInterface;
                uaa.configno = dev->cdesc->bConfigurationValue;
-               ifaces = malloc(nifaces * sizeof(struct usbd_interface),
-                   M_USB, M_NOWAIT);
+               ifaces = malloc(nifaces * sizeof(*ifaces), M_USB, M_NOWAIT);
                if (ifaces == NULL) {
                        err = USBD_NOMEM;
                        goto fail;