mirror of
https://github.com/RsyncProject/rsync.git
synced 2026-09-17 07:38:57 -04:00
Vladimir Marek
0c96e12fd6
Fix three bugs in the Solaris extended-attribute write loop and harden it: - the write() length stayed the full size rather than the remaining (size - bufpos), so a short write would re-write from the wrong offset; - on a failed or zero-byte write, bufpos = -1 wrapped to SIZE_MAX (bufpos is size_t) and the final `bufpos > 0` test then returned success; - an empty value (size == 0) returned failure because `bufpos > 0` was false. Also don't let close() clobber the write error's errno, and report a close() failure on an otherwise-successful write.