This becomes especially important after the tun patch sitting in the -mm
tree is applied, but it works standalone.  Please apply.  (See netdev
posting Message-Id: <[EMAIL PROTECTED]>)

==
The effect of this bug was made worse by Brian Braunstein's patch to keep
net_device.dev_addr and tun.dev_addr in sync.

Signed-off-by: Rusty Russell <[EMAIL PROTECTED]>

diff -r fd8c40f4f533 drivers/net/tun.c
--- a/drivers/net/tun.c Fri Mar 30 17:02:19 2007 +1000
+++ b/drivers/net/tun.c Sat Mar 31 10:30:47 2007 +1000
@@ -507,9 +507,7 @@ static int tun_set_iff(struct file *file
                tun->flags = flags;
                /* Be promiscuous by default to maintain previous behaviour. */
                tun->if_flags = IFF_PROMISC;
-               /* Generate random Ethernet address. */
-               *(u16 *)tun->dev_addr = htons(0x00FF);
-               get_random_bytes(tun->dev_addr + sizeof(u16), 4);
+               random_ether_addr(tun->dev_addr);
                memset(tun->chr_filter, 0, sizeof tun->chr_filter);
 
                tun_net_init(dev);


-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to