diff options
| author | Quentin Young <qlyoung@cumulusnetworks.com> | 2020-02-04 13:19:37 -0500 |
|---|---|---|
| committer | Quentin Young <qlyoung@cumulusnetworks.com> | 2020-02-04 13:19:37 -0500 |
| commit | e1b36e132b7eda2d230f2f95945e231df37a42f5 (patch) | |
| tree | 8a40165810e2ddc3f3d3ca0709134554b766b5ca /zebra/zebra_mlag.c | |
| parent | b3ba5dc7fe769f3719b2812e4fdb9f8f8271032d (diff) | |
*: remove null check before XFREE
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
Diffstat (limited to 'zebra/zebra_mlag.c')
| -rw-r--r-- | zebra/zebra_mlag.c | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/zebra/zebra_mlag.c b/zebra/zebra_mlag.c index f6bd5f4586..fc7804a409 100644 --- a/zebra/zebra_mlag.c +++ b/zebra/zebra_mlag.c @@ -797,13 +797,11 @@ int zebra_mlag_protobuf_encode_client_data(struct stream *s, uint32_t *msg_type) } for (i = 0; i < mlag_msg.msg_cnt; i++) { - if (pay_load[i]->vrf_name) - XFREE(MTYPE_MLAG_PBUF, pay_load[i]->vrf_name); + XFREE(MTYPE_MLAG_PBUF, pay_load[i]->vrf_name); if (pay_load[i]->owner_id == MLAG_OWNER_INTERFACE && pay_load[i]->intf_name) XFREE(MTYPE_MLAG_PBUF, pay_load[i]->intf_name); - if (pay_load[i]) - XFREE(MTYPE_MLAG_PBUF, pay_load[i]); + XFREE(MTYPE_MLAG_PBUF, pay_load[i]); } XFREE(MTYPE_MLAG_PBUF, pay_load); if (cleanup == true) @@ -861,13 +859,11 @@ int zebra_mlag_protobuf_encode_client_data(struct stream *s, uint32_t *msg_type) } for (i = 0; i < mlag_msg.msg_cnt; i++) { - if (pay_load[i]->vrf_name) - XFREE(MTYPE_MLAG_PBUF, pay_load[i]->vrf_name); + XFREE(MTYPE_MLAG_PBUF, pay_load[i]->vrf_name); if (pay_load[i]->owner_id == MLAG_OWNER_INTERFACE && pay_load[i]->intf_name) XFREE(MTYPE_MLAG_PBUF, pay_load[i]->intf_name); - if (pay_load[i]) - XFREE(MTYPE_MLAG_PBUF, pay_load[i]); + XFREE(MTYPE_MLAG_PBUF, pay_load[i]); } XFREE(MTYPE_MLAG_PBUF, pay_load); if (cleanup) @@ -914,8 +910,7 @@ int zebra_mlag_protobuf_encode_client_data(struct stream *s, uint32_t *msg_type) mlag_lib_msgid_to_str(mlag_msg.msg_type, buf, sizeof(buf)), len); - if (hdr.data.data) - XFREE(MTYPE_MLAG_PBUF, hdr.data.data); + XFREE(MTYPE_MLAG_PBUF, hdr.data.data); return len; } |