From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751734AbcFSKD6 (ORCPT ); Sun, 19 Jun 2016 06:03:58 -0400 Received: from mail-wm0-f68.google.com ([74.125.82.68]:35906 "EHLO mail-wm0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751671AbcFSKD4 (ORCPT ); Sun, 19 Jun 2016 06:03:56 -0400 Subject: Re: [PATCH 1/1] staging: iio: initialize data pointer before using it To: Lars-Peter Clausen , Michael Hennerich , Greg Kroah-Hartman , Ksenija Stanojevic References: <20160618211224.23183-1-nicolas.iooss_linux@m4x.org> <576668B7.8080609@metafoo.de> Cc: Jonathan Cameron , Hartmut Knaack , Peter Meerwald-Stadler , linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org From: Nicolas Iooss Message-ID: <19e60ec0-87cc-e79e-89a2-39857119640a@m4x.org> Date: Sun, 19 Jun 2016 12:03:53 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.1.1 MIME-Version: 1.0 In-Reply-To: <576668B7.8080609@metafoo.de> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 06/19/2016 11:41 AM, Lars-Peter Clausen wrote: > On 06/18/2016 11:12 PM, Nicolas Iooss wrote: >> ad7606_spi_read_block() reads SPI big-endian data into a buffer which is >> then converted to CPU byte-order with a second variable. This variable, >> data, has been left unitialized after commit 87787e5ef727 ("Staging: >> iio: Fix sparse endian warning"). > > Hi, > > Thanks for the patch. A similar patch was already merged last week. Nice :) Actually I sent this patch because the bug is still present in linux-next tree [1] and I failed to see a reference to a dedicated development tree in MAINTAINERS. Thanks for your quick reply! -- Nicolas [1] https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/tree/drivers/staging/iio/adc/ad7606_spi.c