[PR] Backport LwM2M changes and fixes #407
Closed
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
Other operations, like write-attributes, still need to be implemented
... also show it in REST /endpoints
LwM2M standard states that the server MUST reinitiate observations, but as a quick fix, just remove them.
LwM2M should call request transaction callback only when a response arrives and not an empty ACK.
…clipse-wakaama#38) Add missing (overwritten) string terminator character.
…ama#46) Otherwise it leaks memory
Transactions have references to context that may be already dealloced when the transaction completes (i.e. client is deregistered, but observation transaction is pending and will time out).
Transactions to queue mode devices must wait for client update.
* lwm2m_observe_cancel instead of sending unobserve packet, now changes observation state * lwm2m_observe_cancel calls callback immediately * Number of pending transactions is tracked to prevent premature free
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.
This pull request fixes issue #2
Due to merge conflict, commit 22a2434 has been reverted because it seems to address the same issue fixed by 8devices@31488c9
Commits have been hand-picked to only leave changes made to core files