aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSteve Holland <sdh4@iastate.edu>2009-06-18 18:37:49 -0400
committerGreg Kroah-Hartman <gregkh@suse.de>2009-09-23 09:46:35 -0400
commit92d07e422df3cc5370d0d9b95a671abb69d50ef1 (patch)
tree497f5e7e185c98405d4966d895e37b9edcbba52d
parentc2cd26e15b84b964c489f2aff278cdaf03840c93 (diff)
USB: usbtmc: inhibit corruption
Limit data copied to userspace to amount requested. Prevents a faulty instrument from overwriting user memory. Signed-off-by: Steve Holland <sdh4@iastate.edu> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-rw-r--r--drivers/usb/class/usbtmc.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/usb/class/usbtmc.c b/drivers/usb/class/usbtmc.c
index f2fde7cd6109..91d3a94eeaa0 100644
--- a/drivers/usb/class/usbtmc.c
+++ b/drivers/usb/class/usbtmc.c
@@ -473,6 +473,10 @@ static ssize_t usbtmc_read(struct file *filp, char __user *buf,
473 n_characters = this_part; 473 n_characters = this_part;
474 } 474 }
475 475
476 /* Bound amount of data received by amount of data requested */
477 if (n_characters > this_part)
478 n_characters = this_part;
479
476 /* Copy buffer to user space */ 480 /* Copy buffer to user space */
477 if (copy_to_user(buf + done, &buffer[12], n_characters)) { 481 if (copy_to_user(buf + done, &buffer[12], n_characters)) {
478 /* There must have been an addressing problem */ 482 /* There must have been an addressing problem */