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

Crashes when opening Thread #6463

Closed
ganfra opened this issue Jul 5, 2022 · 0 comments · Fixed by #6474
Closed

Crashes when opening Thread #6463

ganfra opened this issue Jul 5, 2022 · 0 comments · Fixed by #6474
Assignees
Labels
A-Threads An issue for the Threads project A-Timeline O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Z-Crash

Comments

@ganfra
Copy link
Member

ganfra commented Jul 5, 2022

Found several times in rageshake reports:

Thread: DefaultTimeline_Thread, Exception: java.lang.IllegalStateException: Cannot create asynchronous query while in a write transaction in /tmp/realm-java/realm/realm-library/src/main/cpp/io_realm_internal_OsResults.cpp line 243
at io.realm.internal.OsResults.nativeStartListening(Native Method)
at io.realm.internal.OsResults.addListener(OsResults.java:2)
at org.matrix.android.sdk.internal.session.room.timeline.TimelineChunk.(TimelineChunk.kt:30)
at org.matrix.android.sdk.internal.session.room.timeline.TimelineChunk.createTimelineChunk(TimelineChunk.kt:14)
at org.matrix.android.sdk.internal.session.room.timeline.TimelineChunk$$ExternalSyntheticLambda1.onChange(R8$$SyntheticClass:27)
at io.realm.internal.OsObject.notifyChangeListeners(OsObject.java:10)
at io.realm.internal.OsSharedRealm.nativeBeginTransaction(Native Method)
at io.realm.internal.OsSharedRealm.beginTransaction(OsSharedRealm.java:3)
at io.realm.BaseRealm.beginTransaction(BaseRealm.java:2)
at io.realm.Realm.executeTransaction(Realm.java:7)
at org.matrix.android.sdk.internal.session.room.timeline.DefaultTimeline$openAround$2.invokeSuspend(DefaultTimeline.kt:59)
at org.matrix.android.sdk.internal.session.room.timeline.DefaultTimeline$openAround$2.invoke(Unknown Source:8)
at org.matrix.android.sdk.internal.session.room.timeline.DefaultTimeline$openAround$2.invoke(Unknown Source:4)
at kotlinx.coroutines.intrinsics.UndispatchedKt.startUndispatchedOrReturn(Undispatched.kt:1)
at kotlinx.coroutines.BuildersKt.withContext(Unknown Source:9)
at org.matrix.android.sdk.internal.session.room.timeline.DefaultTimeline.access$openAround(DefaultTimeline.kt:1)
at org.matrix.android.sdk.internal.session.room.timeline.DefaultTimeline$start$2$1.invokeSuspend(DefaultTimeline.kt:29)
at org.matrix.android.sdk.internal.session.room.timeline.DefaultTimeline$start$2$1.invoke(Unknown Source:8)
at org.matrix.android.sdk.internal.session.room.timeline.DefaultTimeline$start$2$1.invoke(Unknown Source:2)
at org.matrix.android.sdk.internal.task.SemaphoreCoroutineSequencer.post$suspendImpl(CoroutineSequencer.kt:7)
at org.matrix.android.sdk.internal.session.room.timeline.DefaultTimeline$start$2.invokeSuspend(DefaultTimeline.kt:8)
at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:3)
at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:18)
at android.os.Handler.handleCallback(Handler.java:883)
at android.os.Handler.dispatchMessage(Handler.java:100)
at android.os.Looper.loop(Looper.java:237)
at android.os.HandlerThread.run(HandlerThread.java:67)
@ganfra ganfra added A-Timeline A-Threads An issue for the Threads project T-Defect Something isn't working: bugs, crashes, hangs and other reported problems S-Major Severely degrades major functionality or product features, with no satisfactory workaround O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience labels Jul 5, 2022
@ganfra ganfra added the Z-Crash label Jul 5, 2022
ganfra pushed a commit that referenced this issue Jul 5, 2022
ouchadam added a commit that referenced this issue Jul 7, 2022
Florian14 pushed a commit that referenced this issue Jul 11, 2022
* develop: (156 commits)
  adding test case for showing html entities are processed
  adding tests around the event html rendering - the test helper is a little hacky in order to covert the spans to something human readable
  removing extra line
  adding changelog entry
  overriding the default list handler with an implementation that takes into account the initial starting position
  trigger CI
  Use executeTransactionAwait (need realm refresh in this case)
  Bump flipper from 0.152.0 to 0.153.0
  Use executeTransactionAwait (need realm refresh in this case)
  generating 1.4.27 changelog and updating version
  Fixing crash when sharing plain text, such as a url
  Fix crashes when opening Thread (#6463)
  Timeline: fix validation of timeline event changes
  Fix ConcurrentModificationException on BackgroundDetectionObserver
  Fix crashes when opening Thread (#6463)
  suppressing unused string resource
  Changelog
  Fix ConcurrentModificationException on BackgroundDetectionObserver
  Fix typo
  adding changelog entry
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Threads An issue for the Threads project A-Timeline O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Z-Crash
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant