Malloc - размер базы файлового буфера в существующем файле.

Это неправильно: sizeof(fileBuff). Это будет размер указателя, который в вашей системе равен 4.

Вы не можете использовать sizeof для «извлечения» размера блока памяти, возвращаемого malloc(). Вы не можете использовать что-нибудь для извлечения этого размера, это просто невозможно в (стандартном) C. Вам нужно использовать значение size, то есть аргумент для malloc().

Кроме того, ftell() возвращает long, а не int, и как malloc(), так и различные вызовы ввода-вывода могут завершиться ошибкой, что вам необходимо принять во внимание.

На мой взгляд, использовать буфер размером с файл для простого копирования - не лучшая идея; Намного лучше использовать «разумный» буфер (точный оптимальный размер которого зависит от множества факторов), а затем повторять пары чтения-записи в цикле, пока вы не пройдете через весь файл.

ОБНОВЛЕНИЕ Дополнительные сведения о вашем коде:

1. Вы говорите об использовании strlen(), но в коде также отображается sizeof после fread().
2. Вы говорите об использовании sizeof для «проверки», есть ли что-нибудь в буфере, это невозможно; любое выражение с sizeof всегда оценивается во время компиляции ¹, его нельзя использовать для проверки подобных динамических вещей. И, опять же, вы не можете использовать его для вычисления размера блока памяти, возвращаемого malloc().
3. Использование strlen() в буфере, содержащем данные файла, работает надежно только в том случае, если файл является двоичным и содержит '\ 0' в последней позиции, иначе у вас будет незавершенная строка, и strlen() может вызвать поведение undefine.
4. Как я уже сказал, вам нужно проверить, возвращает ли malloc() NULL, что он и сделает, если не сможет выделить запрошенный блок памяти.

¹ За исключением гибких массивов в C99, но давайте проигнорируем это .

99 разработчиков теперь ответят, что вы берете размер указателя. Мне даже не нужно смотреть на код.

Размер указателя (char *) на вашей (32-битной) платформе всегда равен 4.

Вы не можете использовать sizeof, чтобы определить, сколько памяти было выделено для буфера.

Чтобы проверить, был ли назначен указатель, проверьте возвращаемое значение malloc():

fileBuff = malloc(size);

if (fileBuff == 0) {
   fprintf(stderr, "Error allocating %d bytes.\n", size);
   abort();
}

sizeof оценивается во время компиляции, поскольку вы запрашиваете sizeof из filebuf, который является char* компилятором, вычисляет, что это 4 байта (поскольку размер указателя составляет 4 байта на вашей платформе), и печатает его. malloc, который вы сделали, не имеет ничего общего с sizeof.

Помимо неправильного использования sizeof (), вы можете подумать еще о двух мыслях:

Если это просто копирование файла: не пытайтесь изобретать велосипед, просто используйте функцию system () и вызовите программу ОС, предназначенную для этого (cp в unix , копировать в DOS / Windows).

Если это в целях обучения, и поэтому вы настаиваете на том, чтобы сделать это самостоятельно: не пытайтесь прочитать файл целиком, а затем записать его снова, а читать и записывать фрагмент за фрагментом. Использование буфера больших размеров приводит только к тому, что кеш-память ЦП становится бесполезной. Обычно размер буфера файловой системы или его простая часть соответствует размеру блока goot, поэтому псевдокод должен выглядеть так:

open input file for reading
open output file for writing
as long as read from input file BUFSIZE bytes and read bytes > 0
     do write read data to output file
close input file
close output file

(и не забывайте проверять ошибки ввода-вывода после каждого вызова подпрограммы ввода-вывода!)

И последнее замечание: не используйте fgets (), если вы не знаете наверняка, что это всегда простой текстовый файл. Если вы решите использовать fread () / fwrite (), вы сохраните, даже если это двоичный файл (и он быстрее).