Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp3821418imu; Mon, 7 Jan 2019 10:03:10 -0800 (PST) X-Google-Smtp-Source: ALg8bN5/NZk5vv2Qsi75LywUF+8JkQRa8WLqxjdjnBcvMiRNhw4Co02jKJCwoKpgty0h437LqLrz X-Received: by 2002:a17:902:9a04:: with SMTP id v4mr63037321plp.34.1546884190694; Mon, 07 Jan 2019 10:03:10 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1546884190; cv=none; d=google.com; s=arc-20160816; b=uRpNilFxNLq0F5aY9/LFUTgEwik+A4Oicv9VwnWm3qDSQqzAF1nM6FNxX7z6n82QBi 4BemG6wiPfVWD1zDIeVkPGtQsdytKaGspa7a2eLAbO53iQF7P3YtZFOnUFByOq+WrZBi OWBt9+kaAINJa7u6jsB+XGshB0QceAEFc7zeZfqLY06b6kNfiu15jzvAx1LyyOliFCAe krYmdkEjXq8E+15mXtyWNX0qYYjk1QNVnOKc71E/B++1CP9JGdCjvJFJl9OzywmpJXwn mhCMyO3qhZw28CHuzOkA0I7M5kybDho7Aa/gRQ1RIx/CUU7ENxhihoMDLKOTQ1lpznT+ qn+g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=Yx/CXCGumEO0HdcC+giwxPW8SUYoXFX0P7uH20hhv6Q=; b=xTQs7zSmhKllVshCIbYtPWJvuDye3JdiR7IJU2F474zK4OGFKH2aBotTYNIFRfe3ME JB/AFEW+Ca+RxJZABmhQVHS8bhulR/lMB1mxFxXxa18MpLAPjUXNVZ5HRzhgDyOgyFuw p5s6WbiS9Xz996Dwx5PRpZjacEr6+6IoFisBhHncYGTbK0Pe3dG8cGfxYcLmeq9bljps LVuGwDLYR1UjUxLmYVEJ0dLWTuMWysq7OLVM2Yd6v9PQsJHSLoDAp6J9EVkM7dnTcSvX yvEp36uP9bpVbsR8ODWTlz1j1wWxi+Mu+Xf/fjrgoOvs4mccSysZfhzfBGIzxMi7PHxQ zxDg== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@sirena.org.uk header.s=20170815-heliosphere header.b=nzDTK5HG; 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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id g83si7730814pfb.278.2019.01.07.10.02.55; Mon, 07 Jan 2019 10:03:10 -0800 (PST) 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; dkim=fail header.i=@sirena.org.uk header.s=20170815-heliosphere header.b=nzDTK5HG; 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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727875AbfAGQ7Z (ORCPT + 99 others); Mon, 7 Jan 2019 11:59:25 -0500 Received: from heliosphere.sirena.org.uk ([172.104.155.198]:38752 "EHLO heliosphere.sirena.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726811AbfAGQ7Y (ORCPT ); Mon, 7 Jan 2019 11:59:24 -0500 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sirena.org.uk; s=20170815-heliosphere; h=In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=Yx/CXCGumEO0HdcC+giwxPW8SUYoXFX0P7uH20hhv6Q=; b=nzDTK5HGQbQJWF9aYVi2dsr4q w+GE9FJcq6ajDRUjcXCYXjsju2FhKlDfOULCex3Thg4Kr6+90ZAVBlLC42wLIqJErPcDO+h3eh4LX VclyVFpUrUOICxz5eGRPGC+PwAiQuf94gkKnYedTSKdUJ8J/9lev8zqWnDiGxktX9YQIc=; Received: from cpc102320-sgyl38-2-0-cust46.18-2.cable.virginm.net ([82.37.168.47] helo=debutante.sirena.org.uk) by heliosphere.sirena.org.uk with esmtpa (Exim 4.89) (envelope-from ) id 1ggYEV-0007VS-5A; Mon, 07 Jan 2019 16:58:35 +0000 Received: by debutante.sirena.org.uk (Postfix, from userid 1000) id 6CF8F1127491; Mon, 7 Jan 2019 16:58:34 +0000 (GMT) Date: Mon, 7 Jan 2019 16:58:34 +0000 From: Mark Brown To: Yizhuo Cc: csong@cs.ucr.edu, zhiyunq@cs.ucr.edu, Bard Liao , Oder Chiou , Liam Girdwood , Jaroslav Kysela , Takashi Iwai , alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] ASoC: rt274: Variable "buf" in function rt274_jack_detect() could be uninitialized Message-ID: <20190107165834.GL9570@sirena.org.uk> References: <20190104202708.14759-1-yzhai003@ucr.edu> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="UK1lfQXsnwKrySH9" Content-Disposition: inline In-Reply-To: <20190104202708.14759-1-yzhai003@ucr.edu> X-Cookie: NOBODY EXPECTS THE SPANISH INQUISITION! User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --UK1lfQXsnwKrySH9 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Fri, Jan 04, 2019 at 12:27:08PM -0800, Yizhuo wrote: > In function rt274_jack_detect(), local variable "buf" could > be uninitialized if function regmap_read() returns -EINVAL. > However, it will be used to calculate "hp" and "mic" and > make their value unpredictable while those value are used > in the caller. This is potentially unsafe. This doesn't apply against current code, please check and resend. --UK1lfQXsnwKrySH9 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAlwzhTkACgkQJNaLcl1U h9BfYAf9HvrQb91OEUf4LOhKH18sRQCLgHXtlFFN/H7s1ST+zIQONgvRr0X3UrHT iiMU0dGygbKV4KNin/sZqJSrHEKJkXc2gSAV1S8iSEgb2VZOMb3kPu2uTbIEQYGA GqYkrSTcMawcOpD7XWn+9/UIe7+pYzqhrWpsdFw2bTvHFskfqesBAa19lnMKTW+6 wj4CCSLrCPw1/xaiK4rTkbeX7eax2okHyqjnNamaiBIgFU7bcN76kblxhVBXIBfs zuTZGqiYj2XktV57oOEr2g3mNyd6eRV/qVZbj7/MIrldAaAGmVixIBbzLQUZwG4E PVXRn4S0NeolG+Rwa6ttkqi34nelIw== =6AXx -----END PGP SIGNATURE----- --UK1lfQXsnwKrySH9--