vilyaem/Xrasher
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

filesystem: minor refactoring in MD5_HashFile, use initializer instead of memset, limit buffer variable scope

Browse source
This commit is contained in:
Alibek Omarov 2025-01-14 11:36:13 +03:00
parent 4be75af214
commit 785632a437
1 changed files with 3 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
filesystem/filesystem.c
View file

@ -2705,25 +2705,20 @@ qboolean CRC32_File( dword *crcvalue, const char *filename )
qboolean MD5_HashFile( byte digest[16], const char *pszFileName, uint seed[4] ) qboolean MD5_HashFile( byte digest[16], const char *pszFileName, uint seed[4] )
{ {
file_t *file; file_t *file;
byte buffer[1024]; MD5Context_t MD5_Hash = { 0 };
MD5Context_t MD5_Hash;
int bytes;
if(( file = FS_Open( pszFileName, "rb", false )) == NULL ) if(( file = FS_Open( pszFileName, "rb", false )) == NULL )
return false; return false;
memset( &MD5_Hash, 0, sizeof( MD5Context_t ));
MD5Init( &MD5_Hash ); MD5Init( &MD5_Hash );
if( seed ) if( seed )
{
MD5Update( &MD5_Hash, (const byte *)seed, 16 ); MD5Update( &MD5_Hash, (const byte *)seed, 16 );
}
while( 1 ) while( 1 )
{ {
bytes = FS_Read( file, buffer, sizeof( buffer )); byte buffer[1024];
int bytes = FS_Read( file, buffer, sizeof( buffer ));
if( bytes > 0 ) if( bytes > 0 )
MD5Update( &MD5_Hash, buffer, bytes ); MD5Update( &MD5_Hash, buffer, bytes );