From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arend van Spriel Subject: Re: [PATCH 0/3] hostap: Fine-tuning for a few functions Date: Sat, 20 Aug 2016 21:26:42 +0200 Message-ID: <3b3c91ad-3225-3483-bb58-fdee33bf2290@broadcom.com> References: <566ABCD9.1060404@users.sourceforge.net> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: LKML , kernel-janitors-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Julia Lawall To: SF Markus Elfring , linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Jouni Malinen , Kalle Valo Return-path: In-Reply-To: Sender: linux-wireless-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: netdev.vger.kernel.org On 20-08-16 18:43, SF Markus Elfring wrote: > From: Markus Elfring > Date: Sat, 20 Aug 2016 18:35:43 +0200 > > A few update suggestions were taken into account > from static source code analysis. Is it worth touching this old stuff especially when you are not making any functional changes. Regards, Arend > Markus Elfring (3): > Use memdup_user() > Delete an unnecessary jump label > Delete unnecessary variable initialisations > > .../net/wireless/intersil/hostap/hostap_ioctl.c | 36 ++++++++-------------- > 1 file changed, 12 insertions(+), 24 deletions(-) >