| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
Also drop the rstat of the parent directory, just let
the remote side send us an error.
|
|
|
|
|
|
|
| |
Pass errors up from the transfer functions so we can
keep track of errors and continue trying to transfer
all files. Make output more consistent and all on
stderr.
|
|
|
|
| |
We never did support recursive copy and probably never will.
|
| |
|
|
|
|
|
| |
We really don't need this field since we
always copy data sequentially.
|
| |
|
|
|
|
|
|
|
|
|
| |
This change breaks the qacp protocol!
Before there was a chance we would exit before receiving and checking
all packets from qarshd. Now we look at all packets and check them.
Use data allow packets and larger buffers.
Handle errors on the write end.
|
|
|
|
|
| |
We are already closing it in recvfiles because we could create multiple
connections. This caused us to close the fd twice.
|
|
|
|
|
|
|
| |
When qarshd is run via xinetd, stderr still goes out the socket
and messages from sockutil.c or qarsh_packet.c can interfere
with the protocol. Create a thin wrapper which qacp and qarsh can
send to stderr and qarshd can send to syslog.
|
| |
|
|
|
|
|
|
|
|
|
| |
This coordinates the buffer sizes with the
max packet size. qarshd and qarsh will probably break
if this value does not match between client and server
builds. Also increase the value to reduce overhead.
A max packet size of 16k only yields 40MB/s. Increase
that to 128k and we can do 500MB/s.
|
| |
|
|
|
|
|
|
| |
Net removal of 12 strdup calls
rstat is properly freed
Don't need to strdup for basename, the original is not modified
|
| |
|
|
|
|
|
|
|
|
| |
Only handle one file transfer at a time so we don't need
an array to track multiple transfers or know the remote's
fd number.
Loop in recv_packet until we read a whole packet.
|
| |
|
|
|
|
|
|
|
|
|
|
| |
I removed the buffering layer from recv_packet because
it made the logic too complex around the pselect in qarshd.
Now only read as much as needed to get each packet.
qarshd adds an array for remote file descriptors which is
only a stub for now. This needs to be expanded to allow
multiple file transfers at the same time for runcmd.
|
| |
|
|
|
|
| |
I don't see any way to coexist with the old "protocol"
|
| |
|
| |
|
|
|
|
|
| |
Freeing from the middle of an allocated string is not right.
Keep a pointer to the strdup so we can free it correctly.
|
| |
|
|
|
|
|
| |
A previous commit added tmprstatp, which wasn't being checked just after
it was assigned.
|
|
|
|
|
|
| |
sendfiles should be re-written and broken into single file and multi-file cases to make this cleaner.
Signed-off-by: Dean Jansa <djansa@redhat.com>
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
ugly warnings. We should always be compiling with 64-bit off_t's so
this should not be a problem.
Change nread and nwrote back to ssize_t's since that is what read and write
return. nleft remains an off_t because we initialize to stat.st_size.
|
|
|
|
| |
everything is sent.
|
|
|
|
|
|
| |
I changed all formats from %zd to %lld, but this adds a lot of warnings
because off_t isn't the same as long long int to the compiler. I don't
know if there is a way to avoid the warnings on all platforms.
|
| |
|
| |
|
| |
|
| |
|
|
|