Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]:55978 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751538AbdF1Rub (ORCPT ); Wed, 28 Jun 2017 13:50:31 -0400 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Subject: Re: [net-next] qtnfmac: fix uninitialized return code in ret From: Kalle Valo In-Reply-To: <20170621132530.11937-1-colin.king@canonical.com> References: <20170621132530.11937-1-colin.king@canonical.com> To: Colin Ian King Cc: Igor Mitsyanko , Avinash Patil , Sergey Matyukevich , "David S . Miller" , Bindu Therthala , Kamlesh Rath , Sergei Maksimenko , Johannes Berg , linux-wireless@vger.kernel.org, netdev@vger.kernel.org, kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org Message-Id: <20170628175030.F05F460AFB@smtp.codeaurora.org> (sfid-20170628_195052_340202_2C868B1E) Date: Wed, 28 Jun 2017 17:50:30 +0000 (UTC) Sender: linux-wireless-owner@vger.kernel.org List-ID: Colin Ian King wrote: > From: Colin Ian King > > The return value ret is unitialized and garbage is being returned > for the three different error conditions when setting up the PCIe > BARs. Fix this by initializing ret to -ENOMEM to indicate that > the BARs failed to be setup correctly. > > Detected by CoverityScan, CID#1437563 ("Unitialized scalar variable") > > Signed-off-by: Colin Ian King > Reviewed-by: Sergey Matyukevich Patch applied to wireless-drivers-next.git, thanks. 3e3d8aa61107 qtnfmac: fix uninitialized return code in ret -- https://patchwork.kernel.org/patch/9801833/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches