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

Fix /omemo fingerprint crashing in a MUC #1913

Merged
merged 1 commit into from
Nov 6, 2023
Merged

Conversation

sjaeckel
Copy link
Member

@sjaeckel sjaeckel commented Nov 6, 2023

Reported by a user in the profanity MUC.

Fixup of e1d137f resp. #1863

Reported by a user in the profanity MUC.

Fixup of e1d137f resp. #1863

Signed-off-by: Steffen Jaeckel <[email protected]>
@sjaeckel sjaeckel force-pushed the fix-omemo-fingerprint-crash branch from 0b2b276 to 0449a42 Compare November 6, 2023 18:29
@jubalh
Copy link
Member

jubalh commented Nov 6, 2023

Oh so the pointer was pointing to a random location and the auto_ tried to free it. Interesting situation. Will have to keep that in mind.

@jubalh jubalh added this to the next milestone Nov 6, 2023
@jubalh jubalh self-requested a review November 6, 2023 18:34
@jubalh jubalh merged commit 716c068 into master Nov 6, 2023
6 checks passed
@jubalh jubalh deleted the fix-omemo-fingerprint-crash branch November 6, 2023 18:39
sjaeckel added a commit that referenced this pull request Nov 6, 2023
Follow-up of #1913

Signed-off-by: Steffen Jaeckel <[email protected]>
sjaeckel added a commit that referenced this pull request Nov 6, 2023
Follow-up of #1913

Signed-off-by: Steffen Jaeckel <[email protected]>
sjaeckel added a commit that referenced this pull request Nov 6, 2023
Follow-up of #1913

Signed-off-by: Steffen Jaeckel <[email protected]>
sjaeckel added a commit that referenced this pull request Nov 6, 2023
Follow-up of #1913

Signed-off-by: Steffen Jaeckel <[email protected]>
sjaeckel added a commit that referenced this pull request Nov 6, 2023
Follow-up of #1913

Signed-off-by: Steffen Jaeckel <[email protected]>
H3rnand3zzz pushed a commit to H3rnand3zzz/profanity that referenced this pull request Nov 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants