Re: [2.6 patch] sound/oss/rme96xx.c: fix two check after use

From: Bodo Eggert (harvested.in.lkml_at_posting.7eggert.dyndns.org)
Date: 04/13/05

  • Next message: Rolf Offermanns: "mmap + dma_alloc_coherent" 
    To: Al Viro <viro@parcelfarce.linux.theplanet.co.uk>, Andrew Morton <akpm@osdl.org>, linux-kernel@vger.kernel.org, Adrian Bunk <bunk@stusta.de>
Date:	Wed, 13 Apr 2005 12:40:38 +0200

    Al Viro <viro@parcelfarce.linux.theplanet.co.uk> wrote:
    > On Wed, Apr 13, 2005 at 04:17:42AM +0200, Adrian Bunk wrote:

    >> This patch fixes two check after use found by the Coverity checker.
    >
    > Bull***. ->private_data is set by rme96xx_open() to guaranteed non-NULL
    > and never changed elsewhere. Same comment about reading the fscking
    > source, BUG_ON(), etc.

    If there are checks, they should be there for a purpose, and any sane
    reader will asume these checks to be nescensary. If they are dead code, you
    can say that, but please don't flame Adrian for fixing obviously buggy code
    in a way that is sane and at least more correct than the original without
    using several days of his lifetime to analyze the whole driver. Instead, you
    could provide the correct fix. 

    -- 
Funny quotes:
33. If lawyers are disbarred and clergymen defrocked, doesn't it follow that
    electricians can be delighted, musicians denoted, cowboys deranged, models
    deposed, tree surgeons debarked, and dry cleaners depressed?
-
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: Rolf Offermanns: "mmap + dma_alloc_coherent"