diff options
| author | ajs <ajs> | 2005-07-26 19:55:31 +0000 |
|---|---|---|
| committer | ajs <ajs> | 2005-07-26 19:55:31 +0000 |
| commit | 7907c6c9d34a4f19dd7d4d8d81c3c8ae5000ee07 (patch) | |
| tree | c57450ba246a26d4c37f1c2339b977d8293ee04d /lib/ChangeLog | |
| parent | 330009f7b3742462ebd90f9c16f1ab734344b68c (diff) | |
2005-07-26 Andrew J. Schorr <ajschorr@alumni.princeton.edu>
* prefix.c: (prefix_ipv4_new, prefix_ipv6_new): Call prefix_new
to allocate the memory to make sure that all struct prefix pointers
point to objects of the same length (avoids memory overruns
on struct prefix assignments).
(prefix_ipv4_free, prefix_ipv6_free): Simply call prefix_free.
It is interesting to note that these functions are never actually
called anywhere in the code. Instead prefix_free was already
being called directly, despite the previous MTYPE incompatibility.
[backport candidate]
Diffstat (limited to 'lib/ChangeLog')
| -rw-r--r-- | lib/ChangeLog | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/lib/ChangeLog b/lib/ChangeLog index f67f2c1e..42e80bd2 100644 --- a/lib/ChangeLog +++ b/lib/ChangeLog @@ -1,4 +1,14 @@ 2005-07-26 Andrew J. Schorr <ajschorr@alumni.princeton.edu> + * prefix.c: (prefix_ipv4_new, prefix_ipv6_new): Call prefix_new + to allocate the memory to make sure that all struct prefix pointers + point to objects of the same length (avoids memory overruns + on struct prefix assignments). + (prefix_ipv4_free, prefix_ipv6_free): Simply call prefix_free. + It is interesting to note that these functions are never actually + called anywhere in the code. Instead prefix_free was already + being called directly, despite the previous MTYPE incompatibility. + +2005-07-26 Andrew J. Schorr <ajschorr@alumni.princeton.edu> * prefix.c: (ip_masklen) While loop should test that 'pnt' pointer is in range before dereferencing it. |