[crypto][NFC] Fix compiler warnings about variable types in libcrypt
Needs ReviewPublic
Actions

Authored by gfunni234_gmail.com on Aug 22 2021, 6:51 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Aug 3 2025, 1:40 PM

	Unknown Object (File)
	Jul 1 2025, 3:40 AM

	Unknown Object (File)
	Jun 26 2025, 8:00 PM

	Unknown Object (File)
	Jun 17 2025, 8:30 AM

	Unknown Object (File)
	Jun 16 2025, 8:48 PM

	Unknown Object (File)
	Jun 16 2025, 8:43 AM

	Unknown Object (File)
	Jun 15 2025, 12:42 PM

	Unknown Object (File)
	Jun 10 2025, 10:34 AM

View All 49 Files

Subscribers

None

Details

Reviewers

imp
jrtc27
dim

Summary

Instead of casting from unsigned and signed and vice versa, we should be consistent with signedness.

NFC this if possible please!

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

gfunni234_gmail.com requested review of this revision.Aug 22 2021, 6:51 PM

gfunni234_gmail.com created this revision.

gfunni234_gmail.com added reviewers: jrtc27, dim.Aug 24 2021, 3:53 PM

gfunni234_gmail.com updated this revision to Diff 94118.Aug 24 2021, 5:46 PM

gfunni234_gmail.com retitled this revision from [crypto][NFC]Fix compiler warnings about variable types in libcrypt to [crypto][NFC] Fix compiler warnings about variable types in libcrypt.

gfunni234_gmail.com edited the summary of this revision. (Show Details)

Most of these changes have nothing to do with what your summary says

lib/libcrypt/crypt-md5.c
46	So now we have a magic constant rather than being automatically calculated by strlen (which, yes, compilers know about and will constant evaluate)?
lib/libcrypt/crypt-sha256.c
186	This regresses code style