Commit 42a74080 authored by Michel Dänzer's avatar Michel Dänzer Committed by Adam Jackson
Browse files

os: Use strtok instead of xstrtokenize in ComputeLocalClient

Fixes leaking the memory pointed to by the members of the array returned
by xstrtokenize.
Reviewed-by: Adam Jackson's avatarAdam Jackson <>
(cherry picked from commit e156c0cc)
parent 3c4cead4
......@@ -1132,19 +1132,20 @@ ComputeLocalClient(ClientPtr client)
* is forwarded from another host via SSH
if (cmdname) {
char **cmd;
char *cmd = strdup(cmdname);
Bool ret;
/* Cut off any colon and whatever comes after it, see
cmd = xstrtokenize(cmdname, ":");
cmd = strtok(cmd, ":");
#if !defined(WIN32) || defined(__CYGWIN__)
cmd[0] = basename(cmd[0]);
ret = strcmp(basename(cmd), "ssh") != 0;
ret = strcmp(cmd, "ssh") != 0;
ret = strcmp(cmd[0], "ssh") != 0;
return ret;
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment