aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHolger Hans Peter Freyther <holger@moiji-mobile.com>2014-04-19 19:22:22 +0200
committerHolger Hans Peter Freyther <holger@moiji-mobile.com>2014-04-19 19:22:22 +0200
commit3e5b6db2d26f279f9238089f8c79de1d71a997aa (patch)
tree46f253ce31ce42007abcb07ff4636bc69c3c0a89
parentd59bbd16a753554747adcdec18b5cea28a839604 (diff)
handover: Call the right function and avoid recursion
Fix a brown paper bag bug and call the right method. The above was an infinite recursion. The stack didn't overflow as the compiler optimized the tail-recursion and coverity didn't complain either. The issue was introduced in the last minutes before the merge when I renamed "reset_handover" to "handover_reset" to follow the object_verb approach throughout the handover.c code. While doing that I sadly replaced reset_handover with handover_frame and not handover_reset.
-rw-r--r--src/common/handover.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/common/handover.c b/src/common/handover.c
index ab3cbcf..26a3f51 100644
--- a/src/common/handover.c
+++ b/src/common/handover.c
@@ -143,7 +143,7 @@ void handover_frame(struct gsm_lchan *lchan)
{
LOGP(DHO, LOGL_INFO,
"%s First valid frame detected\n", gsm_lchan_name(lchan));
- handover_frame(lchan);
+ handover_reset(lchan);
}
/* release handover state */