diff options
author | anlan_cs <anlan_cs@tom.com> | 2022-01-11 02:45:10 +0100 |
---|---|---|
committer | anlan_cs <anlan_cs@tom.com> | 2022-01-11 05:24:24 +0100 |
commit | ff5c476dc2ad09f35e423c5ef7773f5fa45d4a79 (patch) | |
tree | 26f1b1ea8b7aa7fe593ab23aaaccb84c6da47ce5 /ospf6d/ospf6_asbr.c | |
parent | Merge pull request #10299 from donaldsharp/various_stuff (diff) | |
download | frr-ff5c476dc2ad09f35e423c5ef7773f5fa45d4a79.tar.xz frr-ff5c476dc2ad09f35e423c5ef7773f5fa45d4a79.zip |
ospfd,ospf6d: make clear the comparison of routemap name
Comparison of the two pointer is confusing, they have no relevance
except the time both of them are empty.
Additionly modify one variable name and correct some comment words, they
are same in both ospfd and ospf6d.
Signed-off-by: anlan_cs <anlan_cs@tom.com>
Diffstat (limited to 'ospf6d/ospf6_asbr.c')
-rw-r--r-- | ospf6d/ospf6_asbr.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/ospf6d/ospf6_asbr.c b/ospf6d/ospf6_asbr.c index 99f30a4a0..5cb6244bb 100644 --- a/ospf6d/ospf6_asbr.c +++ b/ospf6d/ospf6_asbr.c @@ -1936,13 +1936,13 @@ DEFPY (ospf6_default_route_originate, if (mtype_str == NULL) mtype = -1; - /* To check ,if user is providing same route map */ - if ((rtmap == ROUTEMAP_NAME(red)) - || (rtmap && ROUTEMAP_NAME(red) - && (strcmp(rtmap, ROUTEMAP_NAME(red)) == 0))) + /* To check if user is providing same route map */ + if ((!rtmap && !ROUTEMAP_NAME(red)) || + (rtmap && ROUTEMAP_NAME(red) && + (strcmp(rtmap, ROUTEMAP_NAME(red)) == 0))) sameRtmap = true; - /* Don't allow if the same lsa is aleardy originated. */ + /* Don't allow if the same lsa is already originated. */ if ((sameRtmap) && (red->dmetric.type == mtype) && (red->dmetric.value == mval) && (cur_originate == default_originate)) |