diff options
| author | Sam Bishop <sam@bishop.dhs.org> | 2005-12-22 19:11:02 -0500 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2006-01-04 16:51:45 -0500 |
| commit | c8dd7709c534ab0d713aa698c99132b6c812b57c (patch) | |
| tree | 30daa90dfdf17ef397efe4230f6dfb03060d669d | |
| parent | cb5b3f6950b4fbad9d8d41756f49aba792804b5b (diff) | |
[PATCH] USB: fix usb-skeleton limit resource usage patch.
Prevents a compiler warning and uses down_interruptible() instead of down() in
process context.
Signed-off-by: Sam Bishop <sam@bishop.dhs.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
| -rw-r--r-- | drivers/usb/usb-skeleton.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/usb/usb-skeleton.c b/drivers/usb/usb-skeleton.c index b6652ef56830..5d02f16b7d0e 100644 --- a/drivers/usb/usb-skeleton.c +++ b/drivers/usb/usb-skeleton.c | |||
| @@ -166,7 +166,7 @@ static ssize_t skel_write(struct file *file, const char *user_buffer, size_t cou | |||
| 166 | int retval = 0; | 166 | int retval = 0; |
| 167 | struct urb *urb = NULL; | 167 | struct urb *urb = NULL; |
| 168 | char *buf = NULL; | 168 | char *buf = NULL; |
| 169 | size_t writesize = min(count, MAX_TRANSFER); | 169 | size_t writesize = min(count, (size_t)MAX_TRANSFER); |
| 170 | 170 | ||
| 171 | dev = (struct usb_skel *)file->private_data; | 171 | dev = (struct usb_skel *)file->private_data; |
| 172 | 172 | ||
| @@ -175,7 +175,10 @@ static ssize_t skel_write(struct file *file, const char *user_buffer, size_t cou | |||
| 175 | goto exit; | 175 | goto exit; |
| 176 | 176 | ||
| 177 | /* limit the number of URBs in flight to stop a user from using up all RAM */ | 177 | /* limit the number of URBs in flight to stop a user from using up all RAM */ |
| 178 | down (&dev->limit_sem); | 178 | if (down_interruptible(&dev->limit_sem)) { |
| 179 | retval = -ERESTARTSYS; | ||
| 180 | goto exit; | ||
| 181 | } | ||
| 179 | 182 | ||
| 180 | /* create a urb, and a buffer for it, and copy the data to the urb */ | 183 | /* create a urb, and a buffer for it, and copy the data to the urb */ |
| 181 | urb = usb_alloc_urb(0, GFP_KERNEL); | 184 | urb = usb_alloc_urb(0, GFP_KERNEL); |
