diff options
| author | paco <paco@voltanet.io> | 2018-06-20 19:11:18 +0200 |
|---|---|---|
| committer | paco <paco@voltanet.io> | 2018-06-20 19:14:01 +0200 |
| commit | fa3016309b33395c02cf10e7e198517c5b81e55a (patch) | |
| tree | 0de837fc4e142d8b479766f79212d50688bdd3c3 /bgpd/bgp_clist.c | |
| parent | 76cd18c9ddfbdb8f36b60fac61ebb4d64d12d079 (diff) | |
bgpd, lib: null check (Coverity 1436344, 1451361)
Signed-off-by: F. Aragon <paco@voltanet.io>
Diffstat (limited to 'bgpd/bgp_clist.c')
| -rw-r--r-- | bgpd/bgp_clist.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/bgpd/bgp_clist.c b/bgpd/bgp_clist.c index 7cf1477549..0ffbe174ed 100644 --- a/bgpd/bgp_clist.c +++ b/bgpd/bgp_clist.c @@ -1054,6 +1054,9 @@ int extcommunity_list_set(struct community_list_handler *ch, const char *name, struct ecommunity *ecom = NULL; regex_t *regex = NULL; + if (str == NULL) + return COMMUNITY_LIST_ERR_MALFORMED_VAL; + entry = NULL; /* Get community list. */ @@ -1089,7 +1092,7 @@ int extcommunity_list_set(struct community_list_handler *ch, const char *name, entry = community_entry_new(); entry->direct = direct; entry->style = style; - entry->any = (str ? 0 : 1); + entry->any = 0; if (ecom) entry->config = ecommunity_ecom2str( ecom, ECOMMUNITY_FORMAT_COMMUNITY_LIST, 0); |