From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751779AbcFSL7d (ORCPT ); Sun, 19 Jun 2016 07:59:33 -0400 Received: from saturn.retrosnub.co.uk ([178.18.118.26]:37945 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751697AbcFSL7b (ORCPT ); Sun, 19 Jun 2016 07:59:31 -0400 Subject: Re: [PATCH 1/1] staging: iio: initialize data pointer before using it To: Nicolas Iooss , Lars-Peter Clausen , Michael Hennerich , Greg Kroah-Hartman , Ksenija Stanojevic References: <20160618211224.23183-1-nicolas.iooss_linux@m4x.org> <576668B7.8080609@metafoo.de> <19e60ec0-87cc-e79e-89a2-39857119640a@m4x.org> Cc: Hartmut Knaack , Peter Meerwald-Stadler , linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org From: Jonathan Cameron Message-ID: Date: Sun, 19 Jun 2016 12:58:57 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.1.0 MIME-Version: 1.0 In-Reply-To: <19e60ec0-87cc-e79e-89a2-39857119640a@m4x.org> 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 19/06/16 11:03, Nicolas Iooss wrote: > 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. > This is just me being slow at sending a pull request. Good point of the lack of a listed development tree. Will fix that so no one else wastes their time as you have. Sorry about that! Jonathan > 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 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >