]> git.puffer.fish Git - mirror/frr.git/commitdiff
bgpd: Convert flags_invert/flags_override to uint64_t 12842/head
authorDonatas Abraitis <donatas@opensourcerouting.org>
Sat, 18 Feb 2023 07:58:37 +0000 (09:58 +0200)
committerDonatas Abraitis <donatas@opensourcerouting.org>
Sun, 19 Feb 2023 10:28:54 +0000 (12:28 +0200)
peer->af_flags got this correctly.
peer->flags were already converted a time ago, but these were missed...

Let's fix this.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
bgpd/bgpd.h

index b5faa4d3534cb7997b8422d421f40a7d675877dd..91f0de0cea6df6e4face643beb075dc96a8d9829 100644 (file)
@@ -1270,7 +1270,7 @@ struct peer {
         * so if a flag is unset, the corresponding override flag is unset too.
         * However if a flag is set, the corresponding override flag is set.
         */
-       uint32_t flags_override;
+       uint64_t flags_override;
        /*
         * Parallel array to flags that indicates whether the default behavior
         * of *flags_override* should be inverted. If a flag is unset and the
@@ -1308,11 +1308,13 @@ struct peer {
         * inversion state of the flag differs between peer and peer-group, the
         * newly set value must equal to the inverted state of the peer-group.
         */
-       uint32_t flags_invert;
+       uint64_t flags_invert;
        /*
         * Effective array for storing the peer/peer-group flags. In case of a
         * peer-group, the peer-specific overrides (see flags_override and
         * flags_invert) must be respected.
+        * When changing the structure of flags/af_flags, do not forget to
+        * change flags_invert/flags_override too.
         */
        uint64_t flags;
 #define PEER_FLAG_PASSIVE                   (1ULL << 0) /* passive mode */