int func(char* str)
{
char buffer[100];
unsigned short len = strlen(str);
if(len >= 100)
{
return (-1);
}
strncpy(buffer,str,strlen(str));
return 0;
}
此代码容易受到缓冲区溢出攻击，我正在try 找出原因.我认为这与len分被宣布为short分有关，而不是int分，但我不太确定.
Any ideas?
在大多数编译器上，unsigned short的最大值是65535.
任何高于该值的值都会被绕回，因此65536变为0,65600变为65.
这意味着正确长度的长字符串(例如65600)将通过判断，并使缓冲区溢出.
使用size_t存储strlen()的结果，而不是unsigned short，并将len与直接编码buffer大小的表达式进行比较.例如:
char buffer[100];
size_t len = strlen(str);
if (len >= sizeof(buffer) / sizeof(buffer[0])) return -1;
memcpy(buffer, str, len + 1);