Hi! > + fsync(fd); This should really be SAFE_FSYNC(fd); > + SAFE_FSEEK(stream, 0, SEEK_SET); > + SAFE_READ(1, fd, buff, BUFFSIZE); Otherwise the rest looks fine. With that fixed: Reviewed-by: Cyril Hrubis <chrubis@suse.cz> -- Cyril Hrubis chrubis@suse.cz