Instead of allocating the space for struct stat (and presumably free-ing it later on in the same function), why not declare it on the stack. I hardly ever see any application that actually passes the full 'struct stat' around, it is typically only used to check some timestamp or extract some specific fields and return.

{
struct stat finfo;
stat(..., &finfo);
return (finfo.st_mtime > time);
}
_________________________
40GB - serial #40104051 gpsapp