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 linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 23 Jul 2013 11:55:50 -0700 From: Olof Johansson <olof@...om.net> To: Felipe Balbi <balbi@...com> Cc: linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org, Olof Johansson <olof@...om.net> Subject: [PATCH] usb: gadget: at91_udc: Check gpio lookup results This resolves the following valid build warning: drivers/usb/gadget/at91_udc.c:1685:34: warning: 'flags' may be used uninitialized in this function [-Wmaybe-uninitialized] I switched from ? : to !! mostly to save from wrapping the lines while I was at it. Signed-off-by: Olof Johansson <olof@...om.net> --- Felipe, this would be nice to see fixed for 3.11 but I'd argue that it's been here long enough to not really be needed for -stable. drivers/usb/gadget/at91_udc.c | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/drivers/usb/gadget/at91_udc.c b/drivers/usb/gadget/at91_udc.c index 073b938..f3dbcd0 100644 --- a/drivers/usb/gadget/at91_udc.c +++ b/drivers/usb/gadget/at91_udc.c @@ -1682,12 +1682,20 @@ static void at91udc_of_init(struct at91_udc *udc, board->vbus_pin = of_get_named_gpio_flags(np, "atmel,vbus-gpio", 0, &flags); - board->vbus_active_low = (flags & OF_GPIO_ACTIVE_LOW) ? 1 : 0; + if (board->vbus_pin < 0) + pr_err("%s: Failed to get atmel,vbus-gpio property\n", + np->full_name); + else + board->vbus_active_low = !!(flags & OF_GPIO_ACTIVE_LOW); board->pullup_pin = of_get_named_gpio_flags(np, "atmel,pullup-gpio", 0, &flags); - board->pullup_active_low = (flags & OF_GPIO_ACTIVE_LOW) ? 1 : 0; + if (board->pullup_pin < 0) + pr_err("%s: Failed to get atmel,pullup-gpio property\n", + np->full_name); + else + board->pullup_active_low = !!(flags & OF_GPIO_ACTIVE_LOW); } static int at91udc_probe(struct platform_device *pdev) -- 1.7.10.4 -- 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