[OpenWrt-Devel] [PATCH ustream-ssl v2 0/3] wolfssl updates

Eneas U de Queiroz cotequeiroz at gmail.com
Wed Sep 18 22:18:00 EDT 2019


ustream-ssl does not currently work with client apps.  They fail to
connect immediately.  uclient-fetch, for example, just show a
'Connection failed' message.  This was due to a change in the SSL
session struct.

While making small changes to my first attempt, I realized my testing
was not completely flawed.

Because of that, I failed to realize that while the struct change
happened in in v3.13.2, but the API to change it at the SSL struct was
only created in v4.1.0.

Keeping some of the calls using CyaSSL, but then having to add new
calls, only available as wolfSSL would become messy.  So, I started
by cleaning up the code, removing old CyaSSL remnants.

After fixing that, uclient-fetch failed to run unless
--no-check-certificate is used, which is not ideal.  So I added the
calls to perform CN validation. r

Note that even wolfssl has a X509_check_host function, which could
be used for openssl and wolfssl, they are  not 100% compatible, and its
definition is not really consistent from version to version.

X509_CHECK_FLAG_NO_PARTIAL_WILDCARDS is not defined, and even though
wolfSSL_X509_check_host apparently only exists to implement the openssl
function, they are not both enabled by --enable-opensslextra.
The wolfSSL function is, but the openssl isn't.  So I'm using the
wolfssl call.

As for testing, I run-tested each commit with wolfssl versions 3.10.4,
3.12.2, 3.15.3, and 4.1.0.  Since the fist commit does not fix
client-mode, I tested it using the example client/server apps (using a
client built with a different version).  Version 3.15.3 did not work
with the server app, so I checked the current HEAD, and it does not work
either, so it is not somehting I introduced.  It works after the next
fix is applied.

Everything works as expected from the next commit on.

Eneas

--

Eneas U de Queiroz (3):
  Remove CyaSSL, WolfSSL < 3.10.4 support
  ustream-io-cyassl.c: fix client-mode connections
  wolfssl: enable CN validation

 CMakeLists.txt                              | 25 +++++++----
 ustream-internal.h                          |  3 --
 ustream-io-cyassl.c => ustream-io-wolfssl.c | 47 +++++----------------
 ustream-openssl.c                           | 14 +++---
 ustream-openssl.h                           |  4 ++
 ustream-ssl.c                               |  3 ++
 6 files changed, 43 insertions(+), 53 deletions(-)
 rename ustream-io-cyassl.c => ustream-io-wolfssl.c (62%)


_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel



More information about the openwrt-devel mailing list