diff options
| author | Holger Hans Peter Freyther <holger@moiji-mobile.com> | 2014-04-19 19:22:22 +0200 |
|---|---|---|
| committer | Holger Hans Peter Freyther <holger@moiji-mobile.com> | 2014-04-19 19:22:22 +0200 |
| commit | 3e5b6db2d26f279f9238089f8c79de1d71a997aa (patch) | |
| tree | 46f253ce31ce42007abcb07ff4636bc69c3c0a89 | |
| parent | d59bbd16a753554747adcdec18b5cea28a839604 (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.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/common/handover.c b/src/common/handover.c index ab3cbcf8..26a3f51a 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 */ |