Skip to content

Commit b3f229b

Browse files
committed
fix double getCmd calls
1 parent 7d3a77d commit b3f229b

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

pkg/jobmanager/jobmanager.go

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -244,7 +244,7 @@ func (jm *JobManager) StartJob(msc *MainServerConn, data wshrpc.CommandStartJobD
244244
log.Printf("StartJob: connected stream streamid=%s serverSeq=%d\n", data.StreamMeta.Id, serverSeq)
245245
}
246246

247-
_, cmdPty := jobCmd.GetCmd()
247+
cmd, cmdPty := jobCmd.GetCmd()
248248
if cmdPty != nil {
249249
log.Printf("StartJob: attaching pty reader to stream manager")
250250
err = jm.StreamManager.AttachReader(cmdPty)
@@ -257,7 +257,6 @@ func (jm *JobManager) StartJob(msc *MainServerConn, data wshrpc.CommandStartJobD
257257
log.Printf("StartJob: no pty to attach")
258258
}
259259

260-
cmd, _ := jobCmd.GetCmd()
261260
if cmd == nil || cmd.Process == nil {
262261
log.Printf("StartJob: cmd or process is nil")
263262
return nil, fmt.Errorf("cmd or process is nil")

0 commit comments

Comments
 (0)