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

UpConvertTypes may return invalid values #8

Closed
dcferreira opened this issue Feb 16, 2018 · 1 comment
Closed

UpConvertTypes may return invalid values #8

dcferreira opened this issue Feb 16, 2018 · 1 comment

Comments

@dcferreira
Copy link
Contributor

I'm testing the implementation of the get function, and it doesn't seem to work.
It fails in flows/record.go, because UpConvertTypes returns (what fmt prints as) <bad>.

This happens here:

go-flows/flows/record.go

Lines 290 to 291 in b044915

ie.Type = UpConvertTypes(init[argumentPos[0]].ie.Type, init[argumentPos[1]].ie.Type)
ie.Length = ipfix.DefaultSize[ie.Type]

and then there's an index out of range error, since ie.Type isn't a valid index in ipfix.DefaultSize.

Since I don't understand what this if chain is comparing (and I see some exceptions for select/map/apply in other places), it might also be that the get/slice functions require some changes here.

@dcferreira
Copy link
Contributor Author

Update: the feature in question is {"get": [0, "__label"]}.

notti added a commit that referenced this issue Jun 11, 2018
@notti notti closed this as completed in 630a8a0 Jun 11, 2018
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

1 participant