Skip to content

Fix for MQTT subscription validation loop#374

Open
ra1u wants to merge 3 commits into
FreeRTOS:mainfrom
ra1u:main
Open

Fix for MQTT subscription validation loop#374
ra1u wants to merge 3 commits into
FreeRTOS:mainfrom
ra1u:main

Conversation

@ra1u

@ra1u ra1u commented May 11, 2026

Copy link
Copy Markdown

Issue is that validation can accept invalid elements, because it does not interrupt loop on failure.
Proposed patch improves this.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@ra1u ra1u changed the title Bugfix for MQTT subscription validation loop Fix for MQTT subscription validation loop May 12, 2026

@AniruddhaKanhere AniruddhaKanhere left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @ra1u for taking the time to report this issue.

I have one minor suggestion. Let us know if you'd like us to make the change on your behalf (we'd need permission to collaborate on your fork).

Thanks

Comment thread source/core_mqtt.c Outdated
ra1u and others added 2 commits July 7, 2026 09:24
Co-authored-by: Aniruddha Kanhere <kanherea@amazon.com>
Fix merge issue
@ra1u

ra1u commented Jul 15, 2026

Copy link
Copy Markdown
Author

Thank you Aniruddha Kanhere

I noticed I made merge bug when accepting suggested change over web.
Regarding permission, fell free to change. Let me know if I need to click on some permission button.

Kind regards!

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

Successfully merging this pull request may close these issues.

2 participants