From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757237AbdELMVQ (ORCPT ); Fri, 12 May 2017 08:21:16 -0400 Received: from mx1.redhat.com ([209.132.183.28]:38282 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755430AbdELMVO (ORCPT ); Fri, 12 May 2017 08:21:14 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com B96BB8B11B Authentication-Results: ext-mx04.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com Authentication-Results: ext-mx04.extmail.prod.ext.phx2.redhat.com; spf=pass smtp.mailfrom=emilne@redhat.com DKIM-Filter: OpenDKIM Filter v2.11.0 mx1.redhat.com B96BB8B11B Subject: Re: [PATCH] scsi: libfc: fix incorrect variable assingment From: "Ewan D. Milne" Reply-To: emilne@redhat.com To: "Gustavo A. R. Silva" Cc: Johannes Thumshirn , "James E.J. Bottomley" , "Martin K. Petersen" , fcoe-devel@open-fcoe.org, linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org In-Reply-To: <20170511222444.GA6604@embeddedgus> References: <20170511222444.GA6604@embeddedgus> Content-Type: text/plain; charset="UTF-8" Organization: Red Hat Date: Fri, 12 May 2017 08:21:12 -0400 Message-ID: <1494591672.1081.10.camel@localhost.localdomain> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.28]); Fri, 12 May 2017 12:21:14 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 2017-05-11 at 17:24 -0500, Gustavo A. R. Silva wrote: > Previous assignment was causing the use of the uninitialized variable > _explan_ inside fc_seq_ls_rjt() function, which in this particular > case is being called by fc_seq_els_rsp_send(). > > Addresses-Coverity-ID: 1398125 > Signed-off-by: Gustavo A. R. Silva > --- > drivers/scsi/libfc/fc_rport.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/scsi/libfc/fc_rport.c b/drivers/scsi/libfc/fc_rport.c > index b44c313..5203258 100644 > --- a/drivers/scsi/libfc/fc_rport.c > +++ b/drivers/scsi/libfc/fc_rport.c > @@ -1422,7 +1422,7 @@ static void fc_rport_recv_rtv_req(struct fc_rport_priv *rdata, > fp = fc_frame_alloc(lport, sizeof(*rtv)); > if (!fp) { > rjt_data.reason = ELS_RJT_UNAB; > - rjt_data.reason = ELS_EXPL_INSUF_RES; > + rjt_data.explan = ELS_EXPL_INSUF_RES; > fc_seq_els_rsp_send(in_fp, ELS_LS_RJT, &rjt_data); > goto drop; > } s/assingment/assignment/ Reviewed-by: Ewan D. Milne