-
Notifications
You must be signed in to change notification settings - Fork 214
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
Memory leak in SNTP sync #497
Comments
Edit: as the leakage remains when using |
Seems like in the function picotcp/modules/pico_sntp_client.c Line 180 in 4caf650
@frederikvs Could you verify this? Is it a solution to just add pico_socket_close(s); and pico_socket_del(s); to PICO_SOCK_EV_RD ? Or do we need to close the socket in the other events also?
|
@danielinux Maybe you could take a look at this? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
picotcp/modules/pico_sntp_client.c
Line 469 in 4caf650
When performing a SNTP sync there is a memory leak of about 1k.
I'm at the latest version of the stack.
The text was updated successfully, but these errors were encountered: