diff options
| author | Paul Jakma <paul.jakma@sun.com> | 2006-05-12 22:51:49 +0000 |
|---|---|---|
| committer | Paul Jakma <paul.jakma@sun.com> | 2006-05-12 22:51:49 +0000 |
| commit | 214a4454376e8c96990e4c2ee6edea891c1f3166 (patch) | |
| tree | b3dad9e4643c4959247ef8106929fb1ede2ea6e3 | |
| parent | 894f8b6602f786db42da58c48b1338df5fa98732 (diff) | |
[ospfd] Fix leak in area-range-cost command, CID #46.
2006-05-11 Paul Jakma <paul.jakma@sun.com>
* ospf_abr.c: (ospf_area_range_cost_set) Shouldn't create a new
range, should just lookup to see if one exists, the new range
is just leaked. Fixes CID #46.
| -rw-r--r-- | ospfd/ChangeLog | 3 | ||||
| -rw-r--r-- | ospfd/ospf_abr.c | 2 |
2 files changed, 4 insertions, 1 deletions
diff --git a/ospfd/ChangeLog b/ospfd/ChangeLog index a66cfa06..480e54e3 100644 --- a/ospfd/ChangeLog +++ b/ospfd/ChangeLog @@ -8,6 +8,9 @@ ospf_area_lookup_by_area_id, fixes Coverity CID #69 * ospf_route.c: (ospf_route_delete_same_ext) Fix deref before NULL check by moving into check-protected block, fix CID #49. + * ospf_abr.c: (ospf_area_range_cost_set) Shouldn't create a new + range, should just lookup to see if one exists, the new range + is just leaked. Fixes CID #46. 2006-04-24 Paul Jakma <paul.jakma@sun.com> diff --git a/ospfd/ospf_abr.c b/ospfd/ospf_abr.c index 8f365da7..225cf6ed 100644 --- a/ospfd/ospf_abr.c +++ b/ospfd/ospf_abr.c @@ -244,7 +244,7 @@ ospf_area_range_cost_set (struct ospf *ospf, struct in_addr area_id, if (area == NULL) return 0; - range = ospf_area_range_new (p); + range = ospf_area_range_lookup (area, p); if (range == NULL) return 0; |