• Kinglong Mee's avatar
    nfs/blocklayout: Fix bad using of page offset in bl_read_pagelist · 15ae2c7b
    Kinglong Mee authored
    Blocklayout uses file offset for the read-back page's offset of first writing,
    it's definitely wrong, it writes data to bad address of page that cause userspace
    application segment fault. It must be the page base stored in header->args.pgbase.
    
    Also, the pg_offset has no influence with isect and extent length.
    
    Note: The offset of the non-first page is always zero.
    
    Ps: A test program will segment fault at read() as,
    #define _GNU_SOURCE
    
    #include <stdio.h>
    #include <stdlib.h>
    #include <sys/types.h>
    #include <sys/stat.h>
    #include <unistd.h>
    #include <fcntl.h>
    #include <errno.h>
    
    int main(int argc, char **argv)
    {
            char buf[2049];
            char *filename = NULL;
            int fd = -1;
    
            if (argc < 2) {
                    printf("Usage: %s filename\n", argv[0]);
                    return 0;
            }
    
            filename = argv[1];
            fd = open(filename, O_RDONLY | O_DIRECT);
            if (fd < 0) {
                    printf("Open %s fail: %m\n", filename);
                    return 1;
            }
    
            lseek(fd, 2048, SEEK_SET);
            if (read(fd, buf, sizeof(buf) - 1) != (sizeof(buf) - 1))
                    printf("Read 4096 bityes data from %s fail: %m\n", filename);
    out:
            close(fd);
            return 0;
    }
    Signed-off-by: default avatarKinglong Mee <kinglongmee@gmail.com>
    Signed-off-by: default avatarTrond Myklebust <trond.myklebust@primarydata.com>
    15ae2c7b
blocklayout.c 23.1 KB