Skip to content

Commit ef96f2f

Browse files
authored
Merge pull request FRRouting#18993 from opensourcerouting/fix/bgp_crash_attributes_too_long
bgpd: Do not reuse the same adj->adv when flushing fifo (attributes too long)
2 parents 5323b09 + 6536bd6 commit ef96f2f

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

bgpd/bgp_updgrp_packet.c

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -764,9 +764,11 @@ struct bpacket *subgroup_update_packet(struct update_subgroup *subgrp)
764764
subgrp->update_group->id, subgrp->id);
765765

766766
/* Flush the FIFO update queue */
767-
while (adv)
768-
adv = bgp_advertise_clean_subgroup(
769-
subgrp, adj);
767+
while (adv) {
768+
struct bgp_adj_out *curr_adj = adv->adj;
769+
770+
adv = bgp_advertise_clean_subgroup(subgrp, curr_adj);
771+
}
770772
return NULL;
771773
}
772774

0 commit comments

Comments
 (0)