[njs] Removed dead code introduced in f70929728e86.

Dmitry Volyntsev xeioex at nginx.com
Tue Aug 30 04:13:34 UTC 2022


details:   https://hg.nginx.org/njs/rev/907ec4824fa0
branches:  
changeset: 1938:907ec4824fa0
user:      Dmitry Volyntsev <xeioex at nginx.com>
date:      Mon Aug 29 21:09:11 2022 -0700
description:
Removed dead code introduced in f70929728e86.

Found by Coverity (CID 1512554, 1512556).

diffstat:

 external/njs_webcrypto_module.c |  11 -----------
 1 files changed, 0 insertions(+), 11 deletions(-)

diffs (28 lines):

diff -r b48963b98d97 -r 907ec4824fa0 external/njs_webcrypto_module.c
--- a/external/njs_webcrypto_module.c	Mon Aug 29 21:08:47 2022 -0700
+++ b/external/njs_webcrypto_module.c	Mon Aug 29 21:09:11 2022 -0700
@@ -2112,8 +2112,6 @@ njs_convert_p1363_to_der(njs_vm_t *vm, E
 
     ret = NJS_OK;
     ec_sig = NULL;
-    r = NULL;
-    s = NULL;
 
     n = njs_ec_rs_size(pkey);
 
@@ -2174,15 +2172,6 @@ done:
 
     if (ec_sig != NULL) {
         ECDSA_SIG_free(ec_sig);
-
-    } else {
-        if (s != NULL) {
-            BN_free(s);
-        }
-
-        if (r != NULL) {
-            BN_free(r);
-        }
     }
 
     return ret;



More information about the nginx-devel mailing list