[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)
	Thu, Dec 26, 12:32 PM

	Unknown Object (File)
	Dec 10 2024, 2:36 PM

	Unknown Object (File)
	Oct 3 2024, 11:51 PM

	Unknown Object (File)
	Oct 2 2024, 9:51 AM

	Unknown Object (File)
	Sep 24 2024, 4:52 AM

	Unknown Object (File)
	Sep 19 2024, 11:11 PM

	Unknown Object (File)
	Sep 15 2024, 4:05 PM

	Unknown Object (File)
	Sep 14 2024, 7:02 PM

View All 36 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