From fc423fad15ee3daaf236f211d67a25d03f63b560 Mon Sep 17 00:00:00 2001 From: Hubert Chathi Date: Tue, 8 Oct 2019 17:44:09 -0400 Subject: check return value of snprintf, fix typo, add clarification --- src/session.cpp | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-) (limited to 'src/session.cpp') diff --git a/src/session.cpp b/src/session.cpp index d6761d5..b619e56 100644 --- a/src/session.cpp +++ b/src/session.cpp @@ -404,25 +404,32 @@ void olm::Session::describe(char *describe_buffer, size_t buflen) { describe_buffer[0] = '\0'; char *buf_pos = describe_buffer; - buf_pos += snprintf( + int size; + + size = snprintf( buf_pos, buflen - (buf_pos - describe_buffer), "sender chain index: %d ", ratchet.sender_chain[0].chain_key.index ); + if (size > 0) buf_pos += size; - buf_pos += snprintf(buf_pos, buflen - (buf_pos - describe_buffer), "receiver chain indcies:"); + size = snprintf(buf_pos, buflen - (buf_pos - describe_buffer), "receiver chain indices:"); + if (size > 0) buf_pos += size; for (size_t i = 0; i < ratchet.receiver_chains.size(); ++i) { - buf_pos += snprintf( + size = snprintf( buf_pos, buflen - (buf_pos - describe_buffer), " %d", ratchet.receiver_chains[i].chain_key.index ); + if (size > 0) buf_pos += size; } - buf_pos += snprintf(buf_pos, buflen - (buf_pos - describe_buffer), " skipped message keys:"); + size = snprintf(buf_pos, buflen - (buf_pos - describe_buffer), " skipped message keys:"); + if (size >= 0) buf_pos += size; for (size_t i = 0; i < ratchet.skipped_message_keys.size(); ++i) { - buf_pos += snprintf( + size = snprintf( buf_pos, buflen - (buf_pos - describe_buffer), " %d", ratchet.skipped_message_keys[i].message_key.index ); + if (size > 0) buf_pos += size; } } -- cgit v1.2.3