Re: [linux-usb-devel] 2.6.24: NULL scatter-gather pointer in usb_storage:usb_stor_access_xfer_buf?

2008-01-31 Thread Mark Glines
On Thu, 31 Jan 2008 11:27:39 +0200 Boaz Harrosh <[EMAIL PROTECTED]> wrote: > Please check the below patch. > > one thing that I can see is that the isd200 does an INQUARY transfer > of sizeof(struct inquiry_data) which is 96 bytes, when scsi_scan.c > sends an INQUARY with 36 bytes buffer. So we h

Re: [PATCH 2.6.24] bugfix for an overflow condition in usb storage & isd200.c

2008-02-01 Thread Mark Glines
report this condition > as a negative resid. Should we also set cmnd->status in the overflow > condition? > > Then also isd200.c is fixed to only return the type of INQUIRY && > SENSE the upper layer asked for. > > Signed-off-by: Boaz Harrosh <[EMAIL PROT

Re: [PATCH] bugfix for an underflow condition in usb storage & isd200.c

2008-02-08 Thread Mark Glines
or 2.6.24.stable, > this should do it. Mark, can you confirm that this patch alone fixes > the problem? Confirmed: works just fine. Tested-by: Mark Glines <[EMAIL PROTECTED]> > > Alan Stern > > > > Index: 2.6.24/drivers/usb/storage/protocol.c >