From patchwork Fri Mar 22 14:25:03 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 160907 Delivered-To: patch@linaro.org Received: by 2002:a02:c6d8:0:0:0:0:0 with SMTP id r24csp876689jan; Fri, 22 Mar 2019 07:25:52 -0700 (PDT) X-Google-Smtp-Source: APXvYqxBem5c/J5DLGSYNACns+pZN05lnfUZkm1kYGGx1M5CjBvQUwoApa193TOg7ArzQrMny+FK X-Received: by 2002:a63:6e02:: with SMTP id j2mr8840656pgc.229.1553264752058; Fri, 22 Mar 2019 07:25:52 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1553264752; cv=none; d=google.com; s=arc-20160816; b=g+Ag1KwPB1jPUC9eI0mXByZVVNZk4wjBzz3tsAWKwl1sBS2utwI2zWhSoZ0QnRI18f g4VF59uxJRcdWkNFUYuoqfUFBTkZ26vZDKhCaXOK81hKVgsXn0boZshdIijeZBlx2qq6 4mRmq5KRpM4y51OQRutTErf5GY+roLVY9xrDMhC6P0QBO8hnXO0QFZbsD0TWYTzscqX2 YQNdHTvR6VbVUcWPq4qNgJukwbm4CTgNRwbtxY45XjnQ90RTXVvLBiIZzenWzgJouPuH QGNOxQAJ39rpyhsMp7a2heqjQQhv2DmL5wW7mxWBOMlqGEpPkGWadZv6bQYNhIzJSbeD aPQQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from; bh=45IPgfZ6yvHVxl+jZqMRWwe+qtrsDE66VsFzmNti3Hs=; b=Lh4CCulcbk6K9zBL6OxycORyGLPFLcguEm4DO5tZrjUzhTwxCcv8mTjXig/DwjXMbn zWront1zSSiAAVo/QZzJ5SE9/rQ0ewRmCaRbGkFOpi+cFeou52nNQhVLcSsmKG9PIQJ0 TTHeigsAKKvqWhemzHacv7iUoJB/t01Jjh8YFEAwS/Q13miMNet0rWtnpNqB4CU34m4w ILgYb8H7kUCjjZK0+SVl3Z5tLE+02XKt+4j8A1BSZWpvRMoZLBiRHvzuTPtjuSa6VCfV 5/4YLPruP31HJ0qScf4OLLI0rXyO4zk4KiR9VJ2ZXIwgAu93Fo+YMO7R86V5EYlGT5ZR PunQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id y24si4368028pfm.127.2019.03.22.07.25.51; Fri, 22 Mar 2019 07:25:52 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728753AbfCVOZu (ORCPT + 31 others); Fri, 22 Mar 2019 10:25:50 -0400 Received: from mout.kundenserver.de ([212.227.17.13]:39509 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728100AbfCVOZt (ORCPT ); Fri, 22 Mar 2019 10:25:49 -0400 Received: from wuerfel.lan ([149.172.19.189]) by mrelayeu.kundenserver.de (mreue106 [212.227.15.145]) with ESMTPA (Nemesis) id 1MiIhU-1gT2oE0K5G-00fRHK; Fri, 22 Mar 2019 15:25:32 +0100 From: Arnd Bergmann To: QLogic-Storage-Upstream@qlogic.com, "James E.J. Bottomley" , "Martin K. Petersen" Cc: clang-built-linux@googlegroups.com, Nick Desaulniers , Nathan Chancellor , Arnd Bergmann , Manish Rangankar , Chengguang Xu , Johannes Thumshirn , linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] scsi: qla4xxx: avoid freeing unallocated dma memory Date: Fri, 22 Mar 2019 15:25:03 +0100 Message-Id: <20190322142530.904080-1-arnd@arndb.de> X-Mailer: git-send-email 2.20.0 MIME-Version: 1.0 X-Provags-ID: V03:K1:Ld3F0hrJatNfmdt2OFIuffWFZjGGDCNGuU01maFhBtmKx1JSAzs 82c5XuRHym71YmZa7ql7llyTlp8QOnW+1mNalywMdDJxRBXOffWVuLPI7Ggp0K7u4+XQrHS 4aXr/T9WVT8XdX669vcM2IAK5So35aKlRrtjkvibtd0AXLOZrsqTqZtG4JF6RKP/7GdMy1g kiZEJgOvUKaGpUQUOguMg== X-Spam-Flag: NO X-UI-Out-Filterresults: notjunk:1; V03:K0:CaVxR7jP4bQ=:3jhgTJyc6Qjfc/fcy/+yRp 5sw95f6gxXneMgCqtiXwQAskl+Oy610fVWXOYJYoBHE7bxPicUxwRgSROCjjLhTbZeMIMDn9N kOhyyXkV2yWy7Cmyu9kI79Sv5hRTyRR/79Juy7j7ys2r5nGkz50I/PSAoP72V+GF8x70OmRio Yl6FF0P1cAg3lT+XvMMT/W7elMf3t/rFWctV/s6XcrqxoUSJMyPn9jwyd7B+grir0vyZrDQnY B8YPGIrwCOYDX2uv+YDzSb/cO4BvbFoEwZnuigqAJcM8zMMni6eqYh8p7Zgq1zC+p4kq2AlIf BT5MjVgNN1XlG25x7EI3Pf3ub9ghXQaIpdVOVK76zKZaCffkV7KeLceAT1cKS0JhdmQrkJG0I 9ivg6VFrhyf8iEC40y2VJj2o2QjqhejmSaFCdvlTYYOpwLsHS6xch0RqyiSLecyzZ7N1jLEfu e335MZF+uFs47x+ogqw1lsmcMdyae79wtlx5iUvkYYQ2KV+zwt0zpa6cHBL7GeXQb3HP4NFr6 z4eixQlPK1dA3uWo2pMV1PX6QF5eB1eeK2KyhiD7ynIW368Rvntuc/IhUMeub6/6VROA6Wu3i MoVNyGVI97tSmmCK63haWobnVb+cv9sGqxA+Emd4xySQ75MDdmATRjEkspdut2y41GxM7meRb O7lk+94zDhpFj/2SVDbkp78DHaebr8dUS+lG/I5+32d+sJEOI3aCFJf35FQhy84nsD1K0mtwH hmmykpOsu6CZjv/mUdcUsuv9NGTft/NqvpJvMw== Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Clang -Wuninitialized notices that on is_qla40XX we never allocate any DMA memory in get_fw_boot_info() but attempt to free it anyway: drivers/scsi/qla4xxx/ql4_os.c:5915:7: error: variable 'buf_dma' is used uninitialized whenever 'if' condition is false [-Werror,-Wsometimes-uninitialized] if (!(val & 0x07)) { ^~~~~~~~~~~~~ drivers/scsi/qla4xxx/ql4_os.c:5985:47: note: uninitialized use occurs here dma_free_coherent(&ha->pdev->dev, size, buf, buf_dma); ^~~~~~~ drivers/scsi/qla4xxx/ql4_os.c:5915:3: note: remove the 'if' if its condition is always true if (!(val & 0x07)) { ^~~~~~~~~~~~~~~~~~~ drivers/scsi/qla4xxx/ql4_os.c:5885:20: note: initialize the variable 'buf_dma' to silence this warning dma_addr_t buf_dma; ^ = 0 Skip the call to dma_free_coherent() here. Fixes: 2a991c215978 ("[SCSI] qla4xxx: Boot from SAN support for open-iscsi") Signed-off-by: Arnd Bergmann --- drivers/scsi/qla4xxx/ql4_os.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.20.0 Reviewed-by: Nathan Chancellor diff --git a/drivers/scsi/qla4xxx/ql4_os.c b/drivers/scsi/qla4xxx/ql4_os.c index 16a18d5d856f..daa884fa43f4 100644 --- a/drivers/scsi/qla4xxx/ql4_os.c +++ b/drivers/scsi/qla4xxx/ql4_os.c @@ -5928,7 +5928,7 @@ static int get_fw_boot_info(struct scsi_qla_host *ha, uint16_t ddb_index[]) val = rd_nvram_byte(ha, sec_addr); if (val & BIT_7) ddb_index[1] = (val & 0x7f); - + goto exit_boot_info; } else if (is_qla80XX(ha)) { buf = dma_alloc_coherent(&ha->pdev->dev, size, &buf_dma, GFP_KERNEL);