From patchwork Thu Jan 16 23:17:07 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Greg KH X-Patchwork-Id: 233705 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.8 required=3.0 tests=DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI, SIGNED_OFF_BY, SPF_HELO_NONE, SPF_PASS, USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2F88EC33CAF for ; Thu, 16 Jan 2020 23:44:27 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id EB1FC2072E for ; Thu, 16 Jan 2020 23:44:26 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1579218267; bh=2qbbaDHWTrJc27fMCU/6Faxa3TAE21QTICCAgawihQc=; h=From:To:Cc:Subject:Date:In-Reply-To:References:List-ID:From; b=bRNX+VgCqopG5An36rWmHLAn+vIQfeOX59xQO8beim3KxMMhLO8eUpEkgKOxYQzLk O/Io87XtHXs98dR3fBCMTIHTVNOrT6gZXIuBsGmTk22cHaw4lLpq0FDm0YzB9M88Mo teN4bphiJjHqLf2SGn3b5ZajGzKj1IMl+ogUpxEE= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2388348AbgAPXoZ (ORCPT ); Thu, 16 Jan 2020 18:44:25 -0500 Received: from mail.kernel.org ([198.145.29.99]:52958 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729710AbgAPXYI (ORCPT ); Thu, 16 Jan 2020 18:24:08 -0500 Received: from localhost (83-86-89-107.cable.dynamic.v4.ziggo.nl [83.86.89.107]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 05AFA206D9; Thu, 16 Jan 2020 23:24:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1579217047; bh=2qbbaDHWTrJc27fMCU/6Faxa3TAE21QTICCAgawihQc=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=DHZ+5SLQKAs42yz8xnxIEMimMPBcPVQ/tE106Y1vMWtrg0mHg4JatUtc9TJj7jkor vl0mS4sQix/faCj7Dq4Xk8r2se/zyFGQ4se52acchHGGvfxOh6jmfFJPh9w97XdEhX bLFzqLF0rEjuHL58NBnusEZkaPfUSiKzE9+pso/I= From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Alexandru Ardelean , Jonathan Cameron Subject: [PATCH 5.4 110/203] iio: imu: adis16480: assign bias value only if operation succeeded Date: Fri, 17 Jan 2020 00:17:07 +0100 Message-Id: <20200116231755.105466750@linuxfoundation.org> X-Mailer: git-send-email 2.25.0 In-Reply-To: <20200116231745.218684830@linuxfoundation.org> References: <20200116231745.218684830@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Sender: stable-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org From: Alexandru Ardelean commit 9b742763d9d4195e823ae6ece760c9ed0500c1dc upstream. This was found only after the whole thing with the inline functions, but the compiler actually found something. The value of the `bias` (in adis16480_get_calibbias()) should only be set if the read operation was successful. No actual known problem occurs as users of this function all ultimately check the return value. Hence probably not stable material. Fixes: 2f3abe6cbb6c9 ("iio:imu: Add support for the ADIS16480 and similar IMUs") Signed-off-by: Alexandru Ardelean Signed-off-by: Jonathan Cameron Signed-off-by: Greg Kroah-Hartman --- drivers/iio/imu/adis16480.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) --- a/drivers/iio/imu/adis16480.c +++ b/drivers/iio/imu/adis16480.c @@ -454,12 +454,14 @@ static int adis16480_get_calibbias(struc case IIO_MAGN: case IIO_PRESSURE: ret = adis_read_reg_16(&st->adis, reg, &val16); - *bias = sign_extend32(val16, 15); + if (ret == 0) + *bias = sign_extend32(val16, 15); break; case IIO_ANGL_VEL: case IIO_ACCEL: ret = adis_read_reg_32(&st->adis, reg, &val32); - *bias = sign_extend32(val32, 31); + if (ret == 0) + *bias = sign_extend32(val32, 31); break; default: ret = -EINVAL;