diff options
author | Claudio Jeker <claudio@cvs.openbsd.org> | 2007-01-24 13:24:52 +0000 |
---|---|---|
committer | Claudio Jeker <claudio@cvs.openbsd.org> | 2007-01-24 13:24:52 +0000 |
commit | 85aa5c0549251ff80ea2a312c425cbb4a518da47 (patch) | |
tree | 92471fdd6b7fd10b0012b02b0f416be7b3e7bbbb /usr.bin/cvs | |
parent | 1850f70f87264219304352fed59c8b7ed8fc66d2 (diff) |
Lock the nexthop while parsing an update by increasing the reference count.
This is needed because the nexthop is not yet linked to the aspath attributes
and so a withdraw in the same update imsg could remove this nexthop which in
turn causes a use after free error when the prefix is added later on.
The order of parsing (attributes, withdraws, prefixes instead of withdraws,
attributes, prefixes) was reversed for multiprotocol support.
This should fix all strange nexthop crashes seen by various people.
Tested and OK henning@
Diffstat (limited to 'usr.bin/cvs')
0 files changed, 0 insertions, 0 deletions