diff options
| author | baude <bbaude@redhat.com> | 2019-07-23 10:07:06 -0500 |
|---|---|---|
| committer | baude <bbaude@redhat.com> | 2019-07-23 10:13:04 -0500 |
| commit | a793bccae6d568d3f30534d66bea3d5a1e8d9302 (patch) | |
| tree | 0e35bba61c97582527f9bd02cd48ad1b70e75c6f /pkg/adapter/terminal_linux.go | |
| parent | ce60c4d30c4acfa0c3ec9fc584c7eb88f84ac35f (diff) | |
| download | podman-a793bccae6d568d3f30534d66bea3d5a1e8d9302.tar.gz podman-a793bccae6d568d3f30534d66bea3d5a1e8d9302.tar.bz2 podman-a793bccae6d568d3f30534d66bea3d5a1e8d9302.zip | |
golangci-lint cleanup
a PR slipped through without running the new linter. this cleans things
up for the master branch.
Signed-off-by: baude <bbaude@redhat.com>
Diffstat (limited to 'pkg/adapter/terminal_linux.go')
| -rw-r--r-- | pkg/adapter/terminal_linux.go | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/pkg/adapter/terminal_linux.go b/pkg/adapter/terminal_linux.go index 33ca0833b..de2600b75 100644 --- a/pkg/adapter/terminal_linux.go +++ b/pkg/adapter/terminal_linux.go @@ -27,7 +27,11 @@ func ExecAttachCtr(ctx context.Context, ctr *libpod.Container, tty, privileged b return -1, err } defer cancel() - defer restoreTerminal(oldTermState) + defer func() { + if err := restoreTerminal(oldTermState); err != nil { + logrus.Errorf("unable to restore terminal: %q", err) + } + }() } return ctr.Exec(tty, privileged, env, cmd, user, workDir, streams, preserveFDs, resize, detachKeys) } @@ -121,7 +125,7 @@ func handleTerminalAttach(ctx context.Context, resize chan remotecommand.Termina logrus.SetFormatter(&RawTtyFormatter{}) if _, err := term.SetRawTerminal(os.Stdin.Fd()); err != nil { - return nil, nil, err + return cancel, nil, err } return cancel, oldTermState, nil |