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

Trickle: remove vtimers before (re)scheduling it #373

Merged
merged 1 commit into from Nov 23, 2013

Conversation

OlegHahm
Copy link
Member

Otherwise it might cause an endless loop in queue.c due to multiple nodes of the same instance (e.g. dao_timer).

(Would look like:

$5 = (struct queue_node_t *) 0x809b1e0 <dao_timer>
(gdb) print node->next->next->next->next
$6 = (struct queue_node_t *) 0x8083dfc <con_buf+7644>
(gdb) print node->next->next->next->next->next
$7 = (struct queue_node_t *) 0x809b1e0 <dao_timer>
(gdb) print node->next->next->next->next->next->next
$8 = (struct queue_node_t *) 0x8083dfc <con_buf+7644>
(gdb) print node->next->next->next->next->next->next->next
$9 = (struct queue_node_t *) 0x809b1e0 <dao_timer>
(gdb) print node->next->next->next->next->next->next->next->next
$10 = (struct queue_node_t *) 0x8083dfc <con_buf+7644>
(gdb) print node->next->next->next->next->next->next->next->next->next
$11 = (struct queue_node_t *) 0x809b1e0 <dao_timer>

)

@LudwigKnuepfer
Copy link
Member

Apparently helps, ACK.

OlegHahm added a commit that referenced this pull request Nov 23, 2013
Trickle: remove vtimers before (re)scheduling it
@OlegHahm OlegHahm merged commit e09571c into RIOT-OS:master Nov 23, 2013
@OlegHahm OlegHahm deleted the trickle_dao_fix branch November 23, 2013 02:46
thomaseichinger pushed a commit to thomaseichinger/RIOT that referenced this pull request Dec 9, 2013
Trickle: remove vtimers before (re)scheduling it
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