Re: Linux 2.6.7

From: viro
Date: Wed Jun 16 2004 - 07:19:54 EST


On Wed, Jun 16, 2004 at 01:13:29PM +0200, Tomas Szepe wrote:
> On Jun-15 2004, Tue, 22:56 -0700
> Linus Torvalds <torvalds@xxxxxxxx> 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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/