>char buffer[4096]="";
There is no need to initialize all 4K bytes with Nuls.
>length = fread(buffer, sizeof(char), 4096, fp);
Instead of hardcoding 4096, just use: sizeof(buffer)
(Provided it isn't a pointer.)
>char buffer[4096]="";
There is no need to initialize all 4K bytes with Nuls.
>length = fread(buffer, sizeof(char), 4096, fp);
Instead of hardcoding 4096, just use: sizeof(buffer)
(Provided it isn't a pointer.)