Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Try WolfSSL instead of mbedTLS #67

Closed
revk opened this issue Jul 15, 2021 · 8 comments
Closed

Try WolfSSL instead of mbedTLS #67

revk opened this issue Jul 15, 2021 · 8 comments
Assignees
Milestone

Comments

@revk
Copy link
Owner

revk commented Jul 15, 2021

mbedTLS has huge memory leak for various types of non clean close.

Causes all sorts of issues.

@revk revk added the stuck Waiting on something else label Jul 15, 2021
@revk revk self-assigned this Jul 15, 2021
@ghenry
Copy link

ghenry commented Jul 15, 2021

What's the reason for using mbedTLS vs say WolfSSL or OpenSSL? Size?

@revk
Copy link
Owner Author

revk commented Jul 15, 2021

Just what came with ESP IDF to be honest.

Is WolfSSL better?

@ghenry
Copy link

ghenry commented Jul 15, 2021

Ah, OK. I'm really enjoying WolfSSL. Great examples - https://github.com/wolfSSL/wolfssl/tree/master/examples and lovely docs - https://github.com/wolfSSL/wolfssl-examples/blob/master/tls/README.md

@revk revk changed the title mbedTLS memory leak Try WolfSSL instead of mbedTLS Jul 15, 2021
@revk revk added nextjob What I plan to look at text stuck Waiting on something else and removed stuck Waiting on something else nextjob What I plan to look at text labels Jul 15, 2021
@revk
Copy link
Owner Author

revk commented Jul 16, 2021

After much looking, seems all I need is :-

git submodule add --recursive https://github.com/espressif/esp-wolfssl.git components/esp-wolfssl

This provides the options in make menuconfig, yay!

Sadly, that does not actually work. It seems not to actually build the library!

make: *** No rule to make target '/projects/github/SolarSystem/ESP32/components/esp-wolfssl/wolfssl/lib/esp32/libwolfssl.a', needed by '/projects/github/SolarSystem/ESP32/build/SS.elf'. Stop.

And help welcome.

@ghenry
Copy link

ghenry commented Jul 16, 2021 via email

@ghenry
Copy link

ghenry commented Jul 16, 2021

I'm just following the docs myself to see if I can get it build. The ESPs are nice. Soooo many options in menuconfig. Just like buidling the Linux kernel.

@revk
Copy link
Owner Author

revk commented Jul 16, 2021

Apparently handling (legacy) make command was not in there and will be soon.

@revk revk added this to the Beta test milestone Jul 17, 2021
@revk revk removed help wanted stuck Waiting on something else labels Jul 19, 2021
@revk
Copy link
Owner Author

revk commented Jul 19, 2021

Needs an extra component.mk to fix for Make for now.

But works.

@revk revk closed this as completed Jul 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants