diff options
author | Theo Buehler <tb@cvs.openbsd.org> | 2021-11-06 12:31:41 +0000 |
---|---|---|
committer | Theo Buehler <tb@cvs.openbsd.org> | 2021-11-06 12:31:41 +0000 |
commit | 929f930caf063d97679d8e67142545bd5a406816 (patch) | |
tree | 60ed5b181dc41fb02981cea988a4261e6208bbc0 /usr.bin/ftp/cmds.c | |
parent | 424add47b831f940ee8a7e1ce4a18d0c1be0f307 (diff) |
Start cleaning up X509_STORE_get1_issuer()
Get rid of the last X509_OBJECT_free_contents() call by moving the object
from the stack to the heap. I deliberately kept the obj variable to keep
obj and pobj separate. Rename the out parameter from issuer to out_issuer
to ensure that we only assign it when we have acquired a reference that we
can return. Add a new X509 *issuer. In the first part of the function,
acquire an extra reference before check_issuer/check_time.
In the second part of the function, acquire a reference inside the lock to
avoid a race. Deal with ret only in one place.
ok jsing
Diffstat (limited to 'usr.bin/ftp/cmds.c')
0 files changed, 0 insertions, 0 deletions