diff options
| author | Andrew J. Schorr <ajschorr@alumni.princeton.edu> | 2006-07-11 00:06:49 +0000 |
|---|---|---|
| committer | Andrew J. Schorr <ajschorr@alumni.princeton.edu> | 2006-07-11 00:06:49 +0000 |
| commit | 9d0a3260b2d1b57b7edfd3f72885d861883d4621 (patch) | |
| tree | 019901060c5b0b34154b084e9ae551af33319a6d /lib/ChangeLog | |
| parent | 74542d730198a37a872b7114643e29e99c551bcf (diff) | |
[lib] Do not call vty_close in vty_log_out to avoid possible free memory access
2006-07-10 Andrew J. Schorr <ajschorr@alumni.princeton.edu>
* vty.c: (vty_log_out) Do not call vty_close, because this could
result in a parent function's accessing the freed memory.
Instead, set status VTY_CLOSE and call shutdown(vty->fd, SHUT_RDWR).
And add a comment on vty_close.
Diffstat (limited to 'lib/ChangeLog')
| -rw-r--r-- | lib/ChangeLog | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/lib/ChangeLog b/lib/ChangeLog index 25df2657..02148671 100644 --- a/lib/ChangeLog +++ b/lib/ChangeLog @@ -1,5 +1,12 @@ 2006-07-10 Andrew J. Schorr <ajschorr@alumni.princeton.edu> + * vty.c: (vty_log_out) Do not call vty_close, because this could + result in a parent function's accessing the freed memory. + Instead, set status VTY_CLOSE and call shutdown(vty->fd, SHUT_RDWR). + And add a comment on vty_close. + +2006-07-10 Andrew J. Schorr <ajschorr@alumni.princeton.edu> + * vty.c: (vty_log_out, vty_read, vty_flush, vtysh_flush, vtysh_read) After an I/O error, must set vty->monitor to 0 before calling zlog_warn, otherwise an infinite recursion could occur |