Skip to content

Commit 28804da

Browse files
authored
Merge pull request #77 from ADTRAN/two-tls-fixes
Two small TLS bug fixes
2 parents bd53d50 + 62bc671 commit 28804da

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

src/session_server_tls.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -513,7 +513,7 @@ nc_tlsclb_verify(int preverify_ok, X509_STORE_CTX *x509_ctx)
513513
/* get the last certificate, that is the peer (client) certificate */
514514
if (!session->opts.server.client_cert) {
515515
cert_stack = X509_STORE_CTX_get1_chain(x509_ctx);
516-
session->opts.server.client_cert = sk_X509_value(cert_stack, sk_X509_num(cert_stack) - 1);
516+
session->opts.server.client_cert = sk_X509_value(cert_stack, 0);
517517
X509_up_ref(session->opts.server.client_cert);
518518
sk_X509_pop_free(cert_stack, X509_free);
519519
}
@@ -1406,7 +1406,7 @@ nc_server_tls_add_ctn(uint32_t id, const char *fingerprint, NC_TLS_CTN_MAPTYPE m
14061406
new->next = opts->ctn;
14071407
opts->ctn = new;
14081408
} else {
1409-
for (ctn = opts->ctn; ctn->next && ctn->next->id < id; ctn = ctn->next);
1409+
for (ctn = opts->ctn; ctn->next && ctn->next->id <= id; ctn = ctn->next);
14101410
if (ctn->id == id) {
14111411
/* it exists already */
14121412
new = ctn;

0 commit comments

Comments
 (0)