read() - 如果在包括递归循环的循环中使用，则检查缓冲区边界

Question

我有这段代码，运行它与 Flawinder 一起使用，我在 read() 函数上得到了这个输出：

Check buffer boundaries if used in a loop including recursive loops

谁能看出问题所在？

#include <stdlib.h>
void func(int fd)
{

char *buf;
size_t len;
read(fd, &len, sizeof(len));

if (len > 1024)
return;
buf = malloc(len+1); 
read(fd, buf, len); 
buf[len] = '[=10=]';
}

Answer 1

您应该检查 read() 的 return 值以了解对 read() 的调用是成功还是失败，或者 read() 是否被信号中断然后设置errno。例如

ssize_t ret = read(fd, &len, sizeof len);
if( (ret == -1 || ret != sizeof len) {
   /* error handling @TODO */
}

这里最重要

ret = read(fd, buf, len); /* read() may read less than len characters */

read() returns 读取的字节数，所以不用这个

buf[len] = '[=12=]';

使用

buf[ret] = '[=13=]'; /* correct way */

示例代码

void func(int fd) { /* assume fd is a valid file descriptor */
        char *buf = NULL;
        size_t len;
        errno = 0; /* set this to 0 */
        ssize_t ret = read(fd, &len, sizeof len);
        if( (ret == -1 || ret != sizeof len) {
                /* error handling @TODO */
        }
        if (len > 1024) {
                return;
        }
        buf = malloc(len+1); 
        if(buf == NULL) {
                /* error handling @TODO */
        }
        ret = read(fd, buf, len);
        if(ret!=-1) {
                buf[ret] = '[=14=]';
                /* do something with buf and free it once usage is done*/
        }       free(buf); /* free the buf */
        else { /* if read failed */
                free(buf); /* free the buf */
        }
}

read() - 如果在包括递归循环的循环中使用，则检查缓冲区边界

read() - Check buffer boundaries if used in a loop including recursive loops

c

io

buffer-overflow

flawfinder