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.

Sign up for GitHub

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

Jump to bottom

Fix thoretical possible memory leaks related to topic in error case #1521

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Fix thoretical possible memory leaks related to topic in error case #1521

wants to merge 2 commits into from

Commits on Sep 3, 2024

  1. Release memory of pack->topic together with pack witin error cases of… 

    … MQTTPacket_publish()

As seen in
eclipse-paho#1518
the memory allocated for pack-topic were not released in all cases.

Signed-off-by: Juergen Kosel <juergen.kosel@softing.com>
    @JuergenKosel
    JuergenKosel committed Sep 3, 2024
    Configuration menu
    Copy the full SHA
    27fb21e View commit details
    Browse the repository at this point in the history

Commits on Sep 4, 2024

  1. MQTTAsync_receiveThread() logs an error in case of unexpected and the… 

    …refore unprocessed pack type

Unprocessed pack lead to memory leak, as in
eclipse-paho#1518

Signed-off-by: Juergen Kosel <juergen.kosel@softing.com>
    @JuergenKosel
    JuergenKosel committed Sep 4, 2024
    Configuration menu
    Copy the full SHA
    a2fd510 View commit details
    Browse the repository at this point in the history