stir_shaken: Fix memory leak, typo in config, tn canonicalization

asterisk / asterisk

The official Asterisk Project repository.

https://www.asterisk.org

Other

1.97k stars 924 forks source link

stir_shaken: Fix memory leak, typo in config, tn canonicalization #717

Closed gtjoseph closed 2 weeks ago

gtjoseph commented 3 weeks ago

Fixed possible memory leak in tn_config:tn_get_etn() where we weren't releasing etn if tn or eprofile were null.
We now canonicalize TNs before using them for lookups or adding them to Identity headers.
Fixed a typo in stir_shaken.conf.sample.

Resolves: #716

github-actions[bot] commented 3 weeks ago

REMINDER: If this PR applies to other branches, please add a comment with the appropriate "cherry-pick-to" headers as per the Create a Pull Request process.

If you don't want it cherry-picked, please add a comment with cherry-pick-to: none so we don't keep asking.

If, after adding "cherry-pick-to" comments, you change your mind, please edit the comment to DELETE the header lines and add cherry-pick-to: none.

The currently active branches are now 18, 20, 21 and master.

gtjoseph commented 3 weeks ago

cherry-pick-to: 18 cherry-pick-to: 20 cherry-pick-to: 21

github-actions[bot] commented 2 weeks ago

Successfully merged to branch master and cherry-picked to ["18","20","21"]