diff options
| author | David Kalnischkies <david@kalnischkies.de> | 2014-12-10 22:26:59 +0100 |
|---|---|---|
| committer | David Kalnischkies <david@kalnischkies.de> | 2014-12-23 11:43:16 +0100 |
| commit | e18f6133b254db9e1dc7b202366b067b15a68123 (patch) | |
| tree | cf9fd66c123faa9b96c8c7f4173127ed6a102c1b /test/integration/framework | |
| parent | 92e8c1ff287ab829de825e00cdf94744e699ff97 (diff) | |
do not make PTY slave the controlling terminal
If we have no controlling terminal opening a terminal will make this
terminal our controller, which is a serious problem if this happens to
be the pseudo terminal we created to run dpkg in as we will close this
terminal at the end hanging ourself up in the process…
The offending open is the one we do to have at least one slave fd open
all the time, but for good measure, we apply the flag also to the slave
fd opening in the child process as we set the controlling terminal
explicitely here.
This is a regression from 150bdc9ca5d656f9fba94d37c5f4f183b02bd746 with
the slight twist that this usecase was silently broken before in that it
wasn't logging the output in term.log (as a pseudo terminal wasn't
created).
Closes: 772641
Diffstat (limited to 'test/integration/framework')
| -rw-r--r-- | test/integration/framework | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/integration/framework b/test/integration/framework index ac482a7a0..9e183057f 100644 --- a/test/integration/framework +++ b/test/integration/framework @@ -102,7 +102,7 @@ runapt() { local CMD="$1" shift case $CMD in - sh|aptitude|*/*) ;; + sh|aptitude|*/*|command) ;; *) CMD="${BUILDDIRECTORY}/$CMD";; esac MALLOC_PERTURB_=21 MALLOC_CHECK_=2 APT_CONFIG="$(getaptconfig)" LD_LIBRARY_PATH=${BUILDDIRECTORY} $CMD "$@" |