]> git.proxmox.com Git - mirror_frr.git/commitdiff
eigrpd: Prevent uninitialized value from being used
authorDonald Sharp <sharpd@nvidia.com>
Sat, 30 Jan 2021 18:38:32 +0000 (13:38 -0500)
committerDonald Sharp <sharpd@nvidia.com>
Sat, 30 Jan 2021 18:38:32 +0000 (13:38 -0500)
valgrind is finding:

2141982-==2141982== Conditional jump or move depends on uninitialised value(s)
2141982:==2141982==    at 0x11A7A6: eigrp_metrics_is_same (eigrp_metric.c:134)
2141982-==2141982==    by 0x120360: eigrp_topology_update_distance (eigrp_topology.c:374)
2141982-==2141982==    by 0x124F01: eigrp_get_fsm_event (eigrp_fsm.c:284)
2141982-==2141982==    by 0x12519E: eigrp_fsm_event (eigrp_fsm.c:419)
2141982-==2141982==    by 0x1206A1: eigrp_topology_neighbor_down (eigrp_topology.c:518)
2141982-==2141982==    by 0x11AB3A: eigrp_nbr_delete (eigrp_neighbor.c:178)
2141982-==2141982==    by 0x124494: eigrp_finish_final (eigrpd.c:271)
2141982-==2141982==    by 0x1245A8: eigrp_finish (eigrpd.c:247)
2141982-==2141982==    by 0x124630: eigrp_terminate (eigrpd.c:240)
2141982-==2141982==    by 0x11344B: sigint (eigrp_main.c:112)
2141982-==2141982==    by 0x48F5F32: quagga_sigevent_process (sigevent.c:130)

Prevent this from happening.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>
eigrpd/eigrp_topology.c

index 1b7e9fc15bb37a94ed47da75bd260a67bd3fe38c..6da7756f84bf5f23d20d430eb55275f248578ad9 100644 (file)
@@ -508,6 +508,7 @@ void eigrp_topology_neighbor_down(struct eigrp *eigrp,
                        if (entry->adv_router != nbr)
                                continue;
 
+                       memset(&msg, 0, sizeof(msg));
                        msg.metrics.delay = EIGRP_MAX_METRIC;
                        msg.packet_type = EIGRP_OPC_UPDATE;
                        msg.eigrp = eigrp;