diff options
| author | Paul Jakma <paul.jakma@sun.com> | 2006-10-15 23:50:16 +0000 |
|---|---|---|
| committer | Paul Jakma <paul.jakma@sun.com> | 2006-10-15 23:50:16 +0000 |
| commit | ed3ebfa36b45fe487015e1918e848f0ff4500bff (patch) | |
| tree | 06e26892b62cc4f109e0d63718f3d7fa1c3bf54a /bgpd/ChangeLog | |
| parent | 53d9f67a18dc59fd688fce999cb35653010a54fb (diff) | |
[bgpd] Coverity CID #64: Needless NULL check, CID #64: Deref of potentially NULL pointer.
2006-10-15 Paul Jakma <paul.jakma@sun.com>
* bgp_packet.c: (bgp_update_packet) adv->rn can not be NULL,
check is bogus - changed to assert(), CID#64.
binfo is checked for NULL, but then dereferenced
unconditionally, fix, CID #63.
(bgp_withdraw_packet) Assert adv->rn is valid, as with
bgp_update_packet().
Diffstat (limited to 'bgpd/ChangeLog')
| -rw-r--r-- | bgpd/ChangeLog | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/bgpd/ChangeLog b/bgpd/ChangeLog index 56107329..83f9d493 100644 --- a/bgpd/ChangeLog +++ b/bgpd/ChangeLog @@ -2,6 +2,12 @@ * bgp_route.c: (bgp_table_stats_walker) NULL deref if table is empty, bgp_table_top may return NULL, Coverity CID#73. + * bgp_packet.c: (bgp_update_packet) adv->rn can not be NULL, + check is bogus - changed to assert(), CID#64. + binfo is checked for NULL, but then dereferenced + unconditionally, fix, CID #63. + (bgp_withdraw_packet) Assert adv->rn is valid, as with + bgp_update_packet(). 2006-10-14 Paul Jakma <paul.jakma@sun.com> |