Skip to content

Commit 5551ccb

Browse files
committed
provider: fix potential memory leak in Provider.load
Allocate the wrapper object before loading an OpenSSL provider.
1 parent a27a0d0 commit 5551ccb

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

ext/openssl/ossl_provider.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,11 +57,11 @@ ossl_provider_s_load(VALUE klass, VALUE name)
5757

5858
const char *provider_name_ptr = StringValueCStr(name);
5959

60+
obj = NewProvider(klass);
6061
provider = OSSL_PROVIDER_load(NULL, provider_name_ptr);
6162
if (provider == NULL) {
6263
ossl_raise(eProviderError, "Failed to load %s provider", provider_name_ptr);
6364
}
64-
obj = NewProvider(klass);
6565
SetProvider(obj, provider);
6666

6767
return obj;

0 commit comments

Comments
 (0)