search

aristanetworks / goarista

Fairly general building blocks used in Arista Go code and open-sourced for the benefit of all.
Apache License 2.0
206 stars 66 forks source link

Core Dumped when options are set without path for DELETE, REPLACE, UPDATE #75

Closed Aorta3698 closed 1 year ago

Aorta3698 commented 1 year ago

Instead of displaying a friendly error message stating missing paths, the user is instead greeted with the core dump shown below.

This happens for delete, replace, and update requests. 

@ash% gnmi -username admin -addr al331:6030 update origin=openconfig
.
.
.
goroutine 12 [runnable]:
runtime.gcTrigger.test({0x0?, 0x0?, 0x0?})
    runtime/mgc.go:547 +0xfb fp=0xc00060cd30 sp=0xc00060cd28 pc=0x41c29b
runtime.mallocgc(0x60, 0x0, 0x1)
    runtime/malloc.go:1171 +0x768 fp=0xc00060cd98 sp=0xc00060cd30 pc=0x410468
runtime.makechan(0xc000a326c0?, 0x0)
    runtime/chan.go:96 +0x126 fp=0xc00060cdd8 sp=0xc00060cd98 pc=0x408386
google.golang.org/grpc.(*addrConn).updateConnectivityState(0xc0003ef080, 0x1, {0x0?, 0x0})
    google.golang.org/grpc@v1.56.2/clientconn.go:1282 +0x73 fp=0xc00060ce60 sp=0xc00060cdd8 pc=0x955f33
google.golang.org/grpc.(*addrConn).resetTransport(0xc0003ef080)
    google.golang.org/grpc@v1.56.2/clientconn.go:1334 +0x186 fp=0xc00060cf78 sp=0xc00060ce60 pc=0x956326
google.golang.org/grpc.(*addrConn).connect(0xc0003ef080)
    google.golang.org/grpc@v1.56.2/clientconn.go:978 +0x96 fp=0xc00060cfc8 sp=0xc00060cf78 pc=0x954476
google.golang.org/grpc.(*acBalancerWrapper).Connect.func1()
    google.golang.org/grpc@v1.56.2/balancer_conn_wrappers.go:398 +0x26 fp=0xc00060cfe0 sp=0xc00060cfc8 pc=0x94e7a6
runtime.goexit()
    runtime/asm_amd64.s:1598 +0x1 fp=0xc00060cfe8 sp=0xc00060cfe0 pc=0x46f9a1
created by google.golang.org/grpc.(*acBalancerWrapper).Connect
    google.golang.org/grpc@v1.56.2/balancer_conn_wrappers.go:398 +0x59
Aborted (core dumped)