Improvements to multithread locking and tests.

dgarske commented 8 months ago

Refactor of locking to better detect and handle edge cases. Allow partial writes on all packet types.
Fix for issue processing wait for a publish with ack on a different packet type causing the write state to get reset.
Fix issue with publish write only incorrectly removing pending response. ZD 16769.
Fix for cancelling publish to also attempt to cancel any publish responses expected (Qos 1 and 2).
Fix for read of fixed portion (2 bytes) of header when only 1 is returned.
Remove extra test non-block in MqttClient_Publish_ReadPayload (already handled in MqttSocket_Read).
Improvements to the testing
Cleanup return code 0=success (use MQTT_CODE_SUCCESS)

dgarske commented 7 months ago

@embhorn let's gets this merged in. I'll followup later with any issues found.

dgarske commented 7 months ago

@embhorn I've sorted out the test issues. This PR is ready to review / merge.

wolfSSL / wolfMQTT