On 05/10/2007, James Bottomley <James.Bottomley@xxxxxxxxxxxx> wrote: > On Tue, 2007-10-02 at 14:38 -0700, akpm@xxxxxxxxxxxxxxxxxxxx wrote: > > From: Jesper Juhl <jesper.juhl@xxxxxxxxx> > > > > The Coverity checker noticed that we have a potential NULL pointer > > deref in drivers/scsi/aic7xxx/aic7xxx_core.c::ahc_print_register(). > > This patch handles it by adding the same test against NULL that is > > used elsewhere in the same function. > > > > Signed-off-by: Jesper Juhl <jesper.juhl@xxxxxxxxx> > > Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> > > No. Reasons formerly given here: > > http://marc.info/?l=linux-scsi&m=118632919511846&w=2 > The reasons that patch sticks around was given by Andrew in a mail send to me, you and Justin Gibbs on Aug 8. My reply to that mail (including Andrew's original) is below, but you should be able to also find it in your own archives. Basically Andrew said: "That's the sort of patch I keep around to prevent the issue from getting swept under the table. This usually results in me sitting on the sorry thing for literally years :(" ---[ quote from old email - start ]--- > from Jesper Juhl <jesper.juhl@xxxxxxxxx> > to Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> > cc > James Bottomley <James.Bottomley@xxxxxxxxxxxx>, > Justin T. Gibbs <gibbs@xxxxxxxxxxx> > date 8 Aug 2007 00:43 > subject Re: + avoid-a-small-unlikely-memory-leak-in-proc_read_escd.patch added to > -mm tree > mailed-by gmail.com > On 08/08/07, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> wrote: > > On Wed, 8 Aug 2007 00:26:43 +0200 > > "Jesper Juhl" <jesper.juhl@xxxxxxxxx> wrote: > > > > > This reply was meant for > > > > > > > The patch titled > > > > Fix a potential NULL pointer deref in the aic7xxx, ahc_print_register() function > > > > has been added to the -mm tree. Its filename is > > > > fix-a-potential-null-pointer-deref-in-the-aic7xxx-ahc_print_register-function.patch > > > > > > > > That's the sort of patch I keep around to prevent the issue from getting > > swept under the table. This usually results in me sitting on the sorry > > thing for literally years :( > > > Ok, in that case you may want to keep it around in -mm despite James' > objections - I'll leave that up to you. But it probably shouldn't move > towards mainline... > > > Feel free to send a replacement patch which gets us closer to the real fix. > > Remove all the NULL tests, maybe. > > > I don't have a good idea on how to solve this better at the moment. > But if I come up with something I'll be sure to send it to you and > James. > ---[ quote from old email - end ]--- -- Jesper Juhl <jesper.juhl@xxxxxxxxx> Don't top-post http://www.catb.org/~esr/jargon/html/T/top-post.html Plain text mails only, please http://www.expita.com/nomime.html - To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
