lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 07 Jul 2008 22:39:05 +0200 From: Mattias Nissler <mattias.nissler@....de> To: Vegard Nossum <vegard.nossum@...il.com> Cc: Stefano Brivio <stefano.brivio@...imi.it>, "John W. Linville" <linville@...driver.com>, "David S. Miller" <davem@...emloft.net>, Ingo Molnar <mingo@...e.hu>, Pekka Enberg <penberg@...helsinki.fi>, netdev@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH] rc80211_pid: Fix fast_start parameter handling On Mon, 2008-07-07 at 22:31 +0200, Vegard Nossum wrote: > Hi, > > On Mon, Jul 7, 2008 at 10:23 PM, Mattias Nissler <mattias.nissler@....de> wrote: > > This removes the fast_start parameter from the rc_pid parameters information > > and instead uses the parameter macro when initializing the rc_pid state. Since the > > parameter is only used on initialization, there is no point of making exporting > > it via debugfs. This also fixes uninitialized memory references to the > > fast_start and norm_offset parameters. > > > > Signed-off-by: Mattias Nissler <mattias.nissler@....de> > > --- > > > > This should fix the problem. I think this should go into mainline ASAP. > > Was this code actually causing some problem/regression? If so, can we > please have a reference to bugzilla/prior conversations in the patch > description? I think the problem probably didn't cause any major problem at all. At most, the rate scaling didn't work properly. Maybe the algorithm would even recover from the incorrect initialization (not too sure about this one, better ask Stefano about the rate sorting implications). And no, I have never seen any problem reports that relate to this bug. I wouldn't call it a regression either, since this is exactly the code that went mainline with the first version of rc80211_pid. > > Was the reason for the problem known before I posted the "use of > uninitialized memory" e-mail earlier today? If no, can you please > sneak in a little reference to kmemcheck there in the patch > description? The acknowledgement is sorely needed :-) It was unknown before. Just a sec, I'll post new version of the patch that mentions kmemcheck ;-) Mattias -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists