diff options
| author | Denil Vira <denil@cumulusnetworks.com> | 2015-07-23 08:47:21 -0700 |
|---|---|---|
| committer | Denil Vira <denil@cumulusnetworks.com> | 2015-07-23 08:47:21 -0700 |
| commit | 683de05f8708a43aabde961329b56b83cd856e31 (patch) | |
| tree | 62aafe9358921b39df19cfda0343fb05e9a866e0 /lib/stream.c | |
| parent | 43fdf718a25be053d6e36c8d0aecfaddc2a155cf (diff) | |
Fix bugs reported by coverity scan
1. Fix useless call in bgpd/bgp_mplsvpn.c
Coverity scan ID : 1302528. Calling "decode_label(pnt)" is only useful
for its return value, which is
ignored. Removed the call.
2. Fix logically dead code in lib/stream.c
Coverity Scan ID 1302488. Test for size==0 makes no sense, since assert
immediately before it
would not let this code happen.
3. Fix Free Pointer dereference in lib/filter.c
Coverity Scan ID 23056. access is accessed after free in
access_list_delete
Diffstat (limited to 'lib/stream.c')
| -rw-r--r-- | lib/stream.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/lib/stream.c b/lib/stream.c index cc5898a6db..fcdf56f64f 100644 --- a/lib/stream.c +++ b/lib/stream.c @@ -93,12 +93,6 @@ stream_new (size_t size) assert (size > 0); - if (size == 0) - { - zlog_warn ("stream_new(): called with 0 size!"); - return NULL; - } - s = XCALLOC (MTYPE_STREAM, sizeof (struct stream)); if (s == NULL) |