Re: [PATCH] lib/crypto: use rol32 in md5.c

From: Eric Biggers

Date: Thu Dec 18 2025 - 14:33:21 EST


On Sun, Dec 14, 2025 at 06:15:12PM +0700, Rusydi H. Makarim wrote:
> use rol32 in MD5STEP
>
> ---
> this patch replaces the bitwise left rotation in lib/crypto/md5.c
> with rol32
>
> Signed-off-by: Rusydi H. Makarim <rusydi.makarim@xxxxxxxxxxxxx>
> ---
> lib/crypto/md5.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)

Applied to https://git.kernel.org/pub/scm/linux/kernel/git/ebiggers/linux.git/log/?h=libcrypto-next

I cleaned up your commit message as follows:

commit 325c29e7d11caaf3b4f04f2c8f7d6bc4861cce5a
Author: Rusydi H. Makarim <rusydi.makarim@xxxxxxxxxxxxx>
Date: Sun Dec 14 18:15:12 2025 +0700

lib/crypto: md5: Use rol32() instead of open-coding it

For the bitwise left rotation in MD5STEP, use rol32() from
<linux/bitops.h> instead of open-coding it.

Signed-off-by: Rusydi H. Makarim <rusydi.makarim@xxxxxxxxxxxxx>
Link: https://lore.kernel.org/r/20251214-rol32_in_md5-v1-1-20f5f11a92b2@xxxxxxxxxxxxx
Signed-off-by: Eric Biggers <ebiggers@xxxxxxxxxx>

For future reference: please don't include a scissors line in the middle
of your commit message. Everything below it gets cut off.

- Eric