- (dtucker) [monitor.c] Don't free buffers in audit functions, monitor_read

   will free as needed.  ok tim@ djm@

diff --git a/ChangeLog b/ChangeLog
index ed2e61fc54bb3005a3571fb4ce0415de14c771a6..850ee2eb58f701ec6395d3ec4fa41369f27ceb44 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -7,6 +7,8 @@
      [scp.c]
      since ssh has xstrdup, use it instead of strdup+test. unbreaks -Werror
      build
+ - (dtucker) [monitor.c] Don't free buffers in audit functions, monitor_read
+   will free as needed.  ok tim@ djm@
 
 20050331
  - (dtucker) OpenBSD CVS Sync

diff --git a/monitor.c b/monitor.c
index c6a42e709333be2238d7d8c08fe95cb58260ef23..9dca9c8034c146260017f2e04ef41de10ecc8df9 100644 (file)
--- a/monitor.c
+++ b/monitor.c
@@ -1525,7 +1525,6 @@ mm_answer_audit_event(int socket, Buffer *m)
        debug3("%s entering", __func__);
 
        event = buffer_get_int(m);
-       buffer_free(m);
        switch(event) {
        case SSH_AUTH_FAIL_PUBKEY:
        case SSH_AUTH_FAIL_HOSTBASED:
@@ -1554,7 +1553,6 @@ mm_answer_audit_command(int socket, Buffer *m)
        /* sanity check command, if so how? */
        audit_run_command(cmd);
        xfree(cmd);
-       buffer_free(m);
        return (0);
 }
 #endif /* SSH_AUDIT_EVENTS */