diff options
| author | Andrew Tridgell <tridge@samba.org> | 2003-06-11 06:23:53 +0000 |
|---|---|---|
| committer | Andrew Tridgell <tridge@samba.org> | 2003-06-11 06:23:53 +0000 |
| commit | 1d03ec88f13adb22dfb12efc73c410fca032547b (patch) | |
| tree | aa06dd0c2b0f3f6774aac2d291954c2c1fa8ff04 /source3/client | |
| parent | 98f448b1f43e3edc5687bd373e07b1d022541090 (diff) | |
| download | samba-1d03ec88f13adb22dfb12efc73c410fca032547b.tar.gz samba-1d03ec88f13adb22dfb12efc73c410fca032547b.tar.xz samba-1d03ec88f13adb22dfb12efc73c410fca032547b.zip | |
several places in client.c rely in commands[i].name == NULL being a
termination condition for checking for commands. Make sure the last
element is indeed NULL.
(This used to be commit 294f79884df1ebc4a8d955601e7547bdcadef5d6)
Diffstat (limited to 'source3/client')
| -rw-r--r-- | source3/client/client.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/source3/client/client.c b/source3/client/client.c index bfa587d11a1..f2be268efe2 100644 --- a/source3/client/client.c +++ b/source3/client/client.c @@ -2108,7 +2108,7 @@ static struct /* Yes, this must be here, see crh's comment above. */ {"!",NULL,"run a shell command on the local system",{COMPL_NONE,COMPL_NONE}}, - {"",NULL,NULL,{COMPL_NONE,COMPL_NONE}} + {NULL,NULL,NULL,{COMPL_NONE,COMPL_NONE}} }; @@ -2328,7 +2328,7 @@ static char **completion_fn(const char *text, int start, int end) if (sp == NULL) return NULL; - for (i = 0; commands[i].description; i++) + for (i = 0; commands[i].name; i++) if ((strncmp(commands[i].name, text, sp - buf) == 0) && (commands[i].name[sp - buf] == 0)) break; if (commands[i].name == NULL) |