| Age | Commit message (Collapse) | Author |
|---|
|
|
|
dtach was assuming that writes with a small byte count could never fail,
and was not handling partial writes for larger byte counts in a few
places.
This should also suppress unused result warnings from gcc on systems
that define _FORTIFY_SOURCE by default.
|
|
dtach was not checking if the fchdir back to the old directory worked
when it tried to get around the sun_path length limit.
|
|
gcc now warns that sig is an unused parameter:
attach.c: In function 'win_change':
attach.c:99:16: warning: unused parameter 'sig' [-Wunused-parameter]
99 | win_change(int sig)
| ~~~~^~~
|
|
|
|
Use http://www.gnu.org/licenses/ instead of a postal address as
suggested in PR #7.
|
|
|
|
|
|
dtach -p copies the contents of standard input to a session.
Based on an initial patch by @johnlane.
|
|
If a socket's path name would overflow sun_path, try to make it fit by
temporarily changing to the directory the socket is in and using the
path's basename in sun_path.
This should help for the case where the basename of the socket does fit
in sun_path, even if the rest of the path doesn't.
|
|
The code wasn't checking for overflow before copying the socket path
name to to the sun_path field, which is usually much smaller than
PATH_MAX.
Report and initial patch by Paul Wilkinson.
|
|
valgrind reports:
Syscall param write(buf) points to uninitialised byte(s)
In this case, it is a false positive, but adding this single memset at
attach start-up means the program is now clean when run with "valgrind
--trace-children=yes dtach -c ...", which can make future debugging
easier.
|
|
attach.c did not correctly handle a read from stdin when read returned
an error. The code assigned the return value of read to pkt.len (an
unsigned char) before checking the value. This prevented the error check
from working correctly, since an unsigned integer can never be < 0.
A packet with an invalid length was then sent to the master, which then
sent 255 bytes of garbage to the program.
Fix the bug in attach.c and the unchecked packet length bug in master.c.
Report and initial patch by Enrico Scholz.
|
|
|
|
|
|
paranoid about this, and only remove the specified file if connect says
the connection was refused and stat says the file is a socket.
Also dtach -A now only tries to create the socket if the connection was
refused or the socket did not exist, instead of on any random error as
before.
|
|
exiting silently.
Also make sure the master process waits until the client attaches when using
the -A and -c modes, so that the error message has a chance of being seen.
|
|
the cursor size, which is handled incorrectly by some terminal emulators.
|
|
handler isn't reset on certain systems.
|
|
character method again.
|
|
select when checking the file descriptors for activity.
|
|
|
|
Prevent atexit from being called twice on dtach -A.
|
|
work on: FreeBSD, Debian/alpha, Debian/sparc, Debian/PPC, and Solaris.
Bump version.
|
|
|
