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

Fixed IllegalArgumentException in LocationServicesOkObservableApi23Factory #573

Merged
merged 4 commits into from
May 13, 2019

Conversation

dariuszseweryn
Copy link
Owner

Due to a race condition with subscribing/unsubscribing from different threads it may have happened that unregistering was made before registering finished resulting in an exception being thrown.

Fixes #572

…OkObservableApi23Factory

Due to a race condition and subscribing/unsubscribing from different threads it may have happened that unregistering was made before registering finished.
@dariuszseweryn dariuszseweryn added the bug Bug that is caused by the library label May 13, 2019
@dariuszseweryn dariuszseweryn added this to the 1.9.2 milestone May 13, 2019
@dariuszseweryn dariuszseweryn self-assigned this May 13, 2019
@dariuszseweryn dariuszseweryn merged commit f9d2779 into master May 13, 2019
@dariuszseweryn dariuszseweryn deleted the fix/location_services_ok_unsubscription branch May 13, 2019 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug that is caused by the library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

LocationServicesOkObservable crash
1 participant