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

invalid LEB128 sequence #34

Closed
carlospaulino opened this issue Mar 29, 2019 · 2 comments
Closed

invalid LEB128 sequence #34

carlospaulino opened this issue Mar 29, 2019 · 2 comments

Comments

@carlospaulino
Copy link

carlospaulino commented Mar 29, 2019

For a while I’ve been trying to debug why I get invalid LEB128 sequence when working with some test apks. From what I can tell this occurs more often with binaries built with java 7 compatibility. It doesn't occur with binaries built with java 8 compatibility.

I created a sample repo with two almost identical applications with the only difference being the compile compatibility.

invalid LEB128 sequence
com.linkedin.dex.spec.DexException: invalid LEB128 sequence
	at com.linkedin.dex.spec.Leb128$Companion.readUnsignedLeb128(Leb128.kt:26)
	at com.linkedin.dex.parser.DecodedValue$Companion.create(DecodedValue.kt:56)
	at com.linkedin.dex.parser.AnnotationUtilsKt.getTestAnnotation(AnnotationUtils.kt:70)
	at com.linkedin.dex.parser.AnnotationUtilsKt.getClassAnnotationValues(AnnotationUtils.kt:46)
	at com.linkedin.dex.parser.TestMethodKt.createTestMethods(TestMethod.kt:42)
	at com.linkedin.dex.parser.JUnit4ExtensionsKt.findAllJUnit4Tests(JUnit4Extensions.kt:35)
	at com.linkedin.dex.parser.DexParser$Companion.findTestMethods(DexParser.kt:60)

Repo is available here:
https://github.com/carlospaulino/dex-test-parser-bug

@hongbinh
Copy link
Contributor

hongbinh commented Apr 2, 2019

@kkoser found the actual problem is in DecodedValue.kt:54 , we will send a PR with some added tests soon.

@carlospaulino
Copy link
Author

@hongbinh @kkoser Thanks a lot. I am looking forward to the next release.

hongbinh pushed a commit to hongbinh/dex-test-parser that referenced this issue Apr 4, 2019
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