Hi Simon,

You wrote to Kurt Kuzba:

SA> KK>    buf = malloc(16384);
SA> KK>       file_size += (block_read = fread(buf, 16384, 1, fp));

SA>Erm, where space for terminating null?

What termiating null?

SA>You're reading in 16384 bytes into a space of 16384, and the terminator's
SA>going to overrun, surely?

As the file was opened in binary mode it will _never_ return more
than the requested number of bytes (obtained by multiplying the unit
size to read by the number of units to read).

A valid criticism of the above code is that it should be:

       file_size += (block_read = fread(buf, 1, 16384, fp));

As written it will give the number of complete reads of 16384 bytes, not
the length of the file.

George

 * SLMR 2.1a * Computers eliminate spare time.

--- Maximus/2 3.01