[openssh] - add audit of destruction the session keys
Jan F. Chadima
jfch2222 at fedoraproject.org
Wed Jan 12 10:10:17 UTC 2011
commit 5c20fa8d2d98a81f95fca7c18cca732836a0e41a
Author: Jan F <jfch at kerberos.example.com>
Date: Wed Jan 12 11:09:58 2011 +0100
- add audit of destruction the session keys
openssh-5.6p1-audit4.patch | 232 ++++++++++++++++++++++++++++++++++++++++++++
openssh-5.6p1-wIm.patch | 74 ++++++++++++++
openssh.spec | 12 ++-
3 files changed, 317 insertions(+), 1 deletions(-)
---
diff --git a/openssh-5.6p1-audit4.patch b/openssh-5.6p1-audit4.patch
new file mode 100644
index 0000000..6614f27
--- /dev/null
+++ b/openssh-5.6p1-audit4.patch
@@ -0,0 +1,232 @@
+diff -up openssh-5.6p1/audit-bsm.c.audit4 openssh-5.6p1/audit-bsm.c
+--- openssh-5.6p1/audit-bsm.c.audit4 2011-01-12 10:48:20.000000000 +0100
++++ openssh-5.6p1/audit-bsm.c 2011-01-12 10:48:54.000000000 +0100
+@@ -395,4 +395,10 @@ audit_kex_body(int ctos, char *enc, char
+ {
+ /* not implemented */
+ }
++
++void
++audit_session_key_free_body(int ctos)
++{
++ /* not implemented */
++}
+ #endif /* BSM */
+diff -up openssh-5.6p1/audit.c.audit4 openssh-5.6p1/audit.c
+--- openssh-5.6p1/audit.c.audit4 2011-01-12 10:45:58.000000000 +0100
++++ openssh-5.6p1/audit.c 2011-01-12 10:51:16.000000000 +0100
+@@ -152,6 +152,12 @@ audit_kex(int ctos, char *enc, char *mac
+ PRIVSEP(audit_kex_body(ctos, enc, mac, comp));
+ }
+
++void
++audit_session_key_free(int ctos)
++{
++ PRIVSEP(audit_session_key_free_body(ctos));
++}
++
+ # ifndef CUSTOM_SSH_AUDIT_EVENTS
+ /*
+ * Null implementations of audit functions.
+@@ -254,5 +260,13 @@ audit_kex_body(int ctos, char *enc, char
+ debug("audit procol negotiation euid %d direction %d cipher %s mac %s compresion %s",
+ geteuid(), ctos, enc, mac, compress);
+ }
++
++/*
++ * This will be called on succesfull session key discard
++ */
++audit_session_key_free_body(int ctos)
++{
++ debug("audit session key discard euid %d direction %d", geteuid(), ctos);
++}
+ # endif /* !defined CUSTOM_SSH_AUDIT_EVENTS */
+ #endif /* SSH_AUDIT_EVENTS */
+diff -up openssh-5.6p1/audit.h.audit4 openssh-5.6p1/audit.h
+--- openssh-5.6p1/audit.h.audit4 2011-01-12 10:45:58.000000000 +0100
++++ openssh-5.6p1/audit.h 2011-01-12 10:45:59.000000000 +0100
+@@ -60,5 +60,7 @@ void audit_unsupported(int);
+ void audit_kex(int, char *, char *, char *);
+ void audit_unsupported_body(int);
+ void audit_kex_body(int, char *, char *, char *);
++void audit_session_key_free(int ctos);
++void audit_session_key_free_body(int ctos);
+
+ #endif /* _SSH_AUDIT_H */
+diff -up openssh-5.6p1/audit-linux.c.audit4 openssh-5.6p1/audit-linux.c
+--- openssh-5.6p1/audit-linux.c.audit4 2011-01-12 10:45:58.000000000 +0100
++++ openssh-5.6p1/audit-linux.c 2011-01-12 10:48:00.000000000 +0100
+@@ -174,13 +174,14 @@ audit_unsupported_body(int what)
+ #endif
+ }
+
++const static char *direction[] = { "from-server", "from-client", "both" };
++
+ void
+ audit_kex_body(int ctos, char *enc, char *mac, char *compress)
+ {
+ #ifdef AUDIT_CRYPTO_SESSION
+ char buf[AUDIT_LOG_SIZE];
+ int audit_fd, audit_ok;
+- const static char *direction[] = { "from-server", "from-client", "both" };
+ Cipher *cipher = cipher_by_name(enc);
+
+ snprintf(buf, sizeof(buf), "start direction=%s cipher=%s, ksize=%d rport=%d laddr=%s lport=%d",
+@@ -203,4 +204,22 @@ audit_kex_body(int ctos, char *enc, char
+ #endif
+ }
+
++void
++audit_session_key_free_body(int ctos)
++{
++ char buf[AUDIT_LOG_SIZE];
++ int audit_fd, audit_ok;
++
++ snprintf(buf, sizeof(buf), "destroy kind=session direction=%s", direction[ctos]);
++ audit_fd = audit_open();
++ if (audit_fd < 0)
++ return;
++ audit_ok = audit_log_acct_message(audit_fd, AUDIT_CRYPTO_KEY_USER, NULL,
++ buf, NULL, -1, NULL, get_remote_ipaddr(), NULL, 1);
++ audit_close(audit_fd);
++ /* do not abort if the error is EPERM and sshd is run as non root user */
++ if ((audit_ok < 0) && ((audit_ok != -1) || (getuid() == 0)))
++ error("cannot write into audit");
++}
++
+ #endif /* USE_LINUX_AUDIT */
+diff -up openssh-5.6p1/auditstub.c.audit4 openssh-5.6p1/auditstub.c
+--- openssh-5.6p1/auditstub.c.audit4 2011-01-12 10:45:58.000000000 +0100
++++ openssh-5.6p1/auditstub.c 2011-01-12 10:45:59.000000000 +0100
+@@ -37,3 +37,7 @@ audit_kex(int ctos, char *enc, char *mac
+ {
+ }
+
++void
++audit_session_key_free(int ctos)
++{
++}
+diff -up openssh-5.6p1/monitor.c.audit4 openssh-5.6p1/monitor.c
+--- openssh-5.6p1/monitor.c.audit4 2011-01-12 10:45:59.000000000 +0100
++++ openssh-5.6p1/monitor.c 2011-01-12 10:45:59.000000000 +0100
+@@ -180,6 +180,7 @@ int mm_answer_audit_event(int, Buffer *)
+ int mm_answer_audit_command(int, Buffer *);
+ int mm_answer_audit_unsupported_body(int, Buffer *);
+ int mm_answer_audit_kex_body(int, Buffer *);
++int mm_answer_audit_session_key_free_body(int, Buffer *);
+ #endif
+
+ static Authctxt *authctxt;
+@@ -230,6 +231,7 @@ struct mon_table mon_dispatch_proto20[]
+ {MONITOR_REQ_AUDIT_EVENT, MON_PERMIT, mm_answer_audit_event},
+ {MONITOR_REQ_AUDIT_UNSUPPORTED, MON_PERMIT, mm_answer_audit_unsupported_body},
+ {MONITOR_REQ_AUDIT_KEX, MON_PERMIT, mm_answer_audit_kex_body},
++ {MONITOR_REQ_AUDIT_SESSION_KEY_FREE, MON_PERMIT, mm_answer_audit_session_key_free_body},
+ #endif
+ #ifdef BSD_AUTH
+ {MONITOR_REQ_BSDAUTHQUERY, MON_ISAUTH, mm_answer_bsdauthquery},
+@@ -268,6 +270,7 @@ struct mon_table mon_dispatch_postauth20
+ {MONITOR_REQ_AUDIT_COMMAND, MON_PERMIT, mm_answer_audit_command},
+ {MONITOR_REQ_AUDIT_UNSUPPORTED, MON_PERMIT, mm_answer_audit_unsupported_body},
+ {MONITOR_REQ_AUDIT_KEX, MON_PERMIT, mm_answer_audit_kex_body},
++ {MONITOR_REQ_AUDIT_SESSION_KEY_FREE, MON_PERMIT, mm_answer_audit_session_key_free_body},
+ #endif
+ {0, 0, NULL}
+ };
+@@ -301,6 +304,7 @@ struct mon_table mon_dispatch_proto15[]
+ {MONITOR_REQ_AUDIT_EVENT, MON_PERMIT, mm_answer_audit_event},
+ {MONITOR_REQ_AUDIT_UNSUPPORTED, MON_PERMIT, mm_answer_audit_unsupported_body},
+ {MONITOR_REQ_AUDIT_KEX, MON_PERMIT, mm_answer_audit_kex_body},
++ {MONITOR_REQ_AUDIT_SESSION_KEY_FREE, MON_PERMIT, mm_answer_audit_session_key_free_body},
+ #endif
+ {0, 0, NULL}
+ };
+@@ -314,6 +318,7 @@ struct mon_table mon_dispatch_postauth15
+ {MONITOR_REQ_AUDIT_COMMAND, MON_PERMIT|MON_ONCE, mm_answer_audit_command},
+ {MONITOR_REQ_AUDIT_UNSUPPORTED, MON_PERMIT, mm_answer_audit_unsupported_body},
+ {MONITOR_REQ_AUDIT_KEX, MON_PERMIT, mm_answer_audit_kex_body},
++ {MONITOR_REQ_AUDIT_SESSION_KEY_FREE, MON_PERMIT, mm_answer_audit_session_key_free_body},
+ #endif
+ {0, 0, NULL}
+ };
+@@ -2252,4 +2257,18 @@ mm_answer_audit_kex_body(int sock, Buffe
+ return 0;
+ }
+
++int
++mm_answer_audit_session_key_free_body(int sock, Buffer *m)
++{
++ int ctos;
++
++ ctos = buffer_get_int(m);
++
++ audit_session_key_free_body(ctos);
++
++ buffer_clear(m);
++
++ mm_request_send(sock, MONITOR_ANS_AUDIT_SESSION_KEY_FREE, m);
++ return 0;
++}
+ #endif /* SSH_AUDIT_EVENTS */
+diff -up openssh-5.6p1/monitor.h.audit4 openssh-5.6p1/monitor.h
+--- openssh-5.6p1/monitor.h.audit4 2011-01-12 10:45:59.000000000 +0100
++++ openssh-5.6p1/monitor.h 2011-01-12 10:45:59.000000000 +0100
+@@ -68,6 +68,7 @@ enum monitor_reqtype {
+ MONITOR_REQ_JPAKE_CHECK_CONFIRM, MONITOR_ANS_JPAKE_CHECK_CONFIRM,
+ MONITOR_REQ_AUDIT_UNSUPPORTED, MONITOR_ANS_AUDIT_UNSUPPORTED,
+ MONITOR_REQ_AUDIT_KEX, MONITOR_ANS_AUDIT_KEX,
++ MONITOR_REQ_AUDIT_SESSION_KEY_FREE, MONITOR_ANS_AUDIT_SESSION_KEY_FREE,
+ };
+
+ struct mm_master;
+diff -up openssh-5.6p1/monitor_wrap.c.audit4 openssh-5.6p1/monitor_wrap.c
+--- openssh-5.6p1/monitor_wrap.c.audit4 2011-01-12 10:45:59.000000000 +0100
++++ openssh-5.6p1/monitor_wrap.c 2011-01-12 10:45:59.000000000 +0100
+@@ -1445,4 +1445,17 @@ mm_audit_kex_body(int ctos, char *cipher
+
+ buffer_free(&m);
+ }
++
++void
++mm_audit_session_key_free_body(int ctos)
++{
++ Buffer m;
++
++ buffer_init(&m);
++ buffer_put_int(&m, ctos);
++ mm_request_send(pmonitor->m_recvfd, MONITOR_REQ_AUDIT_SESSION_KEY_FREE, &m);
++ mm_request_receive_expect(pmonitor->m_recvfd, MONITOR_ANS_AUDIT_SESSION_KEY_FREE,
++ &m);
++ buffer_free(&m);
++}
+ #endif /* SSH_AUDIT_EVENTS */
+diff -up openssh-5.6p1/monitor_wrap.h.audit4 openssh-5.6p1/monitor_wrap.h
+--- openssh-5.6p1/monitor_wrap.h.audit4 2011-01-12 10:45:59.000000000 +0100
++++ openssh-5.6p1/monitor_wrap.h 2011-01-12 10:45:59.000000000 +0100
+@@ -76,6 +76,7 @@ void mm_audit_event(ssh_audit_event_t);
+ void mm_audit_run_command(const char *);
+ void mm_audit_unsupported_body(int);
+ void mm_audit_kex_body(int, char *, char *, char *);
++void mm_audit_session_key_free_body(int);
+ #endif
+
+ struct Session;
+diff -up openssh-5.6p1/packet.c.audit4 openssh-5.6p1/packet.c
+--- openssh-5.6p1/packet.c.audit4 2010-07-16 05:58:37.000000000 +0200
++++ openssh-5.6p1/packet.c 2011-01-12 10:45:59.000000000 +0100
+@@ -495,6 +495,7 @@ packet_close(void)
+ buffer_free(&active_state->compression_buffer);
+ buffer_compress_uninit();
+ }
++ audit_session_key_free(2);
+ cipher_cleanup(&active_state->send_context);
+ cipher_cleanup(&active_state->receive_context);
+ }
+@@ -749,6 +750,7 @@ set_newkeys(int mode)
+ }
+ if (active_state->newkeys[mode] != NULL) {
+ debug("set_newkeys: rekeying");
++ audit_session_key_free(mode);
+ cipher_cleanup(cc);
+ enc = &active_state->newkeys[mode]->enc;
+ mac = &active_state->newkeys[mode]->mac;
diff --git a/openssh-5.6p1-wIm.patch b/openssh-5.6p1-wIm.patch
new file mode 100644
index 0000000..c888bbf
--- /dev/null
+++ b/openssh-5.6p1-wIm.patch
@@ -0,0 +1,74 @@
+diff -up openssh-5.6p1/log.h.wIm openssh-5.6p1/log.h
+--- openssh-5.6p1/log.h.wIm 2008-06-13 02:22:54.000000000 +0200
++++ openssh-5.6p1/log.h 2011-01-11 10:35:32.000000000 +0100
+@@ -63,6 +63,7 @@ void verbose(const char *, ...) __at
+ void debug(const char *, ...) __attribute__((format(printf, 1, 2)));
+ void debug2(const char *, ...) __attribute__((format(printf, 1, 2)));
+ void debug3(const char *, ...) __attribute__((format(printf, 1, 2)));
++void debug_wIm(const char *);
+
+ void do_log(LogLevel, const char *, va_list);
+ void cleanup_exit(int) __attribute__((noreturn));
+diff -up openssh-5.6p1/Makefile.in.wIm openssh-5.6p1/Makefile.in
+--- openssh-5.6p1/Makefile.in.wIm 2010-05-12 08:51:39.000000000 +0200
++++ openssh-5.6p1/Makefile.in 2011-01-11 10:35:32.000000000 +0100
+@@ -69,7 +69,7 @@ LIBSSH_OBJS=acss.o authfd.o authfile.o b
+ cipher-bf1.o cipher-ctr.o cipher-3des1.o cleanup.o \
+ compat.o compress.o crc32.o deattack.o fatal.o hostfile.o \
+ log.o match.o md-sha256.o moduli.o nchan.o packet.o \
+- readpass.o rsa.o ttymodes.o xmalloc.o addrmatch.o \
++ readpass.o rsa.o ttymodes.o whereIam.o xmalloc.o addrmatch.o \
+ atomicio.o key.o dispatch.o kex.o mac.o uidswap.o uuencode.o misc.o \
+ monitor_fdpass.o rijndael.o ssh-dss.o ssh-rsa.o dh.o kexdh.o \
+ kexgex.o kexdhc.o kexgexc.o msg.o progressmeter.o dns.o \
+diff -up openssh-5.6p1/sshd.c.wIm openssh-5.6p1/sshd.c
+--- openssh-5.6p1/sshd.c.wIm 2010-04-16 07:56:22.000000000 +0200
++++ openssh-5.6p1/sshd.c 2011-01-11 10:35:32.000000000 +0100
+@@ -139,6 +139,9 @@ int deny_severity;
+
+ extern char *__progname;
+
++/* trace of fork processes */
++extern int whereIam;
++
+ /* Server configuration options. */
+ ServerOptions options;
+
+@@ -652,6 +655,7 @@ privsep_preauth(Authctxt *authctxt)
+ } else {
+ /* child */
+
++ whereIam = 1;
+ close(pmonitor->m_sendfd);
+
+ /* Demote the child */
+@@ -693,6 +697,7 @@ privsep_postauth(Authctxt *authctxt)
+ exit(0);
+ }
+
++ whereIam = 2;
+ close(pmonitor->m_sendfd);
+
+ /* Demote the private keys to public keys. */
+@@ -1299,6 +1304,8 @@ main(int ac, char **av)
+ Key *key;
+ Authctxt *authctxt;
+
++ whereIam = 0;
++
+ #ifdef HAVE_SECUREWARE
+ (void)set_auth_parameters(ac, av);
+ #endif
+diff -up openssh-5.6p1/whereIam.c.wIm openssh-5.6p1/whereIam.c
+--- openssh-5.6p1/whereIam.c.wIm 2011-01-11 10:35:32.000000000 +0100
++++ openssh-5.6p1/whereIam.c 2011-01-11 10:35:32.000000000 +0100
+@@ -0,0 +1,9 @@
++
++int whereIam = -1;
++
++void debug_wIm(const char *txt)
++{
++ debug("%s wIm = %d, euid=%d", txt, whereIam, geteuid());
++}
++
++
diff --git a/openssh.spec b/openssh.spec
index e37f2f3..9a2e8e8 100644
--- a/openssh.spec
+++ b/openssh.spec
@@ -71,7 +71,7 @@
# Do not forget to bump pam_ssh_agent_auth release if you rewind the main package release to 1
%define openssh_ver 5.6p1
-%define openssh_rel 21
+%define openssh_rel 22
%define pam_ssh_agent_ver 0.9.2
%define pam_ssh_agent_rel 29
@@ -93,12 +93,16 @@ Source3: sshd.init
Source4: http://prdownloads.sourceforge.net/pamsshagentauth/pam_ssh_agent_auth/pam_ssh_agent_auth-%{pam_ssh_agent_ver}.tar.bz2
Source5: pam_ssh_agent-rmheaders
+Patch100: openssh-5.6p1-wIm.patch
Patch0: openssh-5.6p1-redhat.patch
#https://bugzilla.mindrot.org/show_bug.cgi?id=1402
Patch1: openssh-5.6p1-audit.patch
Patch2: openssh-5.6p1-audit1a.patch
Patch3: openssh-5.6p1-audit2.patch
Patch4: openssh-5.6p1-audit3.patch
+Patch104: openssh-5.6p1-audit4.patch
+#not yet
+###Patch105: openssh-5.6p1-audit5.patch
#https://bugzilla.mindrot.org/show_bug.cgi?id=1640
Patch5: openssh-5.2p1-vendor.patch
Patch10: pam_ssh_agent_auth-0.9-build.patch
@@ -272,11 +276,14 @@ The module is most useful for su and sudo service stacks.
%prep
%setup -q -a 4
+#Do not enable by default
+###%patch100 -p1 -b .wIm
%patch0 -p1 -b .redhat
%patch1 -p1 -b .audit
%patch2 -p1 -b .audit1a
%patch3 -p1 -b .audit2
%patch4 -p1 -b .audit3
+%patch104 -p1 -b .audit4
%patch5 -p1 -b .vendor
%if %{pam_ssh_agent}
@@ -596,6 +603,9 @@ fi
%endif
%changelog
+* Wed Jan 12 2011 Jan F. Chadima <jchadima at redhat.com> - 5.6p1-22 + 0.9.2-29
+- add audit of destruction the session keys
+
* Fri Dec 10 2010 Jan F. Chadima <jchadima at redhat.com> - 5.6p1-21 + 0.9.2-29
- reenable run sshd as non root user
- renable rekeying
More information about the scm-commits
mailing list