Inadequate error checking in rsync 2.5.5

[David Norwood]

> I've had a chance to think on it.  Attached is a patch that
> allows unmap_file() to report the first read error that
> map_ptr found.  The behaviour is the same.  I doubt this will
> apply against anything but CVS HEAD as of now.
>
> This should probably use FERROR instead of FINFO so that a
> partial transfer is reported.
>
> Any thoughts guys?

Okay, with this patch I now get error messages, but it still creates null
filled output files.  I guess that is what you meant by "the behavior is the
same".

David