https://bz.apache.org/bugzilla/show_bug.cgi?id=59025

            Bug ID: 59025
           Summary: Incorrect OpenSSL version number used in comparisons
                    in Tomcat Native source
           Product: Tomcat Native
           Version: 1.2.4
          Hardware: All
                OS: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Library
          Assignee: dev@tomcat.apache.org
          Reporter: gkiessl...@teradici.com

Both 1.1.x and 1.2.x branches of Tomcat Native have numerous comparisons
between OPENSSL_VERSION_NUMBER and 0x10100000L, the latter being an alpha
release version of OpenSSL that only became available in December 2015 (the
1.2.0 version of Tomcat Native was released in October 2015):

[tomcat-native-1.2.0-src]# grep -r "OPENSSL_VERSION_NUMBER.*0x101" *
native/src/sslutils.c:#if OPENSSL_VERSION_NUMBER < 0x10100000L
native/src/sslutils.c:#if OPENSSL_VERSION_NUMBER < 0x10100000L
native/src/ssl.c:#if (OPENSSL_VERSION_NUMBER < 0x10100000L) ||
defined(OPENSSL_USE_DEPRECATED)
native/src/ssl.c:#if (OPENSSL_VERSION_NUMBER < 0x10100000L) ||
defined(OPENSSL_USE_DEPRECATED)
native/src/ssl.c:#if (OPENSSL_VERSION_NUMBER < 0x10100000L) ||
defined(OPENSSL_USE_DEPRECATED)
native/src/sslinfo.c:#if OPENSSL_VERSION_NUMBER < 0x10100000L
native/src/sslinfo.c:#if OPENSSL_VERSION_NUMBER < 0x10100000L
native/src/sslinfo.c:#if OPENSSL_VERSION_NUMBER < 0x10100000L
native/src/sslinfo.c:#if OPENSSL_VERSION_NUMBER < 0x10100000L
native/src/sslcontext.c:#if OPENSSL_VERSION_NUMBER < 0x10100000L

According to the openssl_version_number man page
(http://linux.die.net/man/3/openssl_version_number), OPENSSL_VERSION_NUMBER is
a numeric release version identifier in this format: MMNNFFPPS: major minor fix
patch status

Thus 0x10100000L refers to OpenSSL version 1.1.0. According to the comments of
SVN revision 1681092 (Git SHA-1 fda408291a5557cacad7259248f6632ad5ae264a), the
intended OpenSSL version was 1.0.1:

"this commit also removes all code with #if or #ifdef checks which only apply
to OpenSSL 1.0.1 or earlier"

The value that should be used in all the comparisons above is 0x10001000L.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to