Bugfixes for issues we've encountered #5
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ues. Co-authored-by: Dmitriy Dubrovenskiy <[email protected]>
… allow multiple uses of the server (on Windows).
…allowed to be written to a socket an one time.
TR341 - adding the ability to resolve multiple dns entries in case of…
…for that in yield code.
Corrected the conversion of RSI error codes to IoT_Error_t and check …
AB#454 - removed duplicate function type definition
AB#644 - Ignore firmware generated code.
…E_RESPONSE_WAIT_TIME if not already defined.
AB#772 - Ensure sockID is valid before indexing the rsi_socket_pool_n…
…it read a single byte even if it times out. This was causing the AWS IoT SDK to get lost when trying to read MQTT messages.
AB#934 - Do not use pkt until it has been checked for NULL.
…added a timer to limit the number of seconds allowed for connection, and added more checks for error situations.
…READY when requesting the fw version. Without this delay, it spins and starves the watchdog.
…erval is in milliseconds
AB#770 - Check the bt_cb pointer for NULL before using it.
Closed
…ad when checking the read buffer. If we fail to read, we need to propagate that failure up the stack.
Outlaw1111
approved these changes
Mar 19, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Many changes are checks for valid socket ids and pointers.
We are no longer using the generic MQTT client; but, the changes were good when we were using it.