c - 有人问我这段代码有什么问题

Question

#include <string.h>

void foo (char *bar)
{

    char c[12];
    strcpy(c, bar);

}
int main (int argc, char **argv)
{

    foo(argv[1]);
    return(1);

}

Answer 1

有两个问题:

• 如果程序没有参数 argv[1] 为 NULL，并且在 foo 中，您执行 strcpy(c, NULL); 具有未定义的行为(通常是崩溃)。
• 如果程序的第一个参数至少有 12 个字符，strcpy(c, bar); 将从 c 中写入，同样具有未定义的行为。

我不会说 strcpy 在最好的情况下是无用的，因为 c 之后没有使用

经过最少更改的程序的安全版本是:

#include <string.h>

void foo (char *bar)
{
  char c[12];

  strncpy(c, bar, sizeof(c) - 1);
  c[sizeof(c) - 1] = 0;
}

int main (int argc, char **argv)
{
  if (argc >= 2)
    foo(argv[1]);
  return(1);
}