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.

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

[bug] panic: runtime error: makeslice: len out of range #221

Open
jarrodb opened this issue May 19, 2023 · 2 comments
Open

[bug] panic: runtime error: makeslice: len out of range #221

jarrodb opened this issue May 19, 2023 · 2 comments

Comments

@jarrodb
Copy link
Contributor

jarrodb commented May 19, 2023

Saving this output here for visibility and future reference when troubleshooting:

E0518 21:13:59.110857       1 gobmpsrv.go:95] fail to recover BMP message Common Header with error: invalid version in common header, expected 3 found 241
panic: runtime error: makeslice: len out of range

goroutine 954 [running]:
github.com/sbezverk/gobmp/pkg/gobmpsrv.(*bmpServer).bmpWorker(0xc000024600, {0x972b30?, 0xc000012030})
        /workdir/pkg/gobmpsrv/gobmpsrv.go:99 +0x62e
created by github.com/sbezverk/gobmp/pkg/gobmpsrv.(*bmpServer).server
        /workdir/pkg/gobmpsrv/gobmpsrv.go:53 +0x16a
@sbezverk
Copy link
Owner

sbezverk commented May 29, 2023

@jarrodb thank you for reporting the panic and sorry for the delay with my reply, I was off. Is there anything specific about the environment you can add where this panic occurred? Is it easily reproducible? Processing of Common Header code has not been changed as far as I remember for ages. Curious if it is result of some new format/field recently added. Please share the vendor of BGP speaker.
If the issue is reproducible, could you please run gobmp with --v=9 and share Common Header hex dump.

@jarrodb
Copy link
Contributor Author

jarrodb commented May 30, 2023

Oh, I was saving it for me to remember later and troubleshoot. The particular vendor that generated this stream was Arista. Once I duplicate it I will submit a PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants