![]() | |||||||||||||
Messages in this thread
Patch in this message |
On Wed, Jun 16, 2004 at 01:13:29PM +0200, Tomas Szepe wrote: > On Jun-15 2004, Tue, 22:56 -0700 > Linus Torvalds <torvalds@osdl.org> wrote: > > > Summary of changes from v2.6.7-rc3 to v2.6.7 > [snip] > > 2.6.7's airo.ko (unlike 2.6.6's) won't allow the user to set > ESSID via "echo myessid >/proc/driver/aironet/ethX/SSID". > > Changes like this shouldn't probably be made in the middle > of a stable series. Changes like this are called bugs. The thing is, original variant of function (actually, both read and write) was also buggy and trivially exploitable, so fixing it was needed. Fscking it up was not, obviously. Fix follows; see if it works for you. --- RC7/drivers/net/wireless/airo.c Mon Jun 7 19:21:27 2004 +++ RC7-current/drivers/net/wireless/airo.c Wed Jun 16 08:11:50 2004 @@ -4527,6 +4527,8 @@ len = priv->maxwritelen - pos; if (copy_from_user(priv->wbuffer + pos, buffer, len)) return -EFAULT; + if (pos + len > priv->writelen) + priv->writelen = pos + len; *offset = pos + len; return len; } - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ | ||||||||||||
| Last update: 2005-03-22 14:03 [from the cache] ©2003-2008 | |||||||||||||