Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix registration update issues. #172

Merged
merged 1 commit into from
Jun 21, 2016
Merged

Fix registration update issues. #172

merged 1 commit into from
Jun 21, 2016

Conversation

dnav
Copy link
Contributor

@dnav dnav commented Jun 13, 2016

A failed registration update has now the same effect as a failed registration.
If there is no valid registration to a LWM2M Server, the client returns to the BOOTSTRAP_REQUIRED state.

Also include fixes for #169 and #170.

Signed-off-by: David Navarro david.navarro@intel.com

Signed-off-by: David Navarro <david.navarro@intel.com>
@@ -192,11 +192,11 @@ static int prv_refreshServerList(lwm2m_context_t * contextP)
while (targetP != NULL)
{
nextP = targetP->next;
targetP->next = NULL;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you lose here the entire list

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The list is saved in nextP the line before, isn't it ?

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you right ...

@dnav dnav merged commit 45b86ae into master Jun 21, 2016
@dnav dnav deleted the registration_rework branch June 21, 2016 12:37
vslapik pushed a commit to vslapik/wakaama that referenced this pull request Apr 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants