From 3fdd82f2a8a776650f10f30c33546ce2dd42f073 Mon Sep 17 00:00:00 2001 From: Aris Adamantiadis Date: Wed, 5 Feb 2014 21:24:12 +0100 Subject: security: fix for vulnerability CVE-2014-0017 When accepting a new connection, a forking server based on libssh forks and the child process handles the request. The RAND_bytes() function of openssl doesn't reset its state after the fork, but simply adds the current process id (getpid) to the PRNG state, which is not guaranteed to be unique. This can cause several children to end up with same PRNG state which is a security issue. --- src/bind.c | 2 ++ 1 file changed, 2 insertions(+) (limited to 'src/bind.c') diff --git a/src/bind.c b/src/bind.c index 8d82d0d4..03d34031 100644 --- a/src/bind.c +++ b/src/bind.c @@ -458,6 +458,8 @@ int ssh_bind_accept_fd(ssh_bind sshbind, ssh_session session, socket_t fd){ return SSH_ERROR; } } + /* force PRNG to change state in case we fork after ssh_bind_accept */ + ssh_reseed(); return SSH_OK; } -- cgit