diff options
| author |   Gabriel Oginski <gabrielx.oginski@intel.com> | 2022-01-24 13:31:45 +0000 |
|---|---|---|
| committer | Gabriel Oginski <gabrielx.oginski@intel.com> | 2022-01-24 14:54:07 +0100 |
| commit | ffd9057493ba75d157d8fd316f4978b64a6efb30 (patch) | |
| tree | 9df8aed9fce580ef545115cd62c12d9317c66425 /src/plugins/wireguard | |
| parent | 56c492aa0502751de2dd9d890096a82c5f04776d (diff) | |
wireguard: fix passing argument
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
Diffstat (limited to 'src/plugins/wireguard')
| -rw-r--r-- | src/plugins/wireguard/wireguard_output_tun.c | 2 |
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, |