diff mbox series

[1/1] package/pppd: fix build with libressl

Message ID 20220725145836.11664-1-fontaine.fabrice@gmail.com
State Accepted
Headers show
Series [1/1] package/pppd: fix build with libressl | expand

Commit Message

Fabrice Fontaine July 25, 2022, 2:58 p.m. UTC
Fix the following build failure with libressl:

eap-tls.c: In function 'ssl_msg_callback':
eap-tls.c:1284:10: error: 'SSL3_RT_HEADER' undeclared (first use in this function); did you mean 'SSL3_RT_ALERT'?
 1284 |     case SSL3_RT_HEADER:
      |          ^~~~~~~~~~~~~~
      |          SSL3_RT_ALERT

Fixes:
 - http://autobuild.buildroot.org/results/7d721833bddf73531fa03b0a626511af6826d0df

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 ...pd-eap-tls.c-fix-build-with-libressl.patch | 36 +++++++++++++++++++
 1 file changed, 36 insertions(+)
 create mode 100644 package/pppd/0002-pppd-eap-tls.c-fix-build-with-libressl.patch

Comments

Thomas Petazzoni July 26, 2022, 4:43 p.m. UTC | #1
On Mon, 25 Jul 2022 16:58:36 +0200
Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote:

> Fix the following build failure with libressl:
> 
> eap-tls.c: In function 'ssl_msg_callback':
> eap-tls.c:1284:10: error: 'SSL3_RT_HEADER' undeclared (first use in this function); did you mean 'SSL3_RT_ALERT'?
>  1284 |     case SSL3_RT_HEADER:
>       |          ^~~~~~~~~~~~~~
>       |          SSL3_RT_ALERT
> 
> Fixes:
>  - http://autobuild.buildroot.org/results/7d721833bddf73531fa03b0a626511af6826d0df
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> ---
>  ...pd-eap-tls.c-fix-build-with-libressl.patch | 36 +++++++++++++++++++
>  1 file changed, 36 insertions(+)
>  create mode 100644 package/pppd/0002-pppd-eap-tls.c-fix-build-with-libressl.patch

Applied to master, thanks.

Thomas
Peter Korsgaard Aug. 18, 2022, 6:26 a.m. UTC | #2
>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:

 > Fix the following build failure with libressl:
 > eap-tls.c: In function 'ssl_msg_callback':
 > eap-tls.c:1284:10: error: 'SSL3_RT_HEADER' undeclared (first use in this function); did you mean 'SSL3_RT_ALERT'?
 >  1284 |     case SSL3_RT_HEADER:
 >       |          ^~~~~~~~~~~~~~
 >       |          SSL3_RT_ALERT

 > Fixes:
 >  - http://autobuild.buildroot.org/results/7d721833bddf73531fa03b0a626511af6826d0df

 > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Committed to 2022.05.x and 2022.02.x, thanks.
diff mbox series

Patch

diff --git a/package/pppd/0002-pppd-eap-tls.c-fix-build-with-libressl.patch b/package/pppd/0002-pppd-eap-tls.c-fix-build-with-libressl.patch
new file mode 100644
index 0000000000..aefdc5b7a4
--- /dev/null
+++ b/package/pppd/0002-pppd-eap-tls.c-fix-build-with-libressl.patch
@@ -0,0 +1,36 @@ 
+From b0a011bc4abac8cb3de8dfff42b754ed236ecf0f Mon Sep 17 00:00:00 2001
+From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+Date: Mon, 4 Jul 2022 10:07:03 +0200
+Subject: [PATCH] pppd/eap-tls.c: fix build with libressl (#338)
+
+Fix the following build failure with libressl:
+
+eap-tls.c: In function 'ssl_msg_callback':
+eap-tls.c:1284:10: error: 'SSL3_RT_HEADER' undeclared (first use in this function); did you mean 'SSL3_RT_ALERT'?
+ 1284 |     case SSL3_RT_HEADER:
+      |          ^~~~~~~~~~~~~~
+      |          SSL3_RT_ALERT
+
+Fixes:
+ - http://autobuild.buildroot.org/results/7d721833bddf73531fa03b0a626511af6826d0df
+
+Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+[Retrieved (and backported) from:
+https://github.com/ppp-project/ppp/commit/b0a011bc4abac8cb3de8dfff42b754ed236ecf0f]
+---
+ pppd/eap-tls.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/pppd/eap-tls.c b/pppd/eap-tls.c
+index b9bab842..40796d58 100644
+--- a/pppd/eap-tls.c
++++ b/pppd/eap-tls.c
+@@ -61,7 +61,7 @@
+ #include "mppe.h"
+ #include "pathnames.h"
+ 
+-#if OPENSSL_VERSION_NUMBER < 0x10100000L
++#if OPENSSL_VERSION_NUMBER < 0x10100000L || defined(LIBRESSL_VERSION_NUMBER)
+ 
+ #define TLS_method SSLv23_method
+