From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753928Ab1JXWvV (ORCPT ); Mon, 24 Oct 2011 18:51:21 -0400 Received: from mx1.redhat.com ([209.132.183.28]:45582 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753287Ab1JXWvU (ORCPT ); Mon, 24 Oct 2011 18:51:20 -0400 Date: Mon, 24 Oct 2011 18:51:08 -0400 From: Dave Jones To: Richard Weinberger Cc: Linux Kernel Subject: Re: Fix kmalloc argument order in um/vdso/vma.c Message-ID: <20111024225108.GA25654@redhat.com> Mail-Followup-To: Dave Jones , Richard Weinberger , Linux Kernel References: <20111024221532.GA22671@redhat.com> <4EA5E573.3080006@nod.at> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4EA5E573.3080006@nod.at> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Oct 25, 2011 at 12:23:47AM +0200, Richard Weinberger wrote: > Am 25.10.2011 00:15, schrieb Dave Jones: > > kmalloc size is 1st arg, not second. > > > > Signed-off-by: Dave Jones > > Applied, Thanks! > > I'm a bit shocked that this obviously wrong code builds and seems to work. Both args are the same types, so gcc didn't know any better. > How have you found this? grep -r kmalloc\(GFP * Dave