0

我正在尝试映射一个 .img 文件，但我不确定我的代码为什么不起作用。

这是我的代码，当我运行代码时，我不断收到错误，即 p 等于 MAP_FAILED

int diskinfo(int argc, char* argv[]){

void *p;
char *size

    if (argc < 2) {
            printf("Please put ./diskinfo <file> \n");
            exit(1);
    }

    int fp = open(argv[1],"rb+");
    if(fp == NULL) {
            printf("Error opening file");
            exit(1);
    }

    struct stat buf;
    fstat(fp, &buf);

    p = mmap(NULL,buf.st_size, PROT_READ, MAP_PRIVATE, fp, 0);

    if(p == MAP_FAILED){
            printf("Error mapping memory\n");
            exit(1);
    }}

如果有人对我的代码错误或我遗漏了一条信息有任何建议，我将不胜感激。

更改为 perror() 不起作用。更改此函数也不会改变 p 仍然等于 MAP_FAILED 的事实

if(p == MAP_FAILED){
        return;
}

我更改了以下解决方案：

int fp = open(argv[1],O_RDWR);
if(fp < 0){ 
    . . .

但我还是回来了

4

2 回答 2

3

您的代码一开始就不起作用，请检查您的open电话。

如果您使用正确的标志进行编译，-Wall -Werror您应该会收到如下警告：

error: passing argument 2 of ‘open’ makes integer from pointer without a cast [-Werror]
note: expected ‘int’ but argument is of type ‘char *’

你应该指定open和fopen函数，它们是不同的东西。

int open(const char *pathname, int flags);
于 2018-03-16T22:00:14.400 回答
1

从您的问题状态来看，您是否能够mmap上班尚不清楚。您最后的编辑添加：

int fp = open(argv[1],O_RDWR);

如果您正在回写已打开的文件，这很好，但如果没有，您应该打开 usingO_RDONLY以防止对原始文件的无意修改。

虽然不是错误，但fp通常用作与文件流操作相关联的文件指针fopen，当使用. 在这里，您使用的是低级 I/O，read/write它使用文件描述符而不是流指针。引用描述符时，一般白话fd用作文件描述符的简写。（就个人而言，看到两者以互换的方式使用是很尴尬的——我怀疑其他人也是如此）

您对 的剩余使用fstat、结果buf.st_size和您的调用mmap都不是问题。您的问题出在其他地方 - 这是您应该发布A Minimal, Complete, and Verifiable Example (MCVE)的主要原因之一。

也就是说，为了确保您以正确的方式合并您的更改，我将为您提供一个简单的示例，即mmapsa 文件并简单地将文件转储到stdout（因此将您的输入文件名限制为一个合理的短文本文件以使用该示例-- 否则你会看到各种奇怪的字符）。完成以下工作：

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <errno.h>
#include <fcntl.h>
#include <limits.h>
#include <sys/mman.h>
#include <sys/stat.h>
#include <sys/types.h>
#include <unistd.h>

int diskinfo (int argc, char *argv[]) {

    char *p = NULL;         /* pointer to mmapped file */
    int fd = 0;             /* file descriptor */
    struct stat buf = {0};  /* struct stat */
    ssize_t size = 0;       /* file size (typed for write return) */

    if (argc < 2) { /* validate at least 2 arguments */
        printf ("Please put %s <file> \n", argv[0]);
        exit (EXIT_FAILURE);
    }

    if ((fd = open (argv[1], O_RDONLY)) == -1) { /* open/validate file */
        perror ("Error opening file");
        exit (EXIT_FAILURE);
    }

    if (fstat (fd, &buf) == -1) {       /* stat file for size */
        perror ("error: fstat buf");
        exit (EXIT_FAILURE);
    }
    size = buf.st_size;                 /* get file size */

    /* mmap file and validate return */
    if ((p = mmap (NULL, buf.st_size, PROT_READ, MAP_PRIVATE, fd, 0)) 
                    == (void *) -1) {
        perror ("mmap failed");
        exit (EXIT_FAILURE);
    }

    /* simple example, output mmapped file to stdout */
    if (write (STDOUT_FILENO, p, size) != size) {
        perror ("error on write");
        exit (EXIT_FAILURE);
    }

    munmap (p, size);   /* unmap file */

    return 1;   /* return success (fn could be void due to exit) */
}

int main (int argc, char **argv) {

    diskinfo (argc, argv);  /* call diskinfo function */

    return 0;
}

（注意：您的检查if (argc < 2)实际上应该在调用函数中完成，在这里。在您验证您有一个要打开的文件名之前，main()没有理由调用。您实际上可以重构您的代码以检查参数和文件并且简单地将打开的文件描述符作为参数传递）diskinfoopenmain()diskinfo

示例使用/输出

$ ./bin/mmapdiskinfo dat/captnjack.txt
This is a tale
Of Captain Jack Sparrow
A Pirate So Brave
On the Seven Seas.

看看事情，如果你有任何问题，请告诉我。如果您仍然无法让您的功能正常工作，请发布 MCVE，以便我们进一步提供帮助。

于 2018-03-21T00:32:11.010 回答