Merge "main/sdp_srtp.c: allow SDP crypto tag to be up to 9 digits"

10 years ago · 8983273cc4
parent 31f0d78d7b d2e998cd68
commit 8983273cc4
1 changed files with 2 additions and 1 deletions
--- 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;
 	}