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

Disconnect won't release TLS memory when network down #96

Closed
andrew0x0 opened this issue Sep 15, 2017 · 2 comments
Closed

Disconnect won't release TLS memory when network down #96

andrew0x0 opened this issue Sep 15, 2017 · 2 comments
Labels

Comments

@andrew0x0
Copy link

andrew0x0 commented Sep 15, 2017

_aws_iot_mqtt_internal_disconnect
/* send the disconnect packet */
if(serialized_len > 0) {
rc = aws_iot_mqtt_internal_send_packet(pClient, serialized_len, &timer);
if(SUCCESS != rc) {
FUNC_EXIT_RC(rc);
}
}
Should remove if (), just no matter disconnect packet send out or not just return 0 let caller release TLS memory

@chaurah
Copy link
Contributor

chaurah commented Sep 15, 2017

Hi @andrew0x0,
Thank you for pointing this out. I have added a task to our internal tracker for the next release of the SDK. Please let us know if you find any further issues.

Rahul

@andrew0x0
Copy link
Author

andrew0x0 commented Sep 17, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants