Page MenuHomePhabricator
Differential D11356

[native] improve error handling in parseOLMPrekeys
ClosedPublic
Actions

Authored by varun on Mar 19 2024, 11:53 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 24, 5:06 AM
Unknown Object (File)
Tue, Dec 24, 5:06 AM
Unknown Object (File)
Tue, Dec 24, 5:06 AM
Unknown Object (File)
Tue, Dec 24, 5:05 AM
Unknown Object (File)
Thu, Dec 5, 7:42 AM
Unknown Object (File)
Nov 25 2024, 12:27 PM
Unknown Object (File)
Nov 25 2024, 7:17 AM
Unknown Object (File)
Nov 12 2024, 3:06 PM
View All 59 Files
Subscribers
ashoat
tomek

Details

Reviewers
marcin
Commits
rCOMM528670e7c977: [native] improve error handling in parseOLMPrekeys
Summary

addressing feedback from D11312

More info here: https://linear.app/comm/issue/ENG-7280/add-checks-to-parseolmprekey

Test Plan

tested both validateAndGet and validateAndUpload and confirmed both still work as expected. Also passed invalid prekey blobs to parseOLMPrekey to trigger the various exceptions there.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
native/
cpp/
CommonCpp/
NativeModules/
64 lines

Diff 38241

View Options

native/cpp/CommonCpp/NativeModules/CommCoreModule.cpp

Loading...