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.

Sign up for GitHub

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

Jump to bottom

Crashes on Timeline [Thread] due to range validation #6461

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

Crashes on Timeline [Thread] due to range validation #6461

ganfra opened this issue Jul 5, 2022 · 0 comments · Fixed by #6462
Assignees
@ganfra
Labels
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

Steps to reproduce

  1. Open a Thread
  2. Put in airplane mode to have a gap when coming back online
  3. Send 11 messages from other session
  4. Get out of airplane
  5. Crashes

Outcome

What did you expect?

Doesn't crash

What happened instead?

Crashes

Your phone model

No response

Operating system version

No response

Application version and app store

No response

Homeserver

No response

Will you send logs?

No

Are you willing to provide a PR?

Yes
@ganfra ganfra added T-Defect Something isn't working: bugs, crashes, hangs and other reported problems A-Timeline S-Major Severely degrades major functionality or product features, with no satisfactory workaround labels Jul 5, 2022
@ganfra ganfra added the O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience label Jul 5, 2022
@ganfra ganfra mentioned this issue Jul 5, 2022
Merged
@bmarty bmarty added the Z-Crash label Jul 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ganfra ganfra

Labels
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
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Timeline: fix validation of timeline event changes element-hq/element-android
2 participants
@bmarty @ganfra