linux.git/drivers/usb, branch v2.6.22

potential compiler error, irqfunc caller sites update

2007-07-06T17:23:43+00:00

In 7d12e780e003f93433d49ce78cfedf4b4c52adc5 David Howells performed
this evolution:
 "IRQ: Maintain regs pointer globally rather than passing to IRQ handlers"

He correctly updated many of the function definitions that were using this
extra regs pointer parameter but forgot to update some caller sites of
those functions.  The reason the modifications was not properly done on all
drivers is that some drivers were rarely compiled because they are for
AMIGA, or that some code sites were inside #ifdefs where the option is not
set or inside #if 0.

Here is the semantic patch that found the occurences
and fixed the problem.

@ rule1 @
identifier fn;
identifier irq, dev_id;
typedef irqreturn_t;
@@

static irqreturn_t fn(int irq, void *dev_id)
{
   ...
}

@@
identifier rule1.fn;
expression E1, E2, E3;
@@

 fn(E1, E2
-   ,E3
   )

Signed-off-by: Yoann Padioleau 
Cc: "David S. Miller" 
Cc: Jeff Garzik 
Cc: Greg KH 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

USB: ftdio_sio: New IPlus device ID

2007-06-26T06:38:06+00:00

Reported by Grzegorz Chimosz 

Signed-off-by: Luiz Fernando N. Capitulino 
Signed-off-by: Greg Kroah-Hartman

USB: add new device id to option driver

2007-06-26T06:38:06+00:00

Cc: Matthias Urlichs 
Signed-off-by: Greg Kroah-Hartman

USB: fix race leading to use after free in io_edgeport

2007-06-26T06:38:06+00:00

usb_unlink_urb() is asynchronous, therefore an URB's buffer may not
be freed without waiting for the completion handler. This patch switches
to usb_kill_urb(), which is synchronous.
Thanks to Alan for making me look at the remaining users of usb_unlink_urb()


Signed-off-by: Oliver Neukum 
Signed-off-by: Al Borchers 
Signed-off-by: Greg Kroah-Hartman

USB: usblcd doesn't limit memory consumption during write

2007-06-26T06:38:06+00:00

usblcd currently has no way to limit memory consumption by fast writers.
This is a security problem, as it allows users with write access to this
device to drive the system into oom despite resource limits.
Here's the fix taken from the modern skeleton driver.

Signed-off-by: Oliver Neukum 
Signed-off-by: Greg Kroah-Hartman

USB: memory leak in iowarrior.c

2007-06-26T06:38:05+00:00

this is a classical memory leak in the ioctl handler. The buffer is simply
never freed. This fixes it the obvious way.

Signed-off-by: Oliver Neukum 
Signed-off-by: Greg Kroah-Hartman

USB: ti serial driver sleeps with spinlock held

2007-06-26T06:38:05+00:00

you are submitting an URB with GFP_KERNEL holding a spinlock.
In this case the spinlock can be dropped earlier.

Signed-off-by: Oliver Neukum 
Cc: Al Borchers 
Signed-off-by: Greg Kroah-Hartman

USB: g_file_storage: call allow_signal()

2007-06-26T06:38:05+00:00

New changes in the signal-handling code require compensating changes
in g_file_storage.  This patch (as913) by Oleg Nesterov makes the
code use allow_signal() instead of sigprocmask().

From: Alan Stern 
Signed-off-by: Alan Stern 
Signed-off-by: Greg Kroah-Hartman

OHCI: Fix machine check in ohci_hub_status_data

2007-06-08T23:24:31+00:00

This patch (as901) fixes an oversight in ohci-hcd.  The
hub_status_data routine must not try to access the controller's
memory-mapped registers if the controller is in a low-power state;
such attempts will cause a crash on some architectures (such as PPC).

Signed-off-by: Alan Stern 
Signed-off-by: Greg Kroah-Hartman

USB: Fix up bogus bInterval values in endpoint descriptors

2007-06-08T23:24:31+00:00

This patch (as904) adds code to check for endpoint descriptor bInterval
values outside the legal limits.  Illegal values are set to 32 ms, which
seems like a reasonable default.

This fixes Bugzilla #8432.

Signed-off-by: Alan Stern 
Signed-off-by: Greg Kroah-Hartman