From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:55839) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bABpq-0004WO-3p for qemu-devel@nongnu.org; Tue, 07 Jun 2016 03:54:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bABpl-00069R-Rn for qemu-devel@nongnu.org; Tue, 07 Jun 2016 03:54:01 -0400 Received: from mail-wm0-x242.google.com ([2a00:1450:400c:c09::242]:36294) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bABpl-00069L-KU for qemu-devel@nongnu.org; Tue, 07 Jun 2016 03:53:57 -0400 Received: by mail-wm0-x242.google.com with SMTP id m124so20794319wme.3 for ; Tue, 07 Jun 2016 00:53:57 -0700 (PDT) Sender: Paolo Bonzini References: <1464178304-12831-1-git-send-email-ppandit@redhat.com> From: Paolo Bonzini Message-ID: <8a644c05-e43d-0a37-4f10-323abb892701@redhat.com> Date: Tue, 7 Jun 2016 09:53:52 +0200 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2] scsi: megasas: initialise local configuration data buffer List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: P J P , Qemu Developers Cc: Li Qiang , Hannes Reinecke , Alexander Graf On 07/06/2016 08:51, P J P wrote: > +-- On Wed, 25 May 2016, P J P wrote --+ > | Update as per > | -> https://lists.gnu.org/archive/html/qemu-devel/2016-05/msg04402.html > | > | diff --git a/hw/scsi/megasas.c b/hw/scsi/megasas.c > | index dcbd3e1..bf642d4 100644 > | --- a/hw/scsi/megasas.c > | +++ b/hw/scsi/megasas.c > | @@ -1293,7 +1293,7 @@ static int megasas_dcmd_ld_get_info(MegasasState *s, MegasasCmd *cmd) > | > | static int megasas_dcmd_cfg_read(MegasasState *s, MegasasCmd *cmd) > | { > | - uint8_t data[4096]; > | + uint8_t data[4096] = { 0 }; > | struct mfi_config_data *info; > | int num_pd_disks = 0, array_offset, ld_offset; > | BusChild *kid; > > Ping..! > > This one came with couple of others, > > -> https://patchwork.ozlabs.org/patch/626089/ > -> https://patchwork.ozlabs.org/patch/626152/ > -> https://patchwork.ozlabs.org/patch/626151/ > > Are these queued? (just checking) They are in already. In particular this is commit d37af740730dbbb93960cd318e040372d04d6dcf. Paolo