使用strcmp的分段错误（Ubuntu）

Question

我有以下代码段：

token = strtok(line, separator);
i = 0;
args[i++] = token;  /* build command array */
while( token != NULL ) /* walk through other tokens */
{
    /* printf( " %s\n", token ); */
    token = strtok(NULL, separator);
    if (strcmp(token, "|") != 0) {
        printf("entered");
    }
    else {
        args[i++] = token;
    }   /* and build command array */

代码可以正常编译，但是在执行时传递任何参数，都会收到错误“分段错误（核心已转储）”。 删除比较两个字符串的if语句即可解决此问题，因此该比较存在问题。

Answer 1

当strtok找不到下一个令牌时，它返回NULL ，因此在尝试strcmp(NULL, "|")

测试strcmp之前token是否为null。

Answer 2

发布的代码：

token = strtok(line, separator);
i=0;
args[i++]=token;  /* build command array */
while( token != NULL ) /* walk through other tokens */
{
  /* printf( " %s\n", token ); */
  token = strtok(NULL, separator);
  if (strcmp(token, "|") != 0) {
      printf("entered");
  }
  else {
      args[i++] = token;
  }   /* and build command array */

包含逻辑错误。 在检查之前，将使用从strtok()返回的值。

建议类似以下内容：

i=0;
token = strtok(line, separator);

while( token != NULL ) /* walk through other tokens */
{
    /* printf( " %s\n", token ); */
    if (strcmp(token, "|") != 0)
    {
        printf("entered");
    }

    else
    {
        args[i++] = token;
    }   /* and build command array */

    ...
    token = strtok(NULL, separator);
} // end while