From 638b73a046e21a2a033e9831cf622e96aa1c8fc9 Mon Sep 17 00:00:00 2001 From: Peter Hunt Date: Mon, 22 Jul 2019 16:10:02 -0400 Subject: remove unnecessary conversions Signed-off-by: Peter Hunt --- pkg/varlinkapi/containers.go | 5 ++--- pkg/varlinkapi/virtwriter/virtwriter.go | 4 ++-- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/pkg/varlinkapi/containers.go b/pkg/varlinkapi/containers.go index 97eb36014..6e2b26c0e 100644 --- a/pkg/varlinkapi/containers.go +++ b/pkg/varlinkapi/containers.go @@ -808,7 +808,7 @@ func (i *LibpodAPI) ExecContainer(call iopodman.VarlinkCall, opts iopodman.ExecO resizeChan := make(chan remotecommand.TerminalSize) reader, writer, _, pipeWriter, streams := setupStreams(call) - //reader, _, _, pipeWriter, streams := setupStreams(call) + type ExitCodeError struct { ExitCode uint32 Error error @@ -837,8 +837,7 @@ func (i *LibpodAPI) ExecContainer(call iopodman.VarlinkCall, opts iopodman.ExecO ecErr := <-ecErrChan - // TODO FIXME prevent all of these conversions - exitCode := int(ecErr.ExitCode) + exitCode := ecErr.ExitCode if errors.Cause(ecErr.Error) == define.ErrOCIRuntimePermissionDenied { exitCode = define.ExecErrorCodeCannotInvoke } diff --git a/pkg/varlinkapi/virtwriter/virtwriter.go b/pkg/varlinkapi/virtwriter/virtwriter.go index 8cdedeed8..bde3107df 100644 --- a/pkg/varlinkapi/virtwriter/virtwriter.go +++ b/pkg/varlinkapi/virtwriter/virtwriter.go @@ -171,11 +171,11 @@ func Reader(r *bufio.Reader, output, errput, input io.Writer, resize chan remote } // HangUp sends message to peer to close connection -func HangUp(writer *bufio.Writer, ec int) (err error) { +func HangUp(writer *bufio.Writer, ec uint32) (err error) { n := 0 msg := make([]byte, 4) - binary.BigEndian.PutUint32(msg, uint32(ec)) + binary.BigEndian.PutUint32(msg, ec) writeQuit := NewVirtWriteCloser(writer, Quit) if n, err = writeQuit.Write(msg); err != nil { -- cgit v1.2.3-54-g00ecf