Commit 73206954 authored by Paul Bakker's avatar Paul Bakker
Browse files

- Made des_key_check_weak() conform to other functions in return values.

 - Added documentation for des_key_check_weak() and des_key_check_key_parity()
parent c43e326d
......@@ -77,14 +77,17 @@ void des_key_set_parity( unsigned char key[DES_KEY_SIZE] );
* a parity bit to allow verification.
*
* \param key 8-byte secret key
*
* \return 0 is parity was ok, 1 if parity was not correct.
*/
int des_key_check_key_parity( const unsigned char key[DES_KEY_SIZE] );
/**
* \brief Check that key is not a weak or semi-weak DES key
*
* \param key 8-byte secret key
*
* \resurn 0 if no weak key was found, 1 if a weak key was identified.
*/
int des_key_check_weak( const unsigned char key[DES_KEY_SIZE] );
......
......@@ -371,9 +371,9 @@ int des_key_check_weak( const unsigned char key[DES_KEY_SIZE] )
for( i = 0; i < WEAK_KEY_COUNT; i++ )
if( memcmp( weak_key_table[i], key, DES_KEY_SIZE) == 0)
return( 0 );
return( 1 );
return( 1 );
return( 0 );
}
static void des_setkey( unsigned long SK[32], const unsigned char key[DES_KEY_SIZE] )
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment