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

safeparcel: Support short #2019

Merged
merged 1 commit into from
Aug 29, 2023
Merged

Conversation

chirayudesai
Copy link
Contributor

  • Fixes crash with certain USB security keys, exception:
Caused by: java.lang.RuntimeException: Type is not yet usable with SafeParcelReflectionUtil: short at org.microg.safeparcel.SafeParcelReflectionUtil$ClassDescriptor$FieldDescriptor.getType(SafeParcelReflectionUtil.java:521)

Full stacktrace: https://zifb.in/raw/Q89aPk61T4
Tested using: https://demo.yubico.com/webauthn-technical/registration

* Fixes crash with certain USB security keys, exception:
	FATAL EXCEPTION: main
	Process: com.google.android.gms:ui, PID: 6301
	java.lang.ExceptionInInitializerError
	       at java.lang.reflect.Field.get(Native Method)
	       at org.microg.safeparcel.SafeParcelReflectionUtil.getCreator(SafeParcelReflectionUtil.java:123)
	       at org.microg.safeparcel.SafeParcelReflectionUtil$ClassDescriptor$FieldDescriptor.<init>(SafeParcelReflectionUtil.java:465)
	       at org.microg.safeparcel.SafeParcelReflectionUtil$ClassDescriptor.<init>(SafeParcelReflectionUtil.java:411)
	       at org.microg.safeparcel.ReflectedSafeParcelableCreatorAndWriter.<init>(ReflectedSafeParcelableCreatorAndWriter.java:18)
	       at org.microg.safeparcel.AutoSafeParcelable$AutoCreator.<init>(AutoSafeParcelable.java:45)
	       at com.google.android.gms.fido.fido2.api.common.UvmEntries.<clinit>(UvmEntries.java:80)
	       <snipped>
	Caused by: java.lang.RuntimeException: Type is not yet usable with SafeParcelReflectionUtil: short
	       at org.microg.safeparcel.SafeParcelReflectionUtil$ClassDescriptor$FieldDescriptor.getType(SafeParcelReflectionUtil.java:521)

Change-Id: Ie1c523d932ac213c531edf1e227c6ddcb72916bd
@mar-v-in mar-v-in merged commit 3b7feb3 into microg:master Aug 29, 2023
1 check passed
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

Successfully merging this pull request may close these issues.

2 participants