[Bug Report] MQTT Core Deinit

Tip / Sign in to post questions, reply, level up, and achieve exciting badges. Know more

cross mob
Anonymous
Not applicable

This is in WICED 5.0.

File: libraries/protocols/MQTT/mqtt_network.c

Function: wiced_result_t mqtt_core_deinit( mqtt_connection_t *conn )

The bug is that it de-inits the queue before waking up the thread, causing a queue error since the thread isn't aware that the queue is deleted already.

I moved wiced_rtos_deinit_queue( &mqtt_socket->queue ); to after deleting the thread. Now it works fine.

Tagging mifo

5 Replies
Anonymous
Not applicable

Please provide the error log for us to analyze.

0 Likes

You can get the error log by yourself by just calling "wiced_mqtt_deinit()".

AFAICT, josh.yu_1995211​'s report is very clear.

0 Likes
Anonymous
Not applicable

Seriously, it's ridiculous that you'd ask for an error log when I've bothered writing up a detailed bug report and even providing a fix here in the forum. Please read my original post again.

Anonymous
Not applicable

Hello Josh,

I have analyzed your post and raised the issue internally. I want to know your implementation for me to suggest any other way other than what you mentioned. Apologies for this same.

Any how, it will be fixed in the next release of the SDK.

0 Likes

rash wrote:

Hello Josh,

I have analyzed your post and raised the issue internally. I want to know your implementation for me to suggest any other way other than what you mentioned. Apologies for this same.

Any how, it will be fixed in the next release of the SDK.

No, this is not true.

sdk-6.0.0 does not include this fix.

0 Likes