Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

goProbe panics when writing out data for an interface without flows #67

Closed
els0r opened this issue Mar 6, 2023 · 2 comments · Fixed by #68
Closed

goProbe panics when writing out data for an interface without flows #67

els0r opened this issue Mar 6, 2023 · 2 comments · Fixed by #68
Assignees
@fako1024
Labels
bug Something isn't working critical Issues that should be looked at with priority help wanted Extra attention is needed
Milestone
v4 Release

Comments

@els0r
Copy link
Owner

els0r commented Mar 6, 2023 

ts=2023-03-06T17:04:46Z level=debug caller=capture/capture.go:183 msg="there are currently no flow records available" app_name=goProbe app_version=8e02f0a7 iface=eth1 state=capturing
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x751a17]

goroutine 9 [running]:
github.com/els0r/goProbe/pkg/goDB.dbData({0xc0001a2000?, 0x88?}, 0x0?, 0xc000121830?)
        /usr/src/packages/src/sdwan/security/pkg/OSAGtmi/goProbe/pkg/goDB/db_writer.go:107 +0xb7
github.com/els0r/goProbe/pkg/goDB.(*DBWriter).Write(0xc0002c2930, 0xc000121d60?, {0xc00002d540?}, 0x4?)
        /usr/src/packages/src/sdwan/security/pkg/OSAGtmi/goProbe/pkg/goDB/db_writer.go:57 +0x23d
github.com/els0r/goProbe/pkg/goprobe/writeout.(*Handler).HandleWriteouts.func1()
        /usr/src/packages/src/sdwan/security/pkg/OSAGtmi/goProbe/pkg/goprobe/writeout/handler.go:217 +0x790
created by github.com/els0r/goProbe/pkg/goprobe/writeout.(*Handler).HandleWriteouts
        /usr/src/packages/src/sdwan/security/pkg/OSAGtmi/goProbe/pkg/goprobe/writeout/handler.go:179 +0xaa
@els0r els0r added bug Something isn't working critical Issues that should be looked at with priority help wanted Extra attention is needed labels Mar 6, 2023
@els0r els0r added this to the v4 Release milestone Mar 6, 2023
@fako1024
Copy link
Collaborator

fako1024 commented Mar 6, 2023

Yeah, saw that already this morning. is already fixed locally. Didn't want to raise an extra issue, but thanks for reporting!

@fako1024 fako1024 self-assigned this Mar 6, 2023
@fako1024 fako1024 mentioned this issue Mar 6, 2023
Merged
@fako1024 fako1024 linked a pull request Mar 6, 2023 that will close this issue
Fix panic during Flatten() if hashmap is empty #68
Merged
@fako1024
Copy link
Collaborator

fako1024 commented Mar 6, 2023

@els0r See attached PR, feel free to merge at your discretion.

@els0r els0r closed this as completed in #68 Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fako1024 fako1024

Labels
bug Something isn't working critical Issues that should be looked at with priority help wanted Extra attention is needed
Projects
None yet
Milestone
v4 Release
Development

Successfully merging a pull request may close this issue.

Fix panic during Flatten() if hashmap is empty els0r/goProbe
2 participants
@els0r @fako1024