It seems that since #310, openssl AEAD backend have been testing if supplied key is NULL and returning success if that is the case, while other backends have continued to assume key is non-NULL and crash if a NULL keys is supplied.
I believe this was an unintentional change is #310 and nobody is passing a NULL key (this call path can executed only once per object and passing a NULL key makes AEAD unusable), and therefore this PR removes that if.
It seems that since #310, openssl AEAD backend have been testing if supplied
key
is NULL and returning success if that is the case, while other backends have continued to assumekey
is non-NULL and crash if a NULL keys is supplied.I believe this was an unintentional change is #310 and nobody is passing a NULL key (this call path can executed only once per object and passing a NULL key makes AEAD unusable), and therefore this PR removes that
if
.