From 7f2fb5e0a3978e8e067ac86ef334f36350859865 Mon Sep 17 00:00:00 2001
From: Hauke Petersen <hauke.petersen@fu-berlin.de>
Date: Wed, 9 May 2018 16:22:54 +0200
Subject: [PATCH] net/emcute: use correct number of retries
syncsend() is used also for the initial packet, so resending
while (retries <= EMCUTE_N_RETRY) is the correct behavior.
---
sys/net/application_layer/emcute/emcute.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/sys/net/application_layer/emcute/emcute.c b/sys/net/application_layer/emcute/emcute.c
index 14ec51d14f..0808dad3bd 100644
--- a/sys/net/application_layer/emcute/emcute.c
+++ b/sys/net/application_layer/emcute/emcute.c
@@ -118,7 +118,7 @@ static int syncsend(uint8_t resp, size_t len, bool unlock)
* remove was called */
thread_flags_clear(TFLAGS_ANY);
- for (unsigned retries = 0; retries < EMCUTE_N_RETRY; retries++) {
+ for (unsigned retries = 0; retries <= EMCUTE_N_RETRY; retries++) {
DEBUG("[emcute] syncsend: sending round %i\n", retries);
sock_udp_send(&sock, tbuf, len, &gateway);
@@ -128,7 +128,7 @@ static int syncsend(uint8_t resp, size_t len, bool unlock)
DEBUG("[emcute] syncsend: got response [%i]\n", result);
xtimer_remove(&timer);
res = result;
- retries = EMCUTE_N_RETRY;
+ break;
}
}
--
GitLab