Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Fri, 7 Feb 2003 12:29:24 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Fri, 7 Feb 2003 12:29:24 -0500 Received: from noodles.codemonkey.org.uk ([213.152.47.19]:51905 "EHLO noodles.internal") by vger.kernel.org with ESMTP id ; Fri, 7 Feb 2003 12:29:23 -0500 Date: Fri, 7 Feb 2003 17:35:16 +0000 From: Dave Jones To: Frank Davis Cc: linux-kernel@vger.kernel.org, trivial@rustcorp.com.au Subject: Re: [PATCH] 2.5.59 : drivers/scsi/advansys.c Message-ID: <20030207173516.GA31217@codemonkey.org.uk> Mail-Followup-To: Dave Jones , Frank Davis , linux-kernel@vger.kernel.org, trivial@rustcorp.com.au References: Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.4i Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 668 Lines: 18 On Fri, Feb 07, 2003 at 12:24:54PM -0500, Frank Davis wrote: > if (scp->request_bufflen != 0 && qdonep->remain_bytes != 0 && > - qdonep->remain_bytes <= scp->request_bufflen != 0) { > + qdonep->remain_bytes <= scp->request_bufflen && scp->request_bufflen!= 0) { you now have two scp->request_bufflen != 0 checks. Dave -- | Dave Jones. http://www.codemonkey.org.uk | SuSE Labs - 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/