correct size for array argument when changing UMAC_OUTPUT_LEN
authordjm <djm@openbsd.org>
Tue, 7 Mar 2023 01:30:52 +0000 (01:30 +0000)
committerdjm <djm@openbsd.org>
Tue, 7 Mar 2023 01:30:52 +0000 (01:30 +0000)
Coverity CID 291845; ok dtucker@

usr.bin/ssh/umac.c

index 707b2a2..1ea6656 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: umac.c,v 1.22 2022/01/01 05:55:06 jsg Exp $ */
+/* $OpenBSD: umac.c,v 1.23 2023/03/07 01:30:52 djm Exp $ */
 /* -----------------------------------------------------------------------
  *
  * umac.c -- C Implementation UMAC Message Authentication
@@ -229,7 +229,8 @@ static void pdf_init(pdf_ctx *pc, aes_int_key prf_key)
     explicit_bzero(buf, sizeof(buf));
 }
 
-static void pdf_gen_xor(pdf_ctx *pc, const UINT8 nonce[8], UINT8 buf[8])
+static void pdf_gen_xor(pdf_ctx *pc, const UINT8 nonce[8],
+    UINT8 buf[UMAC_OUTPUT_LEN])
 {
     /* 'ndx' indicates that we'll be using the 0th or 1st eight bytes
      * of the AES output. If last time around we returned the ndx-1st