linux-kernel - Re: [PATCH 2.6.25-rc4] usb: add support for Motorola ROKR Z6 cellphone in mass storage mode

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20080311182933.GL2820@one-eyed-alien.net>
Date:	Tue, 11 Mar 2008 11:29:33 -0700
From:	Matthew Dharm <mdharm-kernel@...-eyed-alien.net>
To:	Constantin Baranov <const@...st.mimas.ru>
Cc:	linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org,
	USB Storage List <usb-storage@...ts.one-eyed-alien.net>,
	Greg KH <greg@...ah.com>
Subject: Re: [PATCH 2.6.25-rc4] usb: add support for Motorola ROKR Z6 cellphone in mass storage mode

Signed-off-by: Matthew Dharm <mdharm-usb@...-eyed-alien.net>

Greg, please apply.

Matt

On Tue, Mar 11, 2008 at 10:27:03PM +0400, Constantin Baranov wrote:
> From: Constantin Baranov <const@...su.ru>
> 
> Motorola ROKR Z6 cellphone has bugs in its USB, so it is impossible to use
> it as mass storage. Patch describes new "unusual" USB device for it with
> FIX_INQUIRY and FIX_CAPACITY flags and new BULK_IGNORE_TAG flag.
> Last flag relaxes check for equality of bcs->Tag and us->tag in
> usb_stor_Bulk_transport routine.
> 
> Signed-off-by: Constantin Baranov <const@...su.ru>
> ---
>  drivers/usb/storage/transport.c    |    3 ++-
>  drivers/usb/storage/unusual_devs.h |   11 +++++++++++
>  include/linux/usb_usual.h          |    4 +++-
>  3 files changed, 16 insertions(+), 2 deletions(-)
> 
> diff -upr linux-2.6.25-rc4/drivers/usb/storage/transport.c
> linux-2.6.25-rc4-motorokr_z6/drivers/usb/storage/transport.c
> --- linux-2.6.25-rc4/drivers/usb/storage/transport.c	2008-03-11
> 22:08:20.000000000 +0400
> +++ linux-2.6.25-rc4-motorokr_z6/drivers/usb/storage/transport.c
> 2008-03-11 22:09:42.000000000 +0400
> @@ -1009,7 +1009,8 @@ int usb_stor_Bulk_transport(struct scsi_
>  	US_DEBUGP("Bulk Status S 0x%x T 0x%x R %u Stat 0x%x\n",
>  			le32_to_cpu(bcs->Signature), bcs->Tag,
>  			residue, bcs->Status);
> -	if (bcs->Tag != us->tag || bcs->Status > US_BULK_STAT_PHASE) {
> +	if (!(bcs->Tag == us->tag || (us->flags & US_FL_BULK_IGNORE_TAG)) ||
> +		bcs->Status > US_BULK_STAT_PHASE) {
>  		US_DEBUGP("Bulk logical error\n");
>  		return USB_STOR_TRANSPORT_ERROR;
>  	}
> diff -upr linux-2.6.25-rc4/drivers/usb/storage/unusual_devs.h
> linux-2.6.25-rc4-motorokr_z6/drivers/usb/storage/unusual_devs.h
> --- linux-2.6.25-rc4/drivers/usb/storage/unusual_devs.h	2008-03-11
> 22:08:20.000000000 +0400
> +++ linux-2.6.25-rc4-motorokr_z6/drivers/usb/storage/unusual_devs.h
> 2008-03-11 22:14:59.000000000 +0400
> @@ -1589,6 +1589,17 @@ UNUSUAL_DEV(  0x22b8, 0x4810, 0x0001, 0x
>  		US_SC_DEVICE, US_PR_DEVICE, NULL,
>  		US_FL_FIX_CAPACITY),
> 
> +/*
> + * Patch by Constantin Baranov <const@...su.ru>
> + * Report by Andreas Koenecke.
> + * Motorola ROKR Z6.
> + */
> +UNUSUAL_DEV(  0x22b8, 0x6426, 0x0101, 0x0101,
> +		"Motorola",
> +		"MSnc.",
> +		US_SC_DEVICE, US_PR_DEVICE, NULL,
> +		US_FL_FIX_INQUIRY | US_FL_FIX_CAPACITY | US_FL_BULK_IGNORE_TAG),
> +
>  /* Reported by Radovan Garabik <garabik@...siopeia.juls.savba.sk> */
>  UNUSUAL_DEV(  0x2735, 0x100b, 0x0000, 0x9999,
>  		"MPIO",
> diff -upr linux-2.6.25-rc4/include/linux/usb_usual.h
> linux-2.6.25-rc4-motorokr_z6/include/linux/usb_usual.h
> --- linux-2.6.25-rc4/include/linux/usb_usual.h	2008-03-11
> 22:08:21.000000000 +0400
> +++ linux-2.6.25-rc4-motorokr_z6/include/linux/usb_usual.h	2008-03-11
> 22:09:42.000000000 +0400
> @@ -50,7 +50,9 @@
>  	US_FLAG(CAPACITY_HEURISTICS,	0x00001000)		\
>  		/* sometimes sizes is too big */		\
>  	US_FLAG(MAX_SECTORS_MIN,0x00002000)			\
> -		/* Sets max_sectors to arch min */
> +		/* Sets max_sectors to arch min */		\
> +	US_FLAG(BULK_IGNORE_TAG,0x00004000)			\
> +		/* Ignore tag mismatch in bulk operations */
> 
> 
>  #define US_FLAG(name, value)	US_FL_##name = value ,
> 

-- 
Matthew Dharm                              Home: mdharm-usb@...-eyed-alien.net 
Maintainer, Linux USB Mass Storage Driver

C:  Like the Furby?
DP: He gives me the creeps.  Think the SPCA will take him?
					-- Cobb and Dust Puppy
User Friendly, 1/2/1999

Content of type "application/pgp-signature" skipped