From patchwork Thu Jan 7 14:16:50 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Greg KH X-Patchwork-Id: 358237 Delivered-To: patch@linaro.org Received: by 2002:a17:906:4755:0:0:0:0 with SMTP id j21csp336791ejs; Thu, 7 Jan 2021 06:19:27 -0800 (PST) X-Google-Smtp-Source: ABdhPJwkFGy8ycmqEGciiRr28rUdxSpgztoV50rB7yJ6SkHuGIagSp5jtYTfdU22u2HN4kqmLOht X-Received: by 2002:a05:6402:100c:: with SMTP id c12mr1929526edu.356.1610029167539; Thu, 07 Jan 2021 06:19:27 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1610029167; cv=none; d=google.com; s=arc-20160816; b=O7socaBcN2aPO0RNJgSnbSzPKHFluBvEXqTAe3mpwZWq/oiLaumISwKSvxhHhqVggV FnJ9Pg95woB9ZJ8TLdYtihOsgcY20tD4DwkSpGGj0JcIIERKmW/njudVfVSpd/6tCaYI L68Qha8LrS6cuWj/u1IGJ3vlCfZyVKBeAlJkiCO3x5kk1abzZXuBBV4c5TeBiGldGR/E 4mdP8FFYpPxDRvQrHMBRvx0Zk9gg/bHP82BBO/53+aLamj4yoJMusSBqqItRDhNvPS5k QwprUqdgdgkYRXiaIFtVmahs9Gib+4f2J/3DF83F/gOwbdrm0qn98c7hUIdWi1INlrQo HFCg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :user-agent:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=pBwfsyB9urwvVoG4VQHgIElyTbAyWhsQgTuJNmReQ9c=; b=VQ44bezh/LfxOlhXAP9aDqHhgyCGTgxEfRsgcKq8AwRljI+xA2h+H8vQjKOi9VMmU5 +/dWarCNJswu7xYZvzhycwvIDfNT+e31+FFnMzIXArtCBCpG9HdtnRYKdi0xk3jWSE69 fHUaRIxryutr83gpW+MaAUs000jBYkvIJ0yO28NOzfZkVPkrM2w31HEQmOhxWwmbHEgO HoegEhcoAYh3Jrs0XN4xAMsiVjnXPWC3lewMYvNxeGNn/9Z5qsmMs5pWLJDIqn9/Ryrs xP1vv+uLt5znjdEowgwTeTAWRpHNO8FhhVz+gEN7GKFkGaNmh9ygkviHrGE8ybpkzond PZqA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=vltmnBLR; spf=pass (google.com: domain of stable-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=stable-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id de8si2338724edb.319.2021.01.07.06.19.27; Thu, 07 Jan 2021 06:19:27 -0800 (PST) Received-SPF: pass (google.com: domain of stable-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=vltmnBLR; spf=pass (google.com: domain of stable-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=stable-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729228AbhAGORm (ORCPT + 14 others); Thu, 7 Jan 2021 09:17:42 -0500 Received: from mail.kernel.org ([198.145.29.99]:39546 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729225AbhAGORm (ORCPT ); Thu, 7 Jan 2021 09:17:42 -0500 Received: by mail.kernel.org (Postfix) with ESMTPSA id 8FAD623370; Thu, 7 Jan 2021 14:17:16 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1610029037; bh=RswBtloHKf83Hfz4px9bTxlFkA2ersMvDiSVW2cCW6w=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=vltmnBLRPopRxnHUs6n9VTscu5AzZkjzBrBs54+0B3Iefy6u6d1+yjiB0yVJko/+O 3fSSgY+XVPDMVd/9hzQ92Cx26qwE4rax3XCA7qfrX8gb9qW64XHV7Bl+IHFHaFL8eL l3w5Y1SRBfK7mBEc8e7cMOZwrsAe+TKey+I50+Tw= From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Jonathan Cameron , Alexandru Ardelean , Daniel Baluta , Stable@vger.kernel.org, Sudip Mukherjee Subject: [PATCH 4.9 30/32] iio:imu:bmi160: Fix too large a buffer. Date: Thu, 7 Jan 2021 15:16:50 +0100 Message-Id: <20210107140829.292483134@linuxfoundation.org> X-Mailer: git-send-email 2.30.0 In-Reply-To: <20210107140827.866214702@linuxfoundation.org> References: <20210107140827.866214702@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org From: Jonathan Cameron commit dc7de42d6b50a07b37feeba4c6b5136290fcee81 upstream. The comment implies this device has 3 sensor types, but it only has an accelerometer and a gyroscope (both 3D). As such the buffer does not need to be as long as stated. Note I've separated this from the following patch which fixes the alignment for passing to iio_push_to_buffers_with_timestamp() as they are different issues even if they affect the same line of code. Signed-off-by: Jonathan Cameron Reviewed-by: Alexandru Ardelean Cc: Daniel Baluta Cc: Link: https://lore.kernel.org/r/20200920112742.170751-5-jic23@kernel.org Signed-off-by: Sudip Mukherjee Signed-off-by: Greg Kroah-Hartman --- drivers/iio/imu/bmi160/bmi160_core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/drivers/iio/imu/bmi160/bmi160_core.c +++ b/drivers/iio/imu/bmi160/bmi160_core.c @@ -385,8 +385,8 @@ static irqreturn_t bmi160_trigger_handle struct iio_poll_func *pf = p; struct iio_dev *indio_dev = pf->indio_dev; struct bmi160_data *data = iio_priv(indio_dev); - __le16 buf[16]; - /* 3 sens x 3 axis x __le16 + 3 x __le16 pad + 4 x __le16 tstamp */ + __le16 buf[12]; + /* 2 sens x 3 axis x __le16 + 2 x __le16 pad + 4 x __le16 tstamp */ int i, ret, j = 0, base = BMI160_REG_DATA_MAGN_XOUT_L; __le16 sample;