From fade378feec178ec8caabc0db1cefc2e81b19804 Mon Sep 17 00:00:00 2001 From: Michael Vetter Date: Tue, 23 Jul 2019 07:44:41 +0200 Subject: Dont use not needed strdup() in save_identity() I don't see a reason for strdupping here. --- src/omemo/store.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/omemo/store.c') diff --git a/src/omemo/store.c b/src/omemo/store.c index 2efcae4c..fe7cf771 100644 --- a/src/omemo/store.c +++ b/src/omemo/store.c @@ -378,7 +378,7 @@ save_identity(const signal_protocol_address *address, uint8_t *key_data, signal_buffer *buffer = signal_buffer_create(key_data, key_len); - GHashTable *trusted = g_hash_table_lookup(identity_key_store->trusted, strdup(address->name)); + GHashTable *trusted = g_hash_table_lookup(identity_key_store->trusted, address->name); if (!trusted) { trusted = g_hash_table_new_full(g_direct_hash, g_direct_equal, NULL, (GDestroyNotify)signal_buffer_free); g_hash_table_insert(identity_key_store->trusted, strdup(address->name), trusted); -- cgit 1.4.1-2-gfad0 From 1b2e71faac49a946ee713133b6df6c5fa2f252b5 Mon Sep 17 00:00:00 2001 From: Michael Vetter Date: Tue, 23 Jul 2019 08:33:29 +0200 Subject: Fix memleak in save_identity() I don't see a reason for strdupping here. --- src/omemo/store.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/omemo/store.c') diff --git a/src/omemo/store.c b/src/omemo/store.c index fe7cf771..2aabd4d5 100644 --- a/src/omemo/store.c +++ b/src/omemo/store.c @@ -388,7 +388,7 @@ save_identity(const signal_protocol_address *address, uint8_t *key_data, /* Long term storage */ char *key_b64 = g_base64_encode(key_data, key_len); char *device_id = g_strdup_printf("%d", address->device_id); - g_key_file_set_string(omemo_trust_keyfile(), address->name, strdup(device_id), key_b64); + g_key_file_set_string(omemo_trust_keyfile(), address->name, device_id, key_b64); g_free(device_id); g_free(key_b64); -- cgit 1.4.1-2-gfad0