Skip to content

Commit

Permalink
Merge pull request #15 from MonokelPinguin/mismatch-new-delete
Browse files Browse the repository at this point in the history
Fix mismatch between new[] and delete with Olm objects.  Thanks @MonokelPinguin
  • Loading branch information
redsky17 authored May 1, 2019
2 parents 6f7b35a + 4df37aa commit 370c9c1
Showing 1 changed file with 5 additions and 8 deletions.
13 changes: 5 additions & 8 deletions include/mtxclient/crypto/objects.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -8,18 +8,15 @@ namespace crypto {

struct OlmDeleter
{
void operator()(OlmAccount *ptr) { operator delete(ptr, olm_account_size()); }
void operator()(OlmUtility *ptr) { operator delete(ptr, olm_utility_size()); }
void operator()(OlmAccount *ptr) { delete[](reinterpret_cast<uint8_t *>(ptr)); }
void operator()(OlmUtility *ptr) { delete[](reinterpret_cast<uint8_t *>(ptr)); }

void operator()(OlmSession *ptr) { operator delete(ptr, olm_session_size()); }
void operator()(OlmSession *ptr) { delete[](reinterpret_cast<uint8_t *>(ptr)); }
void operator()(OlmOutboundGroupSession *ptr)
{
operator delete(ptr, olm_outbound_group_session_size());
}
void operator()(OlmInboundGroupSession *ptr)
{
operator delete(ptr, olm_inbound_group_session_size());
delete[](reinterpret_cast<uint8_t *>(ptr));
}
void operator()(OlmInboundGroupSession *ptr) { delete[](reinterpret_cast<uint8_t *>(ptr)); }
};

struct UtilityObject
Expand Down

0 comments on commit 370c9c1

Please sign in to comment.