C 读取文件末尾的随机字符

Question

I've been trying to read a file using code I got from this answer.

This is my code:

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

char * readfile(FILE * f)
{
    char * buffer = 0;
    long length;
    

    if (f)
    {
        fseek(f, 0, SEEK_END);
        length = ftell (f);
        fseek(f, 0, SEEK_SET);
        buffer = malloc(length);
        if (buffer) fread(buffer, 1, length, f);
        fclose(f);
    }

    return buffer;
}

int main()
{
    FILE * f = fopen("./file.txt", "r");
    printf(readfile(f));
}

My function does return the contents of the file but it also returns random characters.

contents╪▐>c☻ú

And amazingly they seem to change every time I run the code.

contents╗╙ÿ▓.┤

The contents of my file are "contents".

My question is: How can I fix my code for it not to return random characters?

Answer 1

Strings in C need to be null terminated. Your buffer string is not terminated when you read the contents. Essentially you are printing the files contents as desired, but the read just keeps on going in contiguous memory until it reaches a null character by chance.

Answer 2

My function does return the contents of the file but it also returns random characters.

No. The function returned only a pointer , not any characters.

It is the printf() call that used that pointer, thinking it was a pointer to a string . In C, a string always contains a final '\0' , else it is not a string. It was only a pointer to a character array lacking a null character and led to undefined behavior as printf(s) only knows where to start (at s ), but not where to end.

A better approach: return a pointer to a string .

Best to add error checking too.

// Let calling code close the file
char *readfile_alt(FILE * f) {
  if (f == NULL || fseek(f, 0, SEEK_END) || (length = ftell(f)) == -1) {
    return NULL;
  }

  char *buffer = length < SIZE_MAX ? malloc(length + 1LU) : NULL;
  if (buffer == NULL || fread(buffer, 1, length, f) != length) {
    free(buffer); 
    return NULL;
  }
  buffer[length] = '\0'; // Now buffer points to a string
  return buffer;
}

Also, printf() expects a format string.
Better to use printf("%s", readfile(f)); .

int main() {
  FILE * f = fopen("./file.txt", "r");
  if (f) {
    char *s = readfile(f);
    if (s) {
      printf("%s\n", s);
      free(s); // Free resource when done with them.
    }
    fclose(f); // Free resource when done with them.
  }
}