Re: [PATCH-2.4] Typo in cdrom.c also in linux-2.4

[Willy Tarreau <w@1wt.eu>]

Hi Andreas,

On Thu, Jul 06, 2006 at 10:23:12AM +0200, Andreas Haumer wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> Hi Marcelo,
> hi Willy,
> 
> are you aware of the discussion at
> <http://bugzilla.kernel.org/show_bug.cgi?id=2966> ?

No, I was not aware. Scary !

> The typo seems to exist in linux-2.4 too, at least in
> 2.4.32, 2.4.32-hf32.6 and 2.4.33pre3 (which is what
> I checked today)
> 
> The fix for linux-2.4 would be just like the proposed
> patch for linux-2.6 (see attachment)
> 
> Comments?

Looks fine and conform to Jens' description.
Queuing it into -upstream.

> - - andreas

Thanks,
Willy

> 
> - --
> Andreas Haumer                     | mailto:andreas@xss.co.at
> *x Software + Systeme              | http://www.xss.co.at/
> Karmarschgasse 51/2/20             | Tel: +43-1-6060114-0
> A-1100 Vienna, Austria             | Fax: +43-1-6060114-71
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.3 (GNU/Linux)
> Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
> 
> iD8DBQFErMhbxJmyeGcXPhERAgw6AKCwDRtEyE1EK+oT/nU5v2ysQxmWqACcDh1y
> eZq4acxsutVuP68nDDcEeAE=
> =puYc
> -----END PGP SIGNATURE-----

> Index: cdrom.c
> ===================================================================
> RCS file: /home/cvs/repository/distribution/Base/linux/drivers/cdrom/cdrom.c,v
> retrieving revision 1.1.1.10
> diff -u -r1.1.1.10 cdrom.c
> --- cdrom.c	19 Jan 2005 14:09:43 -0000	1.1.1.10
> +++ cdrom.c	6 Jul 2006 08:22:06 -0000
> @@ -1259,7 +1259,7 @@
>  	init_cdrom_command(&cgc, buf, sizeof(buf), CGC_DATA_READ);
>  	cgc.cmd[0] = GPCMD_READ_DVD_STRUCTURE;
>  	cgc.cmd[7] = s->type;
> -	cgc.cmd[9] = cgc.buflen = 0xff;
> +	cgc.cmd[9] = cgc.buflen & 0xff;
>  
>  	if ((ret = cdo->generic_packet(cdi, &cgc)))
>  		return ret;

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/