Commit b11b2e1b authored by Chen Gang's avatar Chen Gang Committed by Greg Kroah-Hartman

drivers/usb/core: using strlcpy instead of strncpy

  for NUL terminated string, better notice '\0' in the end.
Signed-off-by: default avatarChen Gang <gang.chen@asianux.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 2f076077
...@@ -40,6 +40,7 @@ ...@@ -40,6 +40,7 @@
#include <linux/signal.h> #include <linux/signal.h>
#include <linux/poll.h> #include <linux/poll.h>
#include <linux/module.h> #include <linux/module.h>
#include <linux/string.h>
#include <linux/usb.h> #include <linux/usb.h>
#include <linux/usbdevice_fs.h> #include <linux/usbdevice_fs.h>
#include <linux/usb/hcd.h> /* for usbcore internals */ #include <linux/usb/hcd.h> /* for usbcore internals */
...@@ -1077,7 +1078,7 @@ static int proc_getdriver(struct dev_state *ps, void __user *arg) ...@@ -1077,7 +1078,7 @@ static int proc_getdriver(struct dev_state *ps, void __user *arg)
if (!intf || !intf->dev.driver) if (!intf || !intf->dev.driver)
ret = -ENODATA; ret = -ENODATA;
else { else {
strncpy(gd.driver, intf->dev.driver->name, strlcpy(gd.driver, intf->dev.driver->name,
sizeof(gd.driver)); sizeof(gd.driver));
ret = (copy_to_user(arg, &gd, sizeof(gd)) ? -EFAULT : 0); ret = (copy_to_user(arg, &gd, sizeof(gd)) ? -EFAULT : 0);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment