RE: [PATCH] fix a bug in ioctl(CDROMREADAUDIO) in cdrom.c in 2.2

From: Pierre Etchemaite (petchema@concept-micro.com)
Date: Thu Mar 15 2001 - 08:04:00 EST

Next message: bsuparna@in.ibm.com: "Re: (struct dentry *)->vfsmnt;"
Previous message: Rik van Riel: "Re: Linux on the Unisys ES7000 and CMP2 machines?"
In reply to: Jani Jaakkola: "[PATCH] fix a bug in ioctl(CDROMREADAUDIO) in cdrom.c in 2.2.18"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Le 14-Mar-2001, Jani Jaakkola écrivait :
>
> Using ioctl(CDROMREADAUDIO) with nframes argument being larger than 8 and
> not divisible by 8 causes kernel to read and return more audio data than
> was requested. This is bad since it clobbers up processes memory
> (I noticed this when my patched cdparanoia segfaulted).

Same thing for 2.4.2.

Is my allocation loop "over engineering", or just plain bad thing to do ?

Regards,
Pierre.

-- Linux blade.concept-micro.com 2.4.3-pre4 #1 Wed Mar 14 22:19:14 CET 2001 i686 unknown 2:04pm up 11:29, 4 users, load average: 2.66, 2.80, 2.26

('none' encoding is not supported, stored as-is)

application/octet-stream attachment: myreadaudio.patch

End of MIME message - 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/

Next message: bsuparna@in.ibm.com: "Re: (struct dentry *)->vfsmnt;"
Previous message: Rik van Riel: "Re: Linux on the Unisys ES7000 and CMP2 machines?"
In reply to: Jani Jaakkola: "[PATCH] fix a bug in ioctl(CDROMREADAUDIO) in cdrom.c in 2.2.18"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

This archive was generated by hypermail 2b29 : Thu Mar 15 2001 - 21:00:18 EST