read () - Проверить границы буфера, если используется в цикле, включая рекурсивные циклы - PullRequest
Новая
       7

read () - Проверить границы буфера, если используется в цикле, включая рекурсивные циклы

0 голосов
/ 27 октября 2018

У меня есть этот код, и я запускаю его с Flawinder, и я получаю этот вывод для функций read (): «Проверить границы буфера, если используется в цикле, включающем рекурсивные циклы»

Может кто-нибудь увидеть проблему?

**

#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] = '\0';
}

**

1 Ответ

0 голосов
/ 27 октября 2018

Вы должны проверить возвращаемое значение read(), чтобы узнать, был ли вызов 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() возвращает количество прочитанных байтов, поэтому вместо этого 

buf[len] = '\0';

использование 

buf[ret] = '\0'; /* 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] = '\0';
                /* 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 */
        }
}
...