batman-adv: Don't leak information through uninitialized packet fields
authorSven Eckelmann <sven@narfation.org>
Thu, 28 Jun 2012 09:56:52 +0000 (11:56 +0200)
committerAntonio Quartulli <ordex@autistici.org>
Sun, 1 Jul 2012 20:47:24 +0000 (22:47 +0200)
The reserved fields in batman-adv packets are not set to a constant value. The
content of these memory regions is leaked unintentionally to the network.

This regression was introduced in 3b27ffb00fbe9d9189715ea13ce8712e2f0cb0c5

Signed-off-by: Sven Eckelmann <sven@narfation.org>
Signed-off-by: Antonio Quartulli <ordex@autistici.org>
net/batman-adv/soft-interface.c
net/batman-adv/translation-table.c
net/batman-adv/vis.c

index b7c655cf626aa980561b922a86dd3cb23d18fc56..9e4bb61301ec1e155b9013585c5a02dc371bdc26 100644 (file)
@@ -214,6 +214,7 @@ static int batadv_interface_tx(struct sk_buff *skb,
 
                /* batman packet type: broadcast */
                bcast_packet->header.packet_type = BATADV_BCAST;
+               bcast_packet->reserved = 0;
 
                /* hw address of first interface is the orig mac because only
                 * this mac is known throughout the mesh
index 245cc9a068d89f7b526874e586f1874ac7fb660b..a438f4b582fc8adccba3a67dfb33326b1ecf4606 100644 (file)
@@ -2052,6 +2052,7 @@ static void batadv_send_roam_adv(struct batadv_priv *bat_priv, uint8_t *client,
        roam_adv_packet->header.packet_type = BATADV_ROAM_ADV;
        roam_adv_packet->header.version = BATADV_COMPAT_VERSION;
        roam_adv_packet->header.ttl = BATADV_TTL;
+       roam_adv_packet->reserved = 0;
        primary_if = batadv_primary_if_get_selected(bat_priv);
        if (!primary_if)
                goto out;
index f09cc9ad6ad84b1a119fa3b125f02f64c10758c8..2a2ea06814695f4356f38de38c83cd3465184630 100644 (file)
@@ -589,6 +589,7 @@ static int batadv_generate_vis_packet(struct batadv_priv *bat_priv)
        packet->header.ttl = BATADV_TTL;
        packet->seqno = htonl(ntohl(packet->seqno) + 1);
        packet->entries = 0;
+       packet->reserved = 0;
        skb_trim(info->skb_packet, sizeof(*packet));
 
        if (packet->vis_type == BATADV_VIS_TYPE_CLIENT_UPDATE) {
@@ -890,6 +891,7 @@ int batadv_vis_init(struct batadv_priv *bat_priv)
        packet->header.packet_type = BATADV_VIS;
        packet->header.ttl = BATADV_TTL;
        packet->seqno = 0;
+       packet->reserved = 0;
        packet->entries = 0;
 
        INIT_LIST_HEAD(&bat_priv->vis_send_list);