[FFmpeg-cvslog] avformat/udp: properly check for valid ttl in url

Limin Wang git at videolan.org
Sat Feb 12 02:43:31 EET 2022


ffmpeg | branch: master | Limin Wang <lance.lmwang at gmail.com> | Mon Feb  7 18:22:02 2022 +0800| [1de24ee5120b175f38fa42a6b1fbfeeb9f4a50d5] | committer: Limin Wang

avformat/udp: properly check for valid ttl in url

Zhao Zhili added a ttl upper bound in commit 9daac85da8,
but the check for ttl in url is missing still.

Signed-off-by: Limin Wang <lance.lmwang at gmail.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=1de24ee5120b175f38fa42a6b1fbfeeb9f4a50d5
---

 libavformat/udp.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/libavformat/udp.c b/libavformat/udp.c
index da56c8ee1d..401d9b6275 100644
--- a/libavformat/udp.c
+++ b/libavformat/udp.c
@@ -674,6 +674,11 @@ static int udp_open(URLContext *h, const char *uri, int flags)
         }
         if (av_find_info_tag(buf, sizeof(buf), "ttl", p)) {
             s->ttl = strtol(buf, NULL, 10);
+            if (s->ttl < 0 || s->ttl > 255) {
+                av_log(h, AV_LOG_ERROR, "ttl(%d) should be in range [0,255]\n", s->ttl);
+                ret = AVERROR(EINVAL);
+                goto fail;
+            }
         }
         if (av_find_info_tag(buf, sizeof(buf), "udplite_coverage", p)) {
             s->udplite_coverage = strtol(buf, NULL, 10);



More information about the ffmpeg-cvslog mailing list