From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1765540AbXKOR7G (ORCPT ); Thu, 15 Nov 2007 12:59:06 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1764371AbXKOR6v (ORCPT ); Thu, 15 Nov 2007 12:58:51 -0500 Received: from mx2.suse.de ([195.135.220.15]:33703 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1764195AbXKOR6t (ORCPT ); Thu, 15 Nov 2007 12:58:49 -0500 Date: Thu, 15 Nov 2007 09:58:10 -0800 From: Greg KH To: Chuck Ebbert Cc: linux-kernel@vger.kernel.org, stable@kernel.org, Justin Forbes , Zwane Mwaikambo , "Theodore Ts'o" , Randy Dunlap , Dave Jones , Chuck Wolber , Chris Wedgwood , Michael Krufky , Domenico Andreoli , torvalds@linux-foundation.org, akpm@linux-foundation.org, alan@lxorguk.ukuu.org.uk, Dave Young , Greg KH Subject: Re: [patch 04/13] param_sysfs_builtin memchr argument fix Message-ID: <20071115175810.GC24531@suse.de> References: <20071115042610.731859958@mini.kroah.org> <20071115060927.GE7602@kroah.com> <473C6FCF.6010501@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <473C6FCF.6010501@redhat.com> User-Agent: Mutt/1.5.16 (2007-06-09) Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Nov 15, 2007 at 11:11:59AM -0500, Chuck Ebbert wrote: > On 11/15/2007 01:09 AM, Greg KH wrote: > > -stable review patch. If anyone has any objections, please let us know. > > > > ------------------ > > From: Dave Young > > > > patch faf8c714f4508207a9c81cc94dafc76ed6680b44 in mainline. > > > > If memchr argument is longer than strlen(kp->name), there will be some > > weird result. > > > > It will casuse duplicate filenames in sysfs for the "nousb". kernel > > warning messages are as bellow: > > > > Needs an additional fix: > > Commit: 22800a2830ec07e7cc5c837999890ac47cc7f5de > fix param_sysfs_builtin name length check That just went in yesterday, right? If so, it will have to wait until the next -stable review cycle, unless it's totally broken without that change. Is it? thanks, greg k-h