Skip to content

fix: lib sodium crash on logiin intent [WPB-24366] [WPB-24440]#4684

Merged
MohamadJaara merged 2 commits intodevelopfrom
mo/fix/lib-sodium-crash-on-login-intent
Mar 31, 2026
Merged

fix: lib sodium crash on logiin intent [WPB-24366] [WPB-24440]#4684
MohamadJaara merged 2 commits intodevelopfrom
mo/fix/lib-sodium-crash-on-login-intent

Conversation

@MohamadJaara
Copy link
Copy Markdown
Member

@MohamadJaara MohamadJaara commented Mar 30, 2026

https://wearezeta.atlassian.net/browse/WPB-24366


PR Submission Checklist for internal contributors

  • The PR Title

    • conforms to the style of semantic commits messages¹ supported in Wire's Github Workflow²
    • contains a reference JIRA issue number like SQPIT-764
    • answers the question: If merged, this PR will: ... ³
  • The PR Description

    • is free of optional paragraphs and you have filled the relevant parts to the best of your ability

What's new in this PR?

Issues

lib sodium randomly crashes during init

Causes (Optional)

the crash is when the init return code 1 (aka already initialized).

Solutions

instead of using the kotlin bindings directly use kalium :core:libsodium module

@MohamadJaara MohamadJaara requested a review from Garzas March 30, 2026 21:31
@MohamadJaara MohamadJaara changed the title fix: lib sodium crash on logiin intent fix: lib sodium crash on logiin intent [WPB-24366] Mar 30, 2026
@MohamadJaara MohamadJaara changed the title fix: lib sodium crash on logiin intent [WPB-24366] fix: lib sodium crash on logiin intent [WPB-24366] [WPB-24440] Mar 30, 2026
@MohamadJaara MohamadJaara force-pushed the mo/fix/lib-sodium-crash-on-login-intent branch from ecad0d6 to f70e95e Compare March 30, 2026 21:44
@sonarqubecloud
Copy link
Copy Markdown

@MohamadJaara MohamadJaara added this pull request to the merge queue Mar 31, 2026
Merged via the queue into develop with commit 887f3ac Mar 31, 2026
16 checks passed
@MohamadJaara MohamadJaara deleted the mo/fix/lib-sodium-crash-on-login-intent branch March 31, 2026 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants