SHA3_squeeze(): The next argument is int

Amend the assembler so it uses only 32bit value. Reviewed-by: Matt Caswell <matt@openssl.org> Reviewed-by: Shane Lontis <shane.lontis@oracle.com> Reviewed-by: Hugo Landau <hlandau@openssl.org> (Merged from #22750)
openssl · Nov 23, 2023 · f58d39f · f58d39f
1 parent 69d4d52
commit f58d39f
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/crypto/sha/asm/keccak1600-armv8.pl b/crypto/sha/asm/keccak1600-armv8.pl
@@ -483,7 +483,7 @@
 	mov	$out,x1
 	mov	$len,x2
 	mov	$bsz,x3
-	cmp	x4, #0				// x4 = 'next' argument
+	cmp	w4, #0				// w4 = 'next' argument
 	bne	.Lnext_block
 
 .Loop_squeeze:

diff --git a/crypto/sha/asm/keccak1600-ppc64.pl b/crypto/sha/asm/keccak1600-ppc64.pl
@@ -668,7 +668,7 @@
 	subi	$out,r4,1		; prepare for stbu
 	mr	$len,r5
 	mr	$bsz,r6
-	${UCMP}i r7,0                   ; r7 = 'next' argument
+	cmplwi	r7,0                    ; r7 = 'next' argument
 	bne	.Lnext_block
 	b	.Loop_squeeze
 

diff --git a/crypto/sha/asm/keccak1600-x86_64.pl b/crypto/sha/asm/keccak1600-x86_64.pl
@@ -524,7 +524,7 @@
 	mov	%rsi,$out
 	mov	%rdx,$len
 	mov	%rcx,$bsz
-	bt	\$0,$next
+	bt	\$0,${next}d
 	jc	.Lnext_block
 	jmp	.Loop_squeeze