Modify

Opened 7 years ago

Closed 7 years ago

#7827 closed enhancement (fixed)

Allow uhttpd to be built against OpenSSL as well as CyaSSL

Reported by: dharding@… Owned by: jow
Priority: normal Milestone:
Component: base system Version: Backfire 10.03.1 RC2
Keywords: Cc:

Description

I have packages on my target that require OpenSSL. Thus to save space, I would like to build be able to build uhttpd against OpenSSL so as to not have to install CyaSSL also. I have patched OpenWRT to support building against either library (via a make menuconfig selection) on my development system. I thought this might be useful to other users so I am creating this ticket to track my patches for this feature.

Attachments (6)

0001-uhttpd-add-error-handling-to-uh_tls_client_accept.patch (3.5 KB) - added by dharding@… 7 years ago.
improve uh_tls_client_accept by adding error handling and adding a call to SSL_accept (required when building against OpenSSL)
0002-uhttpd-support-building-with-OpenSSL-instead-of-CyaS.patch (3.2 KB) - added by dharding@… 7 years ago.
update uhttpd makefiles to allow building against either CyaSSL (default) or OpenSSL
0003-uhttpd-minor-cleanups.patch (2.6 KB) - added by dharding@… 7 years ago.
[optional] minor cleanups to the code and makefiles made while implementing this feature
0001-uhttpd-add-error-handling-to-uh_tls_client_accept.2.patch (3.5 KB) - added by dharding@… 7 years ago.
improve uh_tls_client_accept by adding error handling and adding a call to SSL_accept (required when building against OpenSSL)
0002-uhttpd-support-building-with-OpenSSL-instead-of-CyaS.2.patch (3.2 KB) - added by dharding@… 7 years ago.
update uhttpd makefiles to allow building against either CyaSSL (default) or OpenSSL
0003-uhttpd-minor-cleanups.2.patch (2.6 KB) - added by dharding@… 7 years ago.
[optional] minor cleanups to the code and makefiles made while implementing this feature

Download all attachments as: .zip

Change History (12)

Changed 7 years ago by dharding@…

improve uh_tls_client_accept by adding error handling and adding a call to SSL_accept (required when building against OpenSSL)

Changed 7 years ago by dharding@…

update uhttpd makefiles to allow building against either CyaSSL (default) or OpenSSL

Changed 7 years ago by dharding@…

[optional] minor cleanups to the code and makefiles made while implementing this feature

comment:1 Changed 7 years ago by dharding@…

The attached patches are against the latest Backfire. If they don't apply cleanly against the trunk, I can try to regenerate them.

comment:2 in reply to: ↑ description Changed 7 years ago by anonymous

Replying to dharding@…:

I have packages on my target that require OpenSSL. Thus to save space, I would like to build be able to build uhttpd against OpenSSL so as to not have to install CyaSSL also. I have patched OpenWRT to support building against either library (via a make menuconfig selection) on my development system. I thought this might be useful to other users so I am creating this ticket to track my patches for this feature.

An alternative would be to build your other packages against CyaSSL. Let us know if you need help with that. We'd be happy to support your effort. Just contact us at info@… if you want to try to go that route.

comment:3 Changed 7 years ago by dharding@…

Thanks for the offer. The package in question is OpenVPN - some Googling turned up a couple of reports of people trying to build it against CyaSSL, but but no indication of success. I went the OpenSSL route and currently have a single SSL library on my system currently, which I am happy with. If OpenWRT wants to pick up my patches, great - if not, I'll keep being happy :-).

comment:4 Changed 7 years ago by florian

  • Owner changed from developers to jow
  • Status changed from new to assigned

comment:5 Changed 7 years ago by dharding@…

I have been updating my changes to keep up with modifications to uhttpd. I will post fresh patches in a few hours.

Changed 7 years ago by dharding@…

improve uh_tls_client_accept by adding error handling and adding a call to SSL_accept (required when building against OpenSSL)

Changed 7 years ago by dharding@…

update uhttpd makefiles to allow building against either CyaSSL (default) or OpenSSL

Changed 7 years ago by dharding@…

[optional] minor cleanups to the code and makefiles made while implementing this feature

comment:6 Changed 7 years ago by jow

  • Resolution set to fixed
  • Status changed from assigned to closed

Committed in r27686 - thanks!

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.