c - 为什么在 printf 中两次调用 inet_ntoa 会给出错误的输出？

Question

我写了一段代码，但我不确定为什么最后一个 printf 给出了意想不到的结果。我的命令行参数是12和15。代码如下:

#include <stdio.h>
#include <netinet/in.h>
#include <arpa/inet.h>

int main(int argc, char *argv[])
{
  struct in_addr adr, adr2;
  int num, num2;

  num = atoi(argv[1]);
  num2 = atoi(argv[2]);

  adr.s_addr = htonl(num);
  adr2.s_addr = htonl(num2);

  printf("%x %x\n", num, adr.s_addr); # c c000000
  printf("%x %x\n", num2, adr2.s_addr); # f f000000

  printf("%s\n", inet_ntoa(adr)); # 0.0.0.12
  printf("%s\n", inet_ntoa(adr2)); # 0.0.0.15
  printf("%s %s\n", inet_ntoa(adr), inet_ntoa(adr2)); # 0.0.0.12 0.0.0.12
  return(0);
}

我预计最终输出为“0.0.0.12 0.0.0.15”，但事实并非如此。有人可以帮忙吗？

Answer 1

inet_ntoa 函数返回一个指向静态缓冲区的指针。在这种情况下，该函数作为另一个函数的参数被调用两次。函数参数的评估顺序是未指定的，因此静态缓冲区的内容将是最后运行的那个。由于它总是返回相同的缓冲区，因此相同的内容将被打印两次。

所以你不能在没有中间序列点的情况下多次调用这样的函数，即单独的语句，短路运算符，如 && 或 ||，逗号运算符或三元运算符。

在这个特定代码的情况下，两次单独调用 printf，每次调用一次 inet_ntoa(就像您在前两行中所做的那样)是执行此操作的正确方法。

此外，如果您计划将此函数的结果保存在某个地方，则不能只保存指针，因为每次调用 inet_ntoa 时它指向的数据都会更改。您需要使用 strcpy 或 strdup 复制它。