[Bug 61184] New: [PATCH] Fix build with LibreSSL in 2.4.26-dev

classic Classic list List threaded Threaded
9 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 61184] New: [PATCH] Fix build with LibreSSL in 2.4.26-dev

Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61184

            Bug ID: 61184
           Summary: [PATCH] Fix build with LibreSSL in 2.4.26-dev
           Product: Apache httpd-2
           Version: 2.4-HEAD
          Hardware: PC
                OS: FreeBSD
            Status: NEW
          Severity: normal
          Priority: P2
         Component: mod_ssl
          Assignee: [hidden email]
          Reporter: [hidden email]
  Target Milestone: ---

Created attachment 35052
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=35052&action=edit
unified diff for httpd 2.4.26-dev

Hi,

Just tried building httpd 2.4.26-dev with LibreSSL and ran into some compile
failures. These failures are related to the added OpenSSL 1.1 support in
2.4.26.

LibreSSL defines OPENSSL_VERSION_NUMBER as 0x20000000L whereas it does not
implement all post-1.0.1f (point of forking) features. LibreSSL added
LIBRESSL_VERSION_NUMBER allowing checks.

Attached patches touch mod_ssl and ab. Adding checks for
defined(LIBRESSL_VERSION_NUMBER).

Hope you can still include these in the release.

Thanks,

Bernard Spil
Maintainer of OpenSSL and LibreSSL ports in FreeBSD.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 61184] [PATCH] Fix build with LibreSSL in 2.4.26-dev

Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61184

William A. Rowe Jr. <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |PatchAvailable

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 61184] [PATCH] Fix build with LibreSSL in 2.4.26-dev

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61184

--- Comment #1 from Bernard Spil <[hidden email]> ---
Created attachment 35053
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=35053&action=edit
Build log FreeBSD 11.0-p9

Poudriere logs on FreeBSD 11.0-p9 with LibreSSL replacing OpenSSL in base.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 61184] [PATCH] Fix build with LibreSSL in 2.4.26-dev

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61184

Bernard Spil <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #35052|0                           |1
        is obsolete|                            |

--- Comment #2 from Bernard Spil <[hidden email]> ---
Created attachment 35054
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=35054&action=edit
unified diff for httpd 2.4.26-dev

Replace patches, were incomplete. Built OK but still warnings.
This patch-set stopped all compile warnings.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 61184] [PATCH] Fix build with LibreSSL in 2.4.26-dev

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61184

Bernard Spil <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #35053|0                           |1
        is obsolete|                            |

--- Comment #3 from Bernard Spil <[hidden email]> ---
Created attachment 35055
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=35055&action=edit
Build log FreeBSD 11.0-p9

Built with new patch-set

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 61184] [PATCH] Fix build with LibreSSL in 2.4.26-dev

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61184

--- Comment #4 from Stefan Eissing <[hidden email]> ---
Hmm. This looks ugly. Would it make more sense to re#define Libressl's sense of
superiority? Something like

#if defined(LIBRESSL_VERSION_NUMBER) && LIBRESSL_VERSION_NUMBER >= 0x20000000L
#undef OPENSSL_VERSION_NUMBER
#define OPENSSL_VERSION_NUMBER 0x1000200eL
#endif

Or whatever version it currently is closest to? You know better than me.

Regarding the release: how would the impact be, if you need to patch that
yourself for debian? I am not sure if we want to restart the already late
release only for this. If something else comes up, we can take it in of course.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 61184] [PATCH] Fix build with LibreSSL in 2.4.26-dev

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61184

--- Comment #5 from Stefan Eissing <[hidden email]> ---
What I mean was

#if defined(LIBRESSL_VERSION_NUMBER) && OPENSSL_VERSION_NUMBER >= 0x20000000L
...

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 61184] [PATCH] Fix build with LibreSSL in 2.4.26-dev

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61184

Bernard Spil <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #35054|0                           |1
        is obsolete|                            |

--- Comment #6 from Bernard Spil <[hidden email]> ---
Created attachment 35062
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=35062&action=edit
unified diff for Apache 2.4.26

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 61184] [PATCH] Fix build with LibreSSL in 2.4.26-dev

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61184

Bernard Spil <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #35055|0                           |1
        is obsolete|                            |

--- Comment #7 from Bernard Spil <[hidden email]> ---
Created attachment 35063
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=35063&action=edit
Build log FreeBSD 11.0-p9

I went through the code more rigorously checking diff between 2.4.25 and 2.4.26
for changes that I needed to tend to. Further to that I verified method
availability in LibreSSL 2.5.4.

Please do review this thoroughly!

Thanks,

Bernard.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Loading...