]> git.puffer.fish Git - matthieu/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)
committerIgor Ryzhov <iryzhov@nfware.com>
Tue, 16 Feb 2021 17:56:48 +0000 (20:56 +0300)
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 7676af15f283e4d8b941c5ef1005a86f5c151a8c..5247413d2f76d7bda50a17cb717ad6fbf59dbfe3 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;