Merge pull request #162 from djw8605/fix-memory-dealloc

djw8605 · web-flow · commit ff132cf05bd3 · 2025-12-08T13:37:16.000-06:00
Revert "Fix memory leak in rs256_from_coords"
diff --git a/src/scitokens_internal.cpp b/src/scitokens_internal.cpp
@@ -532,6 +532,8 @@ std::string rs256_from_coords(const std::string &e_str,
         throw UnsupportedKeyException("Failed to serialize RSA public key");
     }
 #endif
+    e_bignum.release();
+    n_bignum.release();
 
     char *mem_data;
     size_t mem_len = BIO_get_mem_data(pubkey_bio.get(), &mem_data);

Original file line number	Diff line number	Diff line change
`@@ -532,6 +532,8 @@ std::string rs256_from_coords(const std::string &e_str,`
`532`	`532`	`throw UnsupportedKeyException("Failed to serialize RSA public key");`
`533`	`533`	`}`
`534`	`534`	`#endif`
	`535`	`+ e_bignum.release();`
	`536`	`+ n_bignum.release();`
`535`	`537`
`536`	`538`	`char *mem_data;`
`537`	`539`	`size_t mem_len = BIO_get_mem_data(pubkey_bio.get(), &mem_data);`