Page MenuHomePhabricator

[keyserver] call `free()` on Olm objects in finally block
ClosedPublic

Authored by kamil on Wed, Mar 19, 10:03 AM.
Tags
None
Referenced Files
F5292972: D14464.id47506.diff
Mon, Apr 7, 7:06 PM
Unknown Object (File)
Sun, Apr 6, 3:32 PM
Unknown Object (File)
Sun, Apr 6, 6:48 AM
Unknown Object (File)
Sun, Apr 6, 4:19 AM
Unknown Object (File)
Sun, Apr 6, 4:09 AM
Unknown Object (File)
Thu, Apr 3, 5:03 AM
Unknown Object (File)
Wed, Apr 2, 10:32 PM
Unknown Object (File)
Wed, Apr 2, 10:32 PM
Subscribers

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil held this revision as a draft.
ashoat published this revision for review.Wed, Mar 19, 10:06 AM
ashoat added inline comments.
keyserver/src/utils/olm-objects.js
30 ↗(On Diff #47474)

Don’t need to declare here

32 ↗(On Diff #47474)

Can be const result here

80 ↗(On Diff #47474)

Same here

This revision is now accepted and ready to land.Wed, Mar 19, 10:06 AM