1、gets() 和 fgets()函数
问题:找出下面代码的问题
#include<stdio.h>
int main(void)
{ char buff[10]; memset(buff,0,sizeof(buff)); gets(buff); printf("\n The buffer entered is [%s]\n",buff); return 0;
}
答:上面代码里的问题在于函数gets()的使用,这个函数从stdin接收一个字符串而不检查它所复制的缓存的容积,这可能会导致缓存溢出。这里推荐使用标准函数fgets()代替。
考查知识:
gets()从stdin接收一个字符串而不检查它所复制的缓存的容积,这可能会导致缓存溢出。
fgets()从stdin接收一个字符串会指定大小,如果超出数组大小,会自动根据定义数组的长度截断。
2、strcpy()函数
问题:下面是一个简单的密码保护功能,你能在不知道密码的情况下将其破解吗?
#include<stdio.h> int main(int argc, char *argv[])
{ int flag = 0; char passwd[10]; memset(passwd,0,sizeof(passwd)); strcpy(passwd, argv[1]); if(0 == strcmp("LinuxGeek", passwd)) { flag = 1; } if(flag) { printf("\n Password cracked \n"); } else { printf("\n Incorrect passwd \n"); } return 0;
}
答:破解上述加密的关键在于利用攻破strcpy()函数的漏洞。所以用户在向“passwd”缓存输入随机密码的时候并没有提前检查“passwd”的容量是否足够。所以,如果用户输入一个足够造成缓存溢出并且重写“flag”变量默认值所存在位置的内存的长“密码”,即使这个密码无法通过验证,flag验证位也变成了非零,也就可以获得被保护的数据了。
考查知识:strcpy()函数缺陷
strcpy()函数的原型为:char *strcpy(char *str1, const char *str2);
strcpy(str2, str1)函数调用中,strcpy()无法检查,str1指向的字符串的大小是否适合str2指向的数组。 如果str2指向的字符串长
度为n,如果str1中有不超过n-1个字符,那么复制操作可以顺利完成。但是如果str1指向了一个更长的字符串,操作结果就得
另当别论了。strcpy(str2, str1)会一直复制到str1的第一个字符'\0'为止,所以它会越过str2的数组边界继续复制。无论后面内存中
的是什么,都将被覆盖。
3.main()的返回类型
问:下面的代码能 编译通过吗?如果能,它有什么潜在的问题吗?
#include<stdio.h> void main(void)
{ char *ptr = (char*)malloc(10); if(NULL == ptr) { printf("\n Malloc failed \n"); return; } else { // Do some processing free(ptr); } return;
}
答: 因为main()方法的返回类型,这段代码的错误在大多数编译器里会被当作警告。main()的返回类型应该是“int”而不是“void”。因为“int”返回类型
会让程序返回状态值。这点非常重要,特别当程序是作为依赖于程序成功运行的脚本的一部分运行时。
4.内存泄露
问:下面的代码会导致内存泄漏吗?
#include<stdio.h> void main(void)
{ char *ptr = (char*)malloc(10); if(NULL == ptr) { printf("\n Malloc failed \n"); return; } else { // Do some processing } return;
}
答:尽管上面的代码并没有释放分配给“ptr”的内存,但并不会在程序退出后导致内存泄漏。在程序结束后,所有这个程序分配的内存都会自动被处理掉。
但如果上面的代码处于一个“while循环”中,那将会导致严重的内存泄漏问题!
考查知识:内存分配使用完后要及时释放,malloc/free new/delete
5.free()函数
问:下面的程序会在用户输入'freeze'的时候出问题,而'zebra'则不会,为什么?
#include<stdio.h> int main(int argc, char *argv[])
{ char *ptr = (char*)malloc(10); if(NULL == ptr) { printf("\n Malloc failed \n"); return -1; } else if(argc == 1) { printf("\n Usage \n"); } else { memset(ptr, 0, 10); strncpy(ptr, argv[1], 9); while(*ptr != 'z') { if(*ptr == '') break; else ptr++; } if(*ptr == 'z') { printf("\n String contains 'z'\n"); // Do some more processing } free(ptr); } return 0;
}
答: 这里的问题在于,代码会(通过增加“ptr”)修改while循环里“ptr”存储的地址。当输入“zebra”时,while循环会在执行前被终止,因此传给free()的
变量就是传给malloc()的地址。但在“freeze”时,“ptr”存储的地址会在while循环里被修改,因此导致传给free()的地址出错,也就导致了seg-fault或
崩溃。
[cn@localhost test]$ ./test zebraString contains 'z'
[cn@localhost test]$ ./test freezeString contains 'z'
*** glibc detected *** ./test: munmap_chunk(): invalid pointer: 0x09b2a00c ***
======= Backtrace: =========
/lib/libc.so.6[0xbbfe31]
./test[0x804854d]
/lib/libc.so.6(__libc_start_main+0xe6)[0xb65d26]
./test[0x80483f1]
6.使用_exit退出
问:在下面的代码中,atexit()并没有被调用,为什么?
#include<stdio.h> void func(void)
{ printf("\n Cleanup function called \n"); return;
} int main(void)
{ int i = 0; atexit(func); for(;i<0xffffff;i++); _exit(0);
}
答:输出结果应该是这样:
L
i
考查知识:++ 和 * 优先级问题
因为“++”和“*”的优先权一样,所以“*ptr++”相当于“*(ptr++)”。即应该先执行ptr++,然后才是*ptr,所以操作结果是“L”。第二个结果是“i”
7、 处理printf()的参数问:下面代码会输出什么?
#include<stdio.h> int main(void)
{ int a = 10, b = 20, c = 30; printf("\n %d..%d..%d \n", a+b+c, (b = b*2), (c = c*2)); return 0;
}
结果: 110..40..60
考查知识:这是因为C语言里函数的参数默认是从右往左处理的,输出时是从左往右。