aboutsummaryrefslogtreecommitdiffstats
path: root/openbsc/src/vty/vty.c
diff options
context:
space:
mode:
authorHolger Hans Peter Freyther <zecke@selfish.org>2010-04-30 13:18:05 +0800
committerHolger Hans Peter Freyther <zecke@selfish.org>2010-04-30 13:29:34 +0800
commitf1af306c94358c0de62d0487e543e518083d5710 (patch)
tree75e154ca1c58a507525131c9f420831e27f100cc /openbsc/src/vty/vty.c
parent1700c933f202260345caf733c5d147a1243aef48 (diff)
[vty] Plug memory leak on auto completion.
I assume the original code crashed with a double free as we have a cleanup at the end of the method. Return from the routine like the case label below. This is fixing a memory leak I am experimenting.
Diffstat (limited to 'openbsc/src/vty/vty.c')
-rw-r--r--openbsc/src/vty/vty.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/openbsc/src/vty/vty.c b/openbsc/src/vty/vty.c
index 5e4902441..fffa649c7 100644
--- a/openbsc/src/vty/vty.c
+++ b/openbsc/src/vty/vty.c
@@ -915,7 +915,9 @@ static void vty_complete_command(struct vty *vty)
vty_backward_pure_word(vty);
vty_insert_word_overwrite(vty, matched[0]);
vty_self_insert(vty, ' ');
- //talloc_free(matched[0]);
+ talloc_free(matched[0]);
+ vector_only_index_free(matched);
+ return;
break;
case CMD_COMPLETE_MATCH:
vty_prompt(vty);