[BUG] Wrong conditions in mqtt_pkcs11_demo_helpers.c

[KeitaKashima]

Describe the bug

In mqtt_pkcs11_demo_helpers.c, there would be typo conditions L797 and L805.

I think it should check NOT pd FAIL, likeif( xReturnStatus != pdFAIL ) but it performs if( xReturnStatus == pdFAIL ).

FreeRTOS/FreeRTOS-Plus/Demo/AWS/Mqtt_Demo_Helpers/mqtt_pkcs11_demo_helpers.c

Line 797 in eb97c5d

if( xReturnStatus == pdFAIL )

It seems that the demo is working fine.

Because this xMqttSessionEstablished flag is only used when do the MQTT Disconnect.

And when the flag is true, the device sent MQTT DISCONNECT packet.

But after that, the connection is disconnected immidiately. So, it would not affect the demo.

Target

Below is my target, but it is happened other devices.

• Development board: CK-RX65N (Renesas RX family)
• Instruction Set Architecture: RX65N
• IDE and version: e2studio 2024-01
• Toolchain and version: CC-RX V3.

Host

• Host OS: Win10

To Reproduce

• Use project : URL RX65N AWS FreeRTOS v202210 IoT repo
• Run on : CK-RX65N

Expected behavior

Change to if( xReturnStatus != pdFAIL ) .

[xuelix]

Thanks for reporting. We'll fix it.