Age | Commit message (Collapse) | Author |
|
value < 0. errno is only updated in this case. Change all (most?)
callers of syscalls to follow this better, and let's see if this strictness
helps us in the future.
|
|
ok millert nicm tb, etc
|
|
suggestions and ok naddy@ and ok previois diff florian@.
from Hiltjo Posthuma hiltjo AT codemadness DOT org and
Bjoern Ketelaars bjorn DOT ketelaars AT hydroxide DOT nl,
Thanks.
|
|
Fixes a corner case where the socket was closed twice, reported by
Hiltjo Posthuma.
ok deraadt@
|
|
|
|
a rolling computation for the fast-hash.OB With this openrsync is on
par with gpl rsync for file updates.
From kristaps
|
|
from Hiltjo Posthuma
|
|
|
|
|
|
functions.
ok deraadt@
|
|
|
|
with the current formatting.
OK benno@ naddy@
The original suggestion to delete it rather than fix it came from deraadt@:
we do not want to encourage use of --rsync-path because it is important
that the openrsync client smoothly works with the Samba rsync server.
If it doesn't, we should fix it and not fragment the ecosystem.
|
|
and behaviour is the same as GPL rsync.
Initial diff received feedback from benno@, schwarze@, deraadt@ and
florian@. Thanks!
OK deraadt@
|
|
|
|
composition code for the remote process.
ok florian naddy
|
|
tweaks and OK deraadt@ naddy@
|
|
low-utility parts of the more useful and shorter -D. Nice because it
brings the synopsis down to two lines, and because the approach buys us
additional time before the synopsis may grow disgustingly long again.
"Shrug. Probably helps." deraadt@
|
|
in the io-path, whic is done by seperating int vs uint functions variants.
reviewed by naddy, florian, and jsg
|
|
into two functions, rsync_connect() to establish a TCP connection
to the remote daemon, and rsync_socket() to run the actual protocol.
E.g.:
rsync -av --del -e 'ssh -W localhost:rsync -lanoncvs' \
rsync://anoncvs.spacehopper.org/OpenBSD-CVS/ /cvs
ok deraadt@
|
|
aliases where short or shorter forms exist; of course, the long
versions remain in the full option listing as well as in the code.
Also drop some redundant repetions of .Fl macros.
OK jmc@ deraadt@
|
|
|
|
|
|
we know later how rsync wants to behave and how it interacts with future
option development. This sequence is safest, since it drops setugid bits
in case of failure to chown..
ok florian
|
|
Drop --no-delete. The "no-" prefix is only available for options
that are implied by other options. --delete is never implied and
GPL rsync does not accept --no-delete.
ok deraadt@
|
|
|
|
(previous commit was for read-side)
|
|
Diagnosed by krw, ok florian
|
|
yet. In the meantime, make it a warning-no-op, as most usage cases
will be satisfied by this (and hopefully annoy someone to write support)
ok florian benno
|
|
casting it into a time_t. If the remote doesn't muck things up, we may
make it through 2038.. (best that can be done without protocol repair
and coordination)
ok benno
|
|
|
|
to an enum, that's just alien). Documents the return values in each
function and unifies the protocol incompatibility error message.
from kristaps
|
|
because "man rsync" still ought to show the manual page from the port.
Consistently use the name "openrsync" for now.
Issue reported by espie@; OK jmc@.
|
|
ok millert@
|
|
|
|
|
|
ok deraadt benno
|
|
themselves with using it (as front end -- it will talk to the installed
'rsync' on the remote side). this will allow folks to give it a shot,
identify weaknesses, collaborate, etc etc etc
discussed with benno
|
|
so they don't need to be constructed to return errors.
ok benno florian
|
|
read by a C++ compiler, and __* are no lonmger recommended.
https://www.gnu.org/software/libtool/manual/html_node/C-header-files.html
ok deraadt@
|
|
Document that the gid/uid list may be empty. Fixes
https://github.com/kristapsdz/openrsync/issues/10
|
|
If we don't get a uid/gid map, such as with an rsync:// address, we
might not be able to map. So fall back on numeric ids.
|
|
In order for GPL rsync(1) to work with --delete when openrsync is a
remote sender, we need to set the FLIST_TOP_LEVEL bit for all
top-level directories. This is because the GPL rsync(1) uses this bit
to determine which directories to scan for files for deletion.
|
|
Use style(9) for bit
|
|
Remove another potential deadlock when in sender mode with
multiplexing enabled.
|
|
Avoid crash when -n is specified.
|
|
As found by benno@, if --numeric-ids is passed in, rsync does not send
or receive the uid/gid lists at all. This also means that we need not
process the lists, as we're simply going to copy around the same
value.
|
|
|
|
339cf5998c0c022623cd68de50722b6c14543952 Push "error trail" further into code.
baf58ce5fe1bc6ce431b3b0ac8264b83ae8c7d02 Document all arguments. Add
common -av usage. Remove bits about not supporting anything but
files/dirs.
821a811a8c80e52fb56b241fc65a16cae1b4fb2c Disambiguate as prodded by deraadt@
6c4475b8f226e9031ec0ec1b3f14f7d347132c87 Add -h to usage string
4d344ae6156873b44c95de0c1ed629e637c2d7ab Clarify error message
language, use service name instead of port, specify that the socket is
SOCK_STREAM. From deraadt@. Tweaked for lowercase messages.
f3ec049e76257fc96bcdc872f1d3b967b98f3eb6 In consideration to benno@'s
comments, let the mktemp functions propogate an errno handled by the
caller. Also keep the original line lengths. While in mktemp.c, make
some defines into an enum.
e116c2bd00e634b56e4276120135915ceaa31cf2 Put the FSM of the sender
into its own function. Put dry_run ack and end of phase ack into the
send buffer too, further reducing the possibility of deadlock.
c7745aa4c7394ca89d841f8ee76782256d694340 Make the sender write loop be
fully non-blocking. This frees us of deadlocking the protocol because
the sender will always be able to pull down data.
93c7b4843e80aeac2ec6ae6ffc395df4deaf4a31 Remove "yoda" notation to be
more in tune with OpenBSD. Most found by deraadt@.
|
|
|
|
339cf5998c0c022623cd68de50722b6c14543952 Push "error trail" further into code.
baf58ce5fe1bc6ce431b3b0ac8264b83ae8c7d02 Document all arguments. Add
common -av usage. Remove bits about not supporting anything but
files/dirs.
821a811a8c80e52fb56b241fc65a16cae1b4fb2c Disambiguate as prodded by deraadt@
6c4475b8f226e9031ec0ec1b3f14f7d347132c87 Add -h to usage string
4d344ae6156873b44c95de0c1ed629e637c2d7ab Clarify error message
language, use service name instead of port, specify that the socket is
SOCK_STREAM. From deraadt@. Tweaked for lowercase messages.
f3ec049e76257fc96bcdc872f1d3b967b98f3eb6 In consideration to benno@'s
comments, let the mktemp functions propogate an errno handled by the
caller. Also keep the original line lengths. While in mktemp.c, make
some defines into an enum.
e116c2bd00e634b56e4276120135915ceaa31cf2 Put the FSM of the sender
into its own function. Put dry_run ack and end of phase ack into the
send buffer too, further reducing the possibility of deadlock.
c7745aa4c7394ca89d841f8ee76782256d694340 Make the sender write loop be
fully non-blocking. This frees us of deadlocking the protocol because
the sender will always be able to pull down data.
93c7b4843e80aeac2ec6ae6ffc395df4deaf4a31 Remove "yoda" notation to be
more in tune with OpenBSD. Most found by deraadt@.
|