From ad596a487ff7c45ef6f4f0a0f6b5314e5c0fc942 Mon Sep 17 00:00:00 2001 From: Joachim Bauch Date: Fri, 22 Dec 2017 00:22:39 +0100 Subject: [PATCH] Remove unnecessary "memset". The whole memory block is filled with "memcpy" afterwards. Bug: None Change-Id: Ieb7fe2513184a6abbf6c61f3582590f4bcd16f1f Reviewed-on: https://webrtc-review.googlesource.com/35980 Reviewed-by: Taylor Brandstetter Commit-Queue: Joachim Bauch Cr-Commit-Position: refs/heads/master@{#21420} --- pc/externalhmac.cc | 1 - 1 file changed, 1 deletion(-) diff --git a/pc/externalhmac.cc b/pc/externalhmac.cc index 0e82fffc28..899c33822c 100644 --- a/pc/externalhmac.cc +++ b/pc/externalhmac.cc @@ -111,7 +111,6 @@ srtp_err_status_t external_hmac_init(void* state, return srtp_err_status_bad_param; ExternalHmacContext* context = static_cast(state); - memset(context->key, 0, key_len); memcpy(context->key, key, key_len); context->key_length = key_len; return srtp_err_status_ok;