On Fri, Oct 15, 2010 at 5:40 AM, Dan Carpenter <error27@xxxxxxxxx> wrote:
> Using pdpram_blk here would cause an oops. ÂWe intended to use
> &freercvpool.
>
> Signed-off-by: Dan Carpenter <error27@xxxxxxxxx>
>
> diff --git a/drivers/staging/ft1000/ft1000-usb/ft1000_hw.c b/drivers/staging/ft1000/ft1000-usb/ft1000_hw.c
> index d0637c3..076ba75 100644
> --- a/drivers/staging/ft1000/ft1000-usb/ft1000_hw.c
> +++ b/drivers/staging/ft1000/ft1000-usb/ft1000_hw.c
> @@ -1115,7 +1115,7 @@ u16 init_ft1000_netdev(struct ft1000_device *ft1000dev)
>
>
> Âerr_free:
> - Â Â Â list_for_each_safe(cur, tmp, &pdpram_blk->list) {
> + Â Â Â list_for_each_safe(cur, tmp, &freercvpool) {
> Â Â Â Â Â Â Â Âpdpram_blk = list_entry(cur, DPRAM_BLK, list);
> Â Â Â Â Â Â Â Âlist_del(&pdpram_blk->list);
> Â Â Â Â Â Â Â Âkfree(pdpram_blk->pbuffer);
>
Greg, you can add my Tested by: Marek Belisko <marek.belisko@xxxxxxxxx>
@Dan, good catch. Thanks.
Marek
--
as simple and primitive as possible
-------------------------------------------------
Marek Belisko - OPEN-NANDRA
Freelance Developer
Ruska Nova Ves 219 | Presov, 08005 Slovak Republic
Tel: +421 915 052 184
skype: marekwhite
icq: 290551086
web: http://open-nandra.com
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
