From 2d297c7b9a08adca5a81f39646fe3028084e53d8 Mon Sep 17 00:00:00 2001 From: Corey Edwards Date: Wed, 20 May 2015 13:45:37 -0600 Subject: [PATCH] chan_sip/sdp_crypto.c: allow SDP crypto tag to be up to 9 digits ASTERISK-24887 #close Reported by: Makoto Dei Tested by: tensai Change-Id: I6a96f572adb17f76b3acafe503a01c48eb5dd9bf --- channels/sip/sdp_crypto.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/channels/sip/sdp_crypto.c b/channels/sip/sdp_crypto.c index c78f473b24..db5bf9911d 100644 --- a/channels/sip/sdp_crypto.c +++ b/channels/sip/sdp_crypto.c @@ -220,7 +220,8 @@ int sdp_crypto_process(struct sdp_crypto *p, const char *attr, struct ast_rtp_in return -1; } - if (sscanf(tag, "%30d", &p->tag) != 1 || p->tag <= 0 || p->tag > 9) { + /* RFC4568 9.1 - tag is 1-9 digits, greater than zero */ + if (sscanf(tag, "%30d", &p->tag) != 1 || p->tag <= 0 || p->tag > 999999999) { ast_log(LOG_WARNING, "Unacceptable a=crypto tag: %s\n", tag); return -1; } -- 2.47.2