nodejs
diff --git a/Collapse file
‎src/crypto/crypto_common.cc‎
Copy file name to clipboardExpand all lines: src/crypto/crypto_common.cc
+3-4Lines changed: 3 additions & 4 deletions b/Collapse file
‎src/crypto/crypto_common.cc‎
Copy file name to clipboardExpand all lines: src/crypto/crypto_common.cc
+3-4Lines changed: 3 additions & 4 deletions
@@ -992,18 +992,17 @@ static MaybeLocal<Value> GetX509NameObject(Environment* env, X509* cert) {
     if (value_str_size < 0) {
       return Undefined(env->isolate());
     }
+    auto free_value_str = OnScopeLeave([&]() { OPENSSL_free(value_str); });
 
     Local<String> v8_value;
     if (!String::NewFromUtf8(env->isolate(),
                              reinterpret_cast<const char*>(value_str),
                              NewStringType::kNormal,
-                             value_str_size).ToLocal(&v8_value)) {
-      OPENSSL_free(value_str);
+                             value_str_size)
+             .ToLocal(&v8_value)) {
       return MaybeLocal<Value>();
     }
 
-    OPENSSL_free(value_str);
-
     // For backward compatibility, we only create arrays if multiple values
     // exist for the same key. That is not great but there is not much we can
     // change here without breaking things. Note that this creates nested data