On 05/10/2007, James Bottomley <[EMAIL PROTECTED]> wrote:
> On Tue, 2007-10-02 at 14:38 -0700, [EMAIL PROTECTED] wrote:
> > From: Jesper Juhl <[EMAIL PROTECTED]>
> >
> > 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 <[EMAIL PROTECTED]>
> > Signed-off-by: Andrew Morton <[EMAIL PROTECTED]>
>
> 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 <[EMAIL PROTECTED]>
> to Andrew Morton <[EMAIL PROTECTED]>
> cc
> James Bottomley <[EMAIL PROTECTED]>,
> Justin T. Gibbs <[EMAIL PROTECTED]>
> 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 <[EMAIL PROTECTED]> wrote:
> > On Wed, 8 Aug 2007 00:26:43 +0200
> > "Jesper Juhl" <[EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]>
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 [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
