From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756767AbcECVqR (ORCPT ); Tue, 3 May 2016 17:46:17 -0400 Received: from out5-smtp.messagingengine.com ([66.111.4.29]:51705 "EHLO out5-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756537AbcECVqO (ORCPT ); Tue, 3 May 2016 17:46:14 -0400 X-Sasl-enc: lhcGXLx7WWRGhFZxy5/M+wNpPBTcLWXSX5zUngRp+5Oj 1462311973 Date: Tue, 3 May 2016 14:46:12 -0700 From: Greg Kroah-Hartman To: Kangjie Lu Cc: Johannes Berg , davem@davemloft.net, linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, Kangjie Lu Subject: Re: [PATCH] fix infoleak in wireless Message-ID: <20160503214612.GA13289@kroah.com> References: <1462308013-6032-1-git-send-email-kjlu@gatech.edu> <1462308044.10444.19.camel@sipsolutions.net> <20160503210052.GA23109@kroah.com> <20160503213401.GB23469@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.6.1 (2016-04-27) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org A: http://en.wikipedia.org/wiki/Top_post Q: Were do I find info about this thing called top-posting? A: Because it messes up the order in which people normally read text. Q: Why is top-posting such a bad thing? A: Top-posting. Q: What is the most annoying thing in e-mail? A: No. Q: Should I include quotations after my reply? http://daringfireball.net/2007/07/on_top On Tue, May 03, 2016 at 05:41:46PM -0400, Kangjie Lu wrote: > You are right. But wouldn't it be more general/better if we initialize the > allocation at very beginning? > To avoid information leaks, I think we are supposed to initialize all > allocations properly if  > we are not sure how they are used. But the networking maintainers told you to fix the broken drivers instead. So please do that and send those patches to the correct developers and mailing lists. The fact that only 2 staging drivers got this wrong means that everyone knows how to use this api properly, so I agree with the maintainers here. thanks, greg k-h