проверьте правильность адреса mmap - PullRequest
Новая
       24

проверьте правильность адреса mmap

5 голосов
/ 26 марта 2010

Я пишу высоконагруженный демон, который должен быть запущен на FreeBSD 8.0, а также на Linux. Основное назначение демона - передавать файлы, которые запрашиваются по их идентификатору. Идентификатор конвертируется в локальное имя файла / размер файла через запрос в дб. И затем я использую последовательные mmap() вызовы для передачи файловых блоков с send().

Однако иногда встречаются несоответствия размера файла в БД и размера файла в файловой системе (реальный размер <размер в БД). В этой ситуации я отправил все реальные блоки данных, и когда отображается следующий блок данных - mmap не возвращает ошибок, только обычный адрес (я также проверил переменную errno, она равна нулю после mmap). И когда демон пытается отправить этот блок, он получает ошибку сегментации. (Такое поведение гарантированно распространяется на FreeBSD 8.0 amd64) </p>

Я использовал безопасную проверку перед открытием, чтобы гарантировать размер при вызове stat(). Однако реальная жизнь показывает мне, что в редких ситуациях сегфо все еще может возникать.

Итак, мой вопрос, есть ли способ проверить, доступен ли указатель, перед разыменованием его? Когда я открыл ядро ​​в GDB, GDB говорит, что данный адрес вышел за пределы. Возможно, есть другое решение, которое кто-то может предложить. 

#include <fcntl.h>
#include <stdio.h>
#include <stdlib.h>
#include <sys/mman.h>
#include <sys/stat.h>
#include <sys/types.h>
#include <time.h>
#include <unistd.h>

#define FILENAME        "./datafile"

int main()
{
    unsigned long i, j;

    srand(time(NULL));
    unsigned long pagesize = sysconf(_SC_PAGESIZE);

    unsigned long basesize = 4 * pagesize;
    unsigned long cropsize = 2 * pagesize;

    // create 4*pagesize sized file
    int f = creat(FILENAME, 0644);
    for (i = 0; i < basesize; i++) {
        unsigned char c = (unsigned char)rand();
        if (write(f, &c, 1) < 1) { perror("write"); break; }
    }
    close(f);

    f = open(FILENAME, O_RDONLY);

    // walk trough file
    unsigned char xor = 0;
    unsigned long offset = 0;
    for (j = 0; j < 4; j++) {
        // trunc file to 2*pagesize
        if (j == 2) truncate(FILENAME, cropsize);

        char *data = mmap(NULL, pagesize, PROT_READ, MAP_PRIVATE, f, offset);
        if (data == MAP_FAILED) { perror("mmap"); break; }
        printf("mmap: %lu@%lu for %i\n", pagesize, offset, f);

        for (i = 0; i < pagesize; i++) xor ^= data[i];

        offset += pagesize;
    }

    close(f);

    return 0;
}

1 Ответ

2 голосов
/ 26 марта 2010

Конечно, я не могу доказать это отсюда, но я сильно подозреваю, что у вас просто ошибка учета в вашем коде. Если вы вызываете mmap и передаете размер, и он успешен, вы не должны получать SIGSEGV.

Я рекомендую вам применить валгринд к вашему расследованию.

Во многих системах linux / proc / PID / maps покажет вам, какие регионы сопоставлены с какими правами доступа.

...