Clear Ethernet header when reading packets from a tun device.

This fixes a warning from valgrind about uninitialized bytes, which were being
sent to other nodes.

diff --git a/src/bsd/device.c b/src/bsd/device.c
index d6dd055..c91eb5e 100644 (file)
--- a/src/bsd/device.c
+++ b/src/bsd/device.c
@@ -238,6 +238,7 @@ static bool read_packet(vpn_packet_t *packet) {
                                        return false;
                        }
 
+                       memset(packet->data, 0, 12);
                        packet->len = lenin + 14;
                        break;
 
@@ -269,6 +270,7 @@ static bool read_packet(vpn_packet_t *packet) {
                                        return false;
                        }
 
+                       memset(packet->data, 0, 12);
                        packet->len = lenin + 10;
                        break;
                }

diff --git a/src/linux/device.c b/src/linux/device.c
index 5ae8947..f770f7f 100644 (file)
--- a/src/linux/device.c
+++ b/src/linux/device.c
@@ -155,6 +155,7 @@ static bool read_packet(vpn_packet_t *packet) {
                                return false;
                        }
 
+                       memset(packet->data, 0, 12);
                        packet->len = lenin + 10;
                        break;
                case DEVICE_TYPE_TAP:

diff --git a/src/solaris/device.c b/src/solaris/device.c
index 969b514..26121e2 100644 (file)
--- a/src/solaris/device.c
+++ b/src/solaris/device.c
@@ -151,6 +151,7 @@ static bool read_packet(vpn_packet_t *packet) {
                        return false;
        }
 
+       memset(packet->data, 0, 12);
        packet->len = lenin + 14;
 
        device_total_in += packet->len;