diff options
| author | Pau Espin Pedrol <pespin@sysmocom.de> | 2017-11-02 19:08:14 +0100 |
|---|---|---|
| committer | Neels Hofmeyr <nhofmeyr@sysmocom.de> | 2017-11-10 16:45:02 +0000 |
| commit | 0f7bcb5f17f18682f7f9dc41e2d0fc73c4c08318 (patch) | |
| tree | 57692ea12dfe86b9925b82ed5562e76c9aefb8a1 /src/ctrl/control_if.c | |
| parent | c0fc7940df40de07320c9b2cdd5525659d16805a (diff) | |
vty: Fix bad use of vector_slot()
Commit in e9e9e427b78271941a25a63567fc2ec2bb9e4433 attempted to fix a
compilation warning but introduced a regression documented in OS#2613.
The commit was reverted in 4aa0258269296f078e685e21fb08b115567e814.
After closer lookup and testing, it seems vector_slot(vline, index) is
expected to be NULL in this case as set by vty_complete_command:
/* In case of 'help \t'. */
if (isspace((int)vty->buf[vty->length - 1]))
vector_set(vline, NULL);
As a result, the correct fix for the compilation warning is to test
against NULL instead of testing for empty string.
Change-Id: Id9e02bbf89e0a94e1766b1efd236538712415c8a
Diffstat (limited to 'src/ctrl/control_if.c')
0 files changed, 0 insertions, 0 deletions