Skip to content

Smile format out of index bug #26

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

Closed
Maxxan opened this issue Aug 29, 2015 · 3 comments
Closed

Smile format out of index bug #26

Maxxan opened this issue Aug 29, 2015 · 3 comments
Milestone

Comments

@Maxxan
Copy link

Maxxan commented Aug 29, 2015

With latest version (2.6.1) users are starting to get this error:

java.lang.ArrayIndexOutOfBoundsException: length=8000; index=8000
at com.fasterxml.jackson.dataformat.smile.SmileParser._findDecodedFromSymbols(SmileParser.java:1898)
at com.fasterxml.jackson.dataformat.smile.SmileParser._handleFieldName(SmileParser.java:1521)
at com.fasterxml.jackson.dataformat.smile.SmileParser.nextToken(SmileParser.java:587)

@cowtowncoder
Copy link
Member

Thank you for reporting this. A reproducible test case would be great, if possible.
I am unfortunately (wrt Smile module, not me :) ) on vacation, and will be returning next week.
But once I get back I should be able to fix this, and get 2.6.2 update (or, depending, 2.6.1-1 micro-patch) out very soon.

@cowtowncoder
Copy link
Member

Ok, I can see where the problem occurs. I could have sworn I just fixed this problem, or something similar... need to see what gives.

Ah. Actually, it's code shared with CBOR codec, and fix for that was:

FasterXML/jackson-dataformat-cbor#13

but I hadn't seen that in Smile codec, which I thought I had checked. Apparently I did not.
Will try to reproduce with Smile, fix for 2.6.2.

@cowtowncoder cowtowncoder added this to the 2.6.2 milestone Sep 8, 2015
@cowtowncoder
Copy link
Member

Needed to modify test case a bit, but was able to reproduce the bug as well as the fix.
Will be part of 2.6.2.

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