QPQ-AG/enacl
10
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix a bug in sign_init/0

Browse Source 
The code path was wrong in a lot
of cases in sign_init/0 so even if it
succeeded it would return a failure.

Fixed by cleaning up the code.
This commit is contained in:
Jesper Louis Andersen 2020-02-06 13:57:49 +01:00
parent cceef4530a
commit fd87b9da39
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
c_src/sign.c
View File

@ -56,7 +56,8 @@ ERL_NIF_TERM enacl_crypto_sign_init(ErlNifEnv *env, int argc,
if ((obj = enif_alloc_resource(enacl_sign_ctx_rtype, if ((obj = enif_alloc_resource(enacl_sign_ctx_rtype,
sizeof(enacl_sign_ctx))) == NULL) { sizeof(enacl_sign_ctx))) == NULL) {
goto err; ret = enacl_internal_error(env);
goto done;
} }
obj->alive = 0; obj->alive = 0;
obj->state = enif_alloc(crypto_sign_statebytes()); obj->state = enif_alloc(crypto_sign_statebytes());
@ -90,8 +91,6 @@ free:
release: release:
// This also frees the mutex via the destructor // This also frees the mutex via the destructor
enif_release_resource(obj); enif_release_resource(obj);
err:
ret = enacl_internal_error(env);
done: done:
return ret; return ret;
} }