]> git.puffer.fish Git - mirror/frr.git/commitdiff
pimd: Fixup oil_if_has and oil_if_set to be consistent 11372/head
authorDonald Sharp <sharpd@nvidia.com>
Thu, 9 Jun 2022 11:06:30 +0000 (07:06 -0400)
committerDonald Sharp <sharpd@nvidia.com>
Thu, 9 Jun 2022 13:37:33 +0000 (09:37 -0400)
the v4 and v6 versions were different.  Make them the same.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>
pimd/pim_oil.c
pimd/pim_oil.h

index 9434ae07b703496ed52316974234f673b642de4f..8fa2c96034c0123042f751daf9f427bd2c40d126 100644 (file)
@@ -255,7 +255,7 @@ int pim_channel_del_oif(struct channel_oil *channel_oil, struct interface *oif,
                return 0;
        }
 
-       oil_if_set(channel_oil, pim_ifp->mroute_vif_index, false);
+       oil_if_set(channel_oil, pim_ifp->mroute_vif_index, 0);
        /* clear mute; will be re-evaluated when the OIF becomes valid again */
        channel_oil->oif_flags[pim_ifp->mroute_vif_index] &= ~PIM_OIF_FLAG_MUTE;
 
index eed4b203cd627818fa26b8bbfe9e664b177085eb..d0c4921d0744ff5dc255b720cc7e59bca9615ce2 100644 (file)
@@ -130,9 +130,9 @@ static inline vifi_t *oil_parent(struct channel_oil *c_oil)
        return &c_oil->oil.mfcc_parent;
 }
 
-static inline uint8_t oil_if_has(struct channel_oil *c_oil, vifi_t ifi)
+static inline bool oil_if_has(struct channel_oil *c_oil, vifi_t ifi)
 {
-       return c_oil->oil.mfcc_ttls[ifi];
+       return !!c_oil->oil.mfcc_ttls[ifi];
 }
 
 static inline void oil_if_set(struct channel_oil *c_oil, vifi_t ifi, uint8_t set)
@@ -166,7 +166,8 @@ static inline bool oil_if_has(struct channel_oil *c_oil, mifi_t ifi)
        return !!IF_ISSET(ifi, &c_oil->oil.mf6cc_ifset);
 }
 
-static inline void oil_if_set(struct channel_oil *c_oil, mifi_t ifi, bool set)
+static inline void oil_if_set(struct channel_oil *c_oil, mifi_t ifi,
+                             uint8_t set)
 {
        if (set)
                IF_SET(ifi, &c_oil->oil.mf6cc_ifset);