Ticket #285: fix-ssl-again-1.4.19.patch

File fix-ssl-again-1.4.19.patch, 2.9 kB (added by hoffie, 3 months ago)

same patch as fix-ssl-again.patch (by stbuehler) against 1.4.19

  • a/NEWS

    old new  
    88  * added support for If-Range: <date> (#1346) 
    99  * added support for matching $HTTP["scheme"] in configs 
    1010  * fixed initgroups() called after chroot (#1384) 
     11  * Fix #285 again: read error after SSL_shutdown (thx marton.illes@balabit.com) and clear the error queue before some other calls 
    1112  * fixed case-sensitive check for Auth-Method (#1456) 
    1213  * execute fcgi app without /bin/sh if used as argument to spawn-fcgi (#1428) 
    1314  * fixed a bug that made /-prefixed extensions being handled also when 

  • a/src/connections.c

    old new  
    199199 
    200200        /* don't resize the buffer if we were in SSL_ERROR_WANT_* */ 
    201201 
     202        ERR_clear_error(); 
    202203        do { 
    203204                if (!con->ssl_error_want_reuse_buffer) { 
    204205                        b = buffer_init(); 
    …… 
    16681669                        } 
    16691670#ifdef USE_OPENSSL 
    16701671                        if (srv_sock->is_ssl) { 
    1671                                 int ret; 
     1672                                int ret, ssl_r; 
     1673                                unsigned long err; 
     1674                                ERR_clear_error(); 
    16721675                                switch ((ret = SSL_shutdown(con->ssl))) { 
    16731676                                case 1: 
    16741677                                        /* ok */ 
    16751678                                        break; 
    16761679                                case 0: 
    1677                                         SSL_shutdown(con->ssl); 
    1678                                         break; 
     1680                                        if (-1 != (ret = SSL_shutdown(con->ssl))) break; 
     1681 
     1682                                        // fall through 
    16791683                                default: 
    1680                                         log_error_write(srv, __FILE__, __LINE__, "sds", "SSL:", 
    1681                                                         SSL_get_error(con->ssl, ret), 
    1682                                                         ERR_error_string(ERR_get_error(), NULL)); 
    1683                                         return -1; 
     1684 
     1685                                        switch ((ssl_r = SSL_get_error(con->ssl, ret))) { 
     1686                                        case SSL_ERROR_WANT_WRITE: 
     1687                                        case SSL_ERROR_WANT_READ: 
     1688                                                break; 
     1689                                        case SSL_ERROR_SYSCALL: 
     1690                                                /* perhaps we have error waiting in our error-queue */ 
     1691                                                if (0 != (err = ERR_get_error())) { 
     1692                                                        do { 
     1693                                                                log_error_write(srv, __FILE__, __LINE__, "sdds", "SSL:", 
     1694                                                                                ssl_r, ret, 
     1695                                                                                ERR_error_string(err, NULL)); 
     1696                                                        } while((err = ERR_get_error())); 
     1697                                                } else { 
     1698                                                        log_error_write(srv, __FILE__, __LINE__, "sddds", "SSL (error):", 
     1699                                                                        ssl_r, r, errno, 
     1700                                                                        strerror(errno)); 
     1701                                                } 
     1702         
     1703                                                break; 
     1704                                        default: 
     1705                                                while((err = ERR_get_error())) { 
     1706                                                        log_error_write(srv, __FILE__, __LINE__, "sdds", "SSL:", 
     1707                                                                        ssl_r, ret, 
     1708                                                                        ERR_error_string(err, NULL)); 
     1709                                                } 
     1710         
     1711                                                break; 
     1712                                        } 
    16841713                                } 
    16851714                        } 
     1715                        ERR_clear_error(); 
    16861716#endif 
    16871717 
    16881718                        switch(con->mode) { 

  • a/src/network_openssl.c

    old new  
    8585                         * 
    8686                         */ 
    8787 
     88                        ERR_clear_error(); 
    8889                        if ((r = SSL_write(ssl, offset, toSend)) <= 0) { 
    8990                                unsigned long err; 
    9091 
    …… 
    187188 
    188189                                close(ifd); 
    189190 
     191                                ERR_clear_error(); 
    190192                                if ((r = SSL_write(ssl, s, toSend)) <= 0) { 
    191193                                        unsigned long err; 
    192194