Skip to content

server/mqttd: fix two memory leaks - #15290

Closed
bagder wants to merge 1 commit into
masterfrom
bagder/mqttd-leak
Closed

server/mqttd: fix two memory leaks#15290
bagder wants to merge 1 commit into
masterfrom
bagder/mqttd-leak

Conversation

@bagder

@bagder bagder commented Oct 14, 2024

Copy link
Copy Markdown
Member

No description provided.

@bagder bagder added the tests label Oct 14, 2024
@github-actions github-actions Bot added the MQTT label Oct 14, 2024
@dfandrich

Copy link
Copy Markdown
Contributor

Analysis of PR #15290 at 0db4f680:

Test http/test_17_ssl_use.py::TestSSLUse::test_17_09_ssl_min_max[TLSv1-0--2] failed, which has NOT been flaky recently, so there could be a real issue in this PR.

Test http/test_17_ssl_use.py::TestSSLUse::test_17_09_ssl_min_max[TLSv1-0--1] failed, which has NOT been flaky recently, so there could be a real issue in this PR.

Test http/test_17_ssl_use.py::TestSSLUse::test_17_09_ssl_min_max[TLSv1-0-0] failed, which has NOT been flaky recently, so there could be a real issue in this PR.

Test http/test_17_ssl_use.py::TestSSLUse::test_17_09_ssl_min_max[TLSv1-1--2] failed, which has NOT been flaky recently, so there could be a real issue in this PR.

Test http/test_17_ssl_use.py::TestSSLUse::test_17_09_ssl_min_max[TLSv1-1--1] failed, which has NOT been flaky recently, so there could be a real issue in this PR.

Test http/test_17_ssl_use.py::TestSSLUse::test_17_09_ssl_min_max[TLSv1-1-0] failed, which has NOT been flaky recently, so there could be a real issue in this PR.

Test http/test_17_ssl_use.py::TestSSLUse::test_17_09_ssl_min_max[TLSv1-2--2] failed, which has NOT been flaky recently, so there could be a real issue in this PR.

There are more failures, but that's enough from Gha.

Generated by Testclutch

@bagder bagder closed this in e33cf00 Oct 14, 2024
@bagder
bagder deleted the bagder/mqttd-leak branch October 14, 2024 10:48
pps83 pushed a commit to pps83/curl that referenced this pull request Apr 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants