fread question

[rtnMichael]

hey, can someone tell me why I'm printing out more than the size of the array after I fread into it. Here's what I'm doing:
char strttimestamp[12];
if (!(GetTimeStamp(strttimestamp, sizeof(strttimestamp))))

//sub
GetTimeStamp(timestamp, size)
char timestamp[12];
int size;
{
...
fread(timestamp, 1, size, strttimestamp_fp)
...
}

and this is what it prints out:
101597184028!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

I've tried a number of things, and I'm losing confidence in my programming skills...

Thanks
M

 

[newmangj]

I presume your timestamp is 12 bytes long. If you read these 12 bytes into a 12 byte area of memory there is no trailing NULL after the last of the timestamp characters. A printf will then print characters starting at the beginning of you strtimestamp area and continuing through memory from that address until it finds a NULL byte.

If you timestamp is 12 bytes long then the strtimestamp buffer should be 13 bytes long and you should set the 13th byte to NULL (strtimestamp[12]=0x00)

Cheers

 

[rtnMichael]

now I can see clearly. Thanks for the extra set of eyes.

M

 

[newmangj]

Glad to help...