[Libwebsockets] A couple of patches and a question

Peter Pentchev roam at ringlet.net
Sun Oct 2 01:19:47 CEST 2016


Thanks for keeping libwebsockets going!

What do you think about the attached couple of patches?  The first one
fixes a couple of typographical and grammatical errors, and the second
one tries to catch up with OpenSSL 1.1's changes in functionality.
Yes, the same part of the code had some ifdefs related to OpenSSL 1.1
added before, but very soon after that, in the next prerelease,
things changed again and the various cleanup functions were turned
into no-ops and marked as deprecated, since everything seems to be done
automatically now.  Hopefully now that 1.1 has been published and
has seen a couple of bugfix patchlevel releases, the interface should
remain stable.

I also have a question that was raised by running cppcheck over
the libwebsockets source: what is the lws_ssl_capable_write() function
supposed to do in the LWS_USE_MBEDTLS case?  The ifdefs around
the actual SSL library functions do not seem to do anything in
that case, which leaves the value of the "n" variable undefined,
not to mention no real work has been done, nothing has been written
anywhere.  So, I guess, either assign something to n to explicitly
signal your intention or, I don't know, make the function actually
do something? :)

Thanks again for your work on libwebsockets!


Peter Pentchev  roam at ringlet.net roam at FreeBSD.org pp at storpool.com
PGP key:        http://people.FreeBSD.org/~roam/roam.key.asc
Key fingerprint 2EE7 A7A5 17FC 124C F115  C354 651E EFB0 2527 DF13
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://libwebsockets.org/pipermail/libwebsockets/attachments/20161002/7cfd63da/attachment.asc>

More information about the Libwebsockets mailing list