C语言隐式函数声明带来的错误实例（当隐式声明遇到printf）

关于C语言隐式函数声明的基本问题，请参见我的博文万恶之源：C语言中的隐式函数声明。 下面是最近遇到的一个实例之一。

        client_sock = accept(server_sock, (struct sockaddr*)&client_name, &client_name_len);
        printf("from %s:%d\n", inet_ntoa(client_name.sin_addr), client_name.sin_port);

上述代码段摘自一个网络侦听程序，功能就是打印出客户端的IP地址和端口号。出现的症状是一运行就报段错误(segment Fault)。

网上有很多文章提到了解决方式，但是却没有分析其问题产生的原因。这里我们将彻底分析其原因，并给出解决此类问题的终极方法。

1 原因分析

inet_ntoa()这个函数的调用出了问题，是万恶之源。在万恶之源：C语言中的隐式函数声明中我们说过，C语言编译器对于没有声明原型的函数，通通作为返回类型为整数的函数来处理，参数的类型则由调用的实参自动提升后确定。例如：

non_exist_function(12, 'c');

在编译时，这个没有事先声明的函数将被当作如下形式：

int non_exist_function(int,int);

注意，’c’(char)被提升为了int。

现在回到我们的代码上来。

inet_ntoa(client_name.sin_addr), client_name.sin_port)

将会被当作：

int inet_ntoa(addr_in, unsigned short);

然而实际上真正的inet_ntoa的原型定义在了

extern char *inet_ntoa (struct in_addr __in);

这样返回值本来是指针类型，却被截断成了int类型。对于32位系统来说，由于指针类型和int类型的大小都是32位，恰好不会出现截断的情况。这也是为什么上述代码在32位系统下编译运行不会出现问题的原因。

然而到了64位系统下，char*是64位，int仍然是32位，就出现了截断问题。

然而由于printf(char*, …)是个变参函数，所以调用它时，编译器不会检查可变参数的数据类型，而是按照实参类型进行准备参数入栈。相当于

 printf("from %s:%d\n", 123, 8080);

这样指示符%s对应的第一个参数的类型却是int，从而导致printf()内部在通过va_arg()提取参数时产生错误，最终导致了段错误。

如果把上述代码改写为：

client_sock = accept(server_sock, (struct sockaddr*)&client_name, &client_name_len);
char* s = inet_ntoa(client_name.sin_addr);
printf("from %s:%d\n", s, client_name.sin_port);

编译器就会给出警告信息：

warning: initialization makes pointer from integer without a cast [enabled by default]

这样程序员就容易发现存在的隐式函数声明。
然而我们的实际代码确实非常简洁的一行代码，导致编译器不会给出警告。

* 隐式函数声明+printf()将会导致非常隐蔽的错误！*

2 终极解决方案

GCC有个开关名为： -Wimplicit-function-declaration。只要把这个开关打开就会对所有的隐式声明函数的调用发出警告。

[smstong@cf-19 ~]$ gcc -Wimplicit-function-declaration  1.c
1.c: In function ‘main’:
1.c:61:3: warning: implicit declaration of function ‘inet_ntoa’ [-Wimplicit-function-declaration]
   printf("from %s:%d\n", inet_ntoa(client_name.sin_addr), client_name.sin_port);

这种警告比错误还严重！代码一定要彻底清除这种警告。

知道了原因，解决方法异常简单，只要把包含函数原型声明的头文件包含进来就可以了。

#include <arpa/inet.h>