Re: [RFC 2/3] mei: unstaging mei driver

2012-03-01 Thread Dan Carpenter
> - - Updated MAINTAINERS I don't think we actually crossed this off the todo list. You'll maintain this right? Will you push changes directly to Linus or to someone else? regards, dan carpenter signature.asc Description: Digital signature ___ d

[patch] Staging: ozwpan: prevent bogus dereference

2012-03-01 Thread Dan Carpenter
app_id comes from the network and can't be trusted. If it's zero then it will lead to a kernel crash. Signed-off-by: Dan Carpenter diff --git a/drivers/staging/ozwpan/ozpd.c b/drivers/staging/ozwpan/ozpd.c index 8c460f0..e3381ad 100644 --- a/drivers/staging/ozwpan/ozpd.c +++ b/drivers/staging/o

[PATCH] Staging: bcm: fix possible memory leak of 'pstAddIndication' in CmHost.c and removes a whitespace

2012-03-01 Thread Kevin McKinney
Memory is being allocated by kmalloc and stored in variable pstAddIndication. However, this memory is not being freed in all cases. Therefore, this patch frees it on several exit paths. This patch also removes a whitespace. Signed-off-by: Kevin McKinney --- drivers/staging/bcm/CmHost.c | 27 +

Re: [PATCH 1/5] staging: lirc_serial: Fix init/exit order

2012-03-01 Thread Ben Hutchings
On Thu, 2012-03-01 at 21:45 -0600, Jonathan Nieder wrote: [...] > From I see that you tested these patches: > > affc9a0d59ac [media] staging: lirc_serial: Do not assume error codes > returned by request_irq() > 9b98d6067971 [media] staging: lirc_seri

Re: [PATCH 1/5] staging: lirc_serial: Fix init/exit order

2012-03-01 Thread Jonathan Nieder
Hi Ben, Ben Hutchings wrote[1]: > Currently the module init function registers a platform_device and > only then allocates its IRQ and I/O region. This allows allocation to > race with the device's suspend() function. Instead, allocate > resources in the platform driver's probe() function and f