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

Bodo Eggert <harvested.in.lkml@posting.7eggert.dyndns.org>
2005-04-13T10:41:03Z

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.
>
> Bullshit. ->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?