Skip to content

Commit

Permalink
tls_mgm: fix crash when the client_sip_domain_avp AVP is not set
Browse files Browse the repository at this point in the history
This is a regression introduced by commit e94ca5f that would lead to
a crash when the client_sip_domain_avp modparam is set but the actual
AVP is not set before a TLS connect.

Credits to @ovidiusas for reporting and suggesting the fix.

Closes #2909

(cherry picked from commit 3c05ae7)
  • Loading branch information
rvlad-patrascu committed Sep 30, 2022
1 parent 8b2abc1 commit fe3ae05
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions modules/tls_mgm/tls_domain.c
Original file line number Diff line number Diff line change
Expand Up @@ -475,6 +475,8 @@ struct tls_domain *tls_find_client_domain(struct ip_addr *ip, unsigned short por
if (sip_domain) {
LM_DBG("Match TLS domain by sip domain AVP: '%.*s'\n",
sip_domain->len, ZSW(sip_domain->s));
} else {
sip_domain = &match_any_dom;
}
}
} else {
Expand Down

0 comments on commit fe3ae05

Please sign in to comment.