From 03532ec40252493bb846fa6ffea1af28af79116b Mon Sep 17 00:00:00 2001 From: Rob Crittenden Date: Thu, 27 Aug 2015 13:04:44 -0400 Subject: Switch set_cipher_value() to void since there is no return value --- nss_engine_cipher.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/nss_engine_cipher.c b/nss_engine_cipher.c index 37ef338..012e269 100644 --- a/nss_engine_cipher.c +++ b/nss_engine_cipher.c @@ -142,7 +142,7 @@ int nss_parse_ciphers(server_rec *s, char *ciphers, PRBool cipher_list[ciphernum * This is needed because the + action doesn't do anything in the NSS * context. In OpenSSL it will re-order the cipher list. */ -static int set_cipher_value(PRBool cipher_list[ciphernum], int index, int action) +static void set_cipher_value(PRBool cipher_list[ciphernum], int index, int action) { int i; @@ -153,9 +153,7 @@ static int set_cipher_value(PRBool cipher_list[ciphernum], int index, int action } } - if (cipher_list[index] == -1) /* cipher is disabled */ - return; - else + if (cipher_list[index] != -1) /* cipher is disabled */ cipher_list[index] = action; } -- cgit