Rewrite timeout computation in a way that is less prone to overflow.

(Problem reported by Peter Sylvester.)

diff --git a/ssl/ssl_sess.c b/ssl/ssl_sess.c
index 5286964e694142b651b15d997aad77ec817813f1..7a4bb92b5c2ced151bbda55ab4529981a43e2428 100644 (file)
--- a/ssl/ssl_sess.c
+++ b/ssl/ssl_sess.c
@@ -389,7 +389,7 @@ int ssl_get_prev_session(SSL *s, unsigned char *session_id, int len)
        CRYPTO_add(&ret->references,1,CRYPTO_LOCK_SSL_SESSION);
 #endif
 
-       if ((long)(ret->time+ret->timeout) < (long)time(NULL)) /* timeout */
+       if (ret->timeout < (long)(time(NULL) - ret->time)) /* timeout */
                {
                s->ctx->stats.sess_timeout++;
                /* remove it from the cache */