timoreo/olm
1
1
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Use _olm_unset instead of memset

Browse source 
memset is at risk of being optimised away, so use _olm_unset instead.
This commit is contained in:
Richard van der Hoff 2016-05-20 12:35:59 +01:00
parent 173cbe112c
commit fa1e9446ac
2 changed files with 6 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/inbound_group_session.c
View file

@ -21,6 +21,7 @@
#include "olm/cipher.h"
#include "olm/error.h"
#include "olm/megolm.h"
#include "olm/memory.h"
#include "olm/message.h"
#include "olm/pickle.h"
#include "olm/pickle_encoding.h"
@ -60,7 +61,7 @@ const char *olm_inbound_group_session_last_error(
size_t olm_clear_inbound_group_session(
OlmInboundGroupSession *session
) {
memset(session, 0, sizeof(OlmInboundGroupSession));
_olm_unset(session, sizeof(OlmInboundGroupSession));
return sizeof(OlmInboundGroupSession);
}
@ -85,7 +86,7 @@ size_t olm_init_inbound_group_session(
_olm_decode_base64(session_key, session_key_length, key_buf);
megolm_init(&session->initial_ratchet, key_buf, message_index);
megolm_init(&session->latest_ratchet, key_buf, message_index);
memset(key_buf, 0, MEGOLM_RATCHET_LENGTH);
_olm_unset(key_buf, MEGOLM_RATCHET_LENGTH);
return 0;
}
@ -265,7 +266,7 @@ size_t olm_group_decrypt(
plaintext, max_plaintext_length
);
memset(&tmp_megolm, 0, sizeof(tmp_megolm));
_olm_unset(&tmp_megolm, sizeof(tmp_megolm));
if (r == (size_t)-1) {
session->last_error = OLM_BAD_MESSAGE_MAC;
return r;

3
src/outbound_group_session.c
View file

@ -22,6 +22,7 @@
#include "olm/cipher.h"
#include "olm/error.h"
#include "olm/megolm.h"
#include "olm/memory.h"
#include "olm/message.h"
#include "olm/pickle.h"
#include "olm/pickle_encoding.h"
@ -63,7 +64,7 @@ const char *olm_outbound_group_session_last_error(
size_t olm_clear_outbound_group_session(
OlmOutboundGroupSession *session
) {
memset(session, 0, sizeof(OlmOutboundGroupSession));
_olm_unset(session, sizeof(OlmOutboundGroupSession));
return sizeof(OlmOutboundGroupSession);
}