summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGabriel Oginski <gabrielx.oginski@intel.com>2022-01-24 13:31:45 +0000
committerGabriel Oginski <gabrielx.oginski@intel.com>2022-01-25 11:36:28 +0000
commit9d2db2eb2ed212d205310142ab49b1f883d513ce (patch)
tree0ec8c6f34bfc30009e2bd9416acb9baf9e849a44
parente3d36d18c1ae30438fa2aa91aee4f14c43688a6e (diff)
wireguard: fix passing argumentv22.02-rc2
Fixed coverity-issue CID 248456. Originally passing argument of type "uint64_t *" to function: "memcopy_s_inline". This patch fixes the problem by changing type of passing argument and make a portable assumption. Type: fix Signed-off-by: Gabriel Oginski <gabrielx.oginski@intel.com> Change-Id: I17e4583a05ea1263e4d8a4acc9949454e5fd92c0 (cherry picked from commit ffd9057493ba75d157d8fd316f4978b64a6efb30)
-rw-r--r--src/plugins/wireguard/wireguard_output_tun.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/plugins/wireguard/wireguard_output_tun.c b/src/plugins/wireguard/wireguard_output_tun.c
index f7ae4d174c0..14df692eebc 100644
--- a/src/plugins/wireguard/wireguard_output_tun.c
+++ b/src/plugins/wireguard/wireguard_output_tun.c
@@ -282,7 +282,7 @@ wg_add_to_async_frame (vlib_main_t *vm, wg_per_thread_data_t *ptd,
*r_idx = kp->kp_remote_index;
clib_memset (iv, 0, 4);
- clib_memcpy (iv + 4, nonce, sizeof (nonce));
+ clib_memcpy (iv + 4, nonce, sizeof (*nonce));
/* this always succeeds because we know the frame is not full */
wg_output_tun_add_to_frame (vm, async_frame, kp->kp_send_index, payload_len,