diff options
| author | Joshua Colp <jcolp@digium.com> | 2015-05-21 05:15:29 -0500 |
|---|---|---|
| committer | Gerrit Code Review <gerrit2@gerrit.digium.api> | 2015-05-21 05:15:29 -0500 |
| commit | 3c98544543a2f42e40e65f7ffee7ec077a25c210 (patch) | |
| tree | 69aed4203e4f0e5722af74c46512fc711a4f364a | |
| parent | 7fcf0a97b897c958de83ee49fe403c52c8a5d3ed (diff) | |
| parent | 17d6ede337439fe84e4f9efa7b65eb4ceaace0c1 (diff) | |
Merge "main/sdp_srtp.c: allow SDP crypto tag to be up to 9 digits" into 13
| -rw-r--r-- | main/sdp_srtp.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/main/sdp_srtp.c b/main/sdp_srtp.c index fee3fea56..2c49fd240 100644 --- a/main/sdp_srtp.c +++ b/main/sdp_srtp.c @@ -238,7 +238,8 @@ int ast_sdp_crypto_process(struct ast_rtp_instance *rtp, struct ast_sdp_srtp *sr return -1; } - if (sscanf(tag, "%30d", &crypto->tag) != 1 || crypto->tag <= 0 || crypto->tag > 9) { + /* RFC4568 9.1 - tag is 1-9 digits, greater than zero */ + if (sscanf(tag, "%30d", &crypto->tag) != 1 || crypto->tag <= 0 || crypto->tag > 999999999) { ast_log(LOG_WARNING, "Unacceptable a=crypto tag: %s\n", tag); return -1; } |