diff options
| -rw-r--r-- | Makefile | 1 | ||||
| -rw-r--r-- | gdb-native.go | 3 | ||||
| -rw-r--r-- | gdb-xds.go | 8 |
3 files changed, 12 insertions, 0 deletions
@@ -94,6 +94,7 @@ package-all: @echo " WARNING: build on Windows not supported for now." @echo "# Build darwin amd64..." GOOS=darwin GOARCH=amd64 RELEASE=1 make -f $(ROOT_SRCDIR)/Makefile package + make -f $(ROOT_SRCDIR)/Makefile clean vendor: tools/glide glide.yaml ./tools/glide install --strip-vendor diff --git a/gdb-native.go b/gdb-native.go index a4e6189..4bae3d3 100644 --- a/gdb-native.go +++ b/gdb-native.go @@ -151,6 +151,9 @@ func (g *GdbNative) Write(args ...interface{}) error { // SendSignal is used to send a signal to remote process/gdb func (g *GdbNative) SendSignal(sig os.Signal) error { + if g.exeCmd == nil { + return fmt.Errorf("exeCmd not initialized") + } return g.exeCmd.Process.Signal(sig) } @@ -76,6 +76,9 @@ func (g *GdbXds) SetConfig(name string, value interface{}) error { // Init initializes gdb XDS func (g *GdbXds) Init() (int, error) { + // Reset command ID (also used to enable sending of signals) + g.cmdID = "" + // Define HTTP and WS url baseURL := g.uri if !strings.HasPrefix(g.uri, "http://") { @@ -167,6 +170,7 @@ func (g *GdbXds) Close() error { g.cbOnExit = nil g.cbRead = nil g.cbInferiorRead = nil + g.cmdID = "" return nil } @@ -285,6 +289,10 @@ func (g *GdbXds) Write(args ...interface{}) error { // SendSignal is used to send a signal to remote process/gdb func (g *GdbXds) SendSignal(sig os.Signal) error { + if g.cmdID == "" { + return fmt.Errorf("cmdID not set") + } + var body []byte body, err := json.Marshal(apiv1.ExecSignalArgs{ CmdID: g.cmdID, |