函数malloc分配了一个指针并读取了数据，但调用者无法访问它

Question

我有一个特定的问题：尝试使用自己的函数将文件加载到内存中时遇到问题。

load_file_on_memory()接收文件名和指针。 它只是打开文件，为文件内容动态分配内存并填充它，然后将destiny作为参数将指针传递给调用方。

但是我有一个问题：在load_file_on_memory()函数中，我在malloc的内容空间上取得了成功。 我也可以将数据放在指针上并获取它。

当我尝试使用外部指针功能时，调用方只会被回收。 我将在此处发布我的代码（不久）。

我不明白为什么会这样。 我正在将Windows 7与Tiny C编译器一起使用。 我不知道该环境是否会导致错误。

这是我的资料

#include <stdio.h>
#include <stdlib.h>

int load_file_on_buffer(char filename[], void *buffer_destiny){
    FILE *file_loaded;
    unsigned int file_size;

    file_loaded = fopen(filename, "rb");
    if(file_loaded == NULL)
        return -1;

    fseek(file_loaded, 0, SEEK_END);
    file_size = ftell(file_loaded);
    fseek(file_loaded, 0, SEEK_SET);

    buffer_destiny = (void *) malloc(file_size);

    fread((char *) buffer_destiny, file_size, 1, file_loaded);

    printf("BEGIN DEBUG FILE\n");
    printf("%s", buffer_destiny);
    printf("\n\nEND DEBUG FILE\n");

    fclose(file_loaded);

    return 0;
}

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

    char *buffered_input_file;

    ///First, load input file
    if(load_file_on_buffer("test.txt", (void *)buffered_input_file) != 0)
        return -1;

    printf("Input file loaded successfully\n");
    printf("This function must print file content: \n\n");
    printf("%s", buffered_input_file);
}

Answer 1

您正在传递参数void * buffer_destiny。 输入函数后，这只是一个普通变量。 您使用malloc（）调用覆盖了它。 这对调用函数没有影响。 而是将函数声明为

int load_file_on_buffer(char filename[], void **pbuffer_destiny)

称其为

void* buffered_input_file;
load_file_on_buffer("test.txt", &buffered_input_file)

并在函数中写

void* buffer_destiny = malloc (...);
*pbuffer_destiny = buffer_destiny; 

Answer 2

指针buffer_destiny是在函数的局部变量load_file_on_buffer 。 更改buffer_destiny对指针没有影响buffer_input_file在main 。 解决该问题的一种方法是将指针return给main 。

另一个问题是您正在使用"%s"来打印文件内容。 为了使其正常工作，您需要适当地NUL终止缓冲区。 换句话说，缓冲区需要比文件大小大一个字节，并且NUL终止符'\\0'需要放置在该额外字节中。

#include <stdio.h>
#include <stdlib.h>

char *load_file_on_buffer( const char *filename )
{
    FILE *file_loaded;
    unsigned int file_size;

    file_loaded = fopen(filename, "rb");
    if(file_loaded == NULL)
        return NULL;

    fseek(file_loaded, 0, SEEK_END);
    file_size = ftell(file_loaded);
    fseek(file_loaded, 0, SEEK_SET);

    char *buffer_destiny = malloc(file_size + 1);

    fread( buffer_destiny, file_size, 1, file_loaded );
    buffer_destiny[file_size] = '\0';

    printf("BEGIN DEBUG FILE\n");
    printf("%s", buffer_destiny);
    printf("\n\nEND DEBUG FILE\n");

    fclose(file_loaded);

    return buffer_destiny;
}

int main(int argc, char *argv[])
{
    char *buffered_input_file;

    ///First, load input file
    if( (buffered_input_file = load_file_on_buffer("test.txt")) == NULL )
        return -1;

    printf("Input file loaded successfully\n");
    printf("This function must print file content: \n\n");
    printf("%s", buffered_input_file);
}

Answer 3

如果您仍然想将buffered_input_file作为参数传递，那么它将对您有用，我在某些地方对代码进行了注释，我认为需要澄清。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

int load_file_on_buffer(char filename[], void **buffer_destiny)
{
    FILE *file_loaded;
    unsigned int file_size;

    if (buffer_destiny == NULL)
        return -1;

    *buffer_destiny = NULL; /* to check in the calling function */
    file_loaded     = fopen(filename, "r"); /* rb is no longer needed in new code */
    if(file_loaded == NULL)
        return -1;

    fseek(file_loaded, 0, SEEK_END);
    file_size = ftell(file_loaded);
    fseek(file_loaded, 0, SEEK_SET);

    *buffer_destiny = malloc(1 + file_size); /* +1 for the null terminator */
    if (*buffer_destiny == NULL)
    {
        fclose(file_loaded);
        return -1;
    }
    /* add a null terminator so the string is acceptable by printf */
    memset(*buffer_destiny + file_size, '\0', 1);

    /* you got this wrong
    *
    * fread((char *) buffer_destiny, file_size, 1, file_loaded);
    * 
    * the correct way is
    * 
    * fread(void *ptr, size_t size, size_t count, FILE *fp);
    * 
    * where size is the size of each element and count is the number of elements.
    */
    if (fread(*buffer_destiny, 1, file_size, file_loaded) != file_size)
    {
        free(*buffer_destiny);
        *buffer_destiny = NULL;

        fclose(file_loaded);

        return -1;
    }

    printf("BEGIN DEBUG FILE\n");
    printf("%s", (char *)*buffer_destiny);
    printf("END DEBUG FILE\n");

    fclose(file_loaded);

    return 0;
}

int main(int argc, char *argv[])
{
    char *buffered_input_file;

    if (load_file_on_buffer("data.dat", (void **)&buffered_input_file) != 0)
        return -1;
    printf("Input file loaded successfully\n");
    printf("This function must print file content: \n\n");
    printf("%s", buffered_input_file);

    free(buffered_input_file);
    return 0;
}

Answer 4

在main（）中，buffered_input_file最初为NULL，并按值传递给load_file_on_buffer（）。 buffer_destiny最初从main（）获得NULL，但在分配内存后的内部函数中，它应如您所述正常运行。 但是buffer_destiny是一个局部变量，而不是传递变量的指针，因此在函数结束时，它会丢失其信息，并且main（）中的buffered_input_file仍为NULL。