Don't take the address of a variable whose scope is about to disappear.

[tinc] / src / net_packet.c

diff --git a/src/net_packet.c b/src/net_packet.c
index 67ebc22..5290fd6 100644 (file)
--- a/src/net_packet.c
+++ b/src/net_packet.c
@@ -151,7 +151,7 @@ static void send_mtu_probe_handler(int fd, short events, void *data) {
        }
 
 end:
-       event_add(&n->mtuevent, &(struct timeval){timeout, 0});
+       event_add(&n->mtuevent, &(struct timeval){timeout, rand() % 100000});
 }
 
 void send_mtu_probe(node_t *n) {
@@ -536,11 +536,11 @@ static void send_udppacket(node_t *n, vpn_packet_t *origpkt) {
 
        sockaddr_t *sa;
        int sock;
+       sockaddr_t broadcast;
 
        /* Overloaded use of priority field: -1 means local broadcast */
 
        if(origpriority == -1 && n->prevedge) {
-               sockaddr_t broadcast;
                broadcast.in.sin_family = AF_INET;
                broadcast.in.sin_addr.s_addr = -1;
                broadcast.in.sin_port = n->prevedge->address.in.sin_port;