From: Zhao Zhili <zhiliz...@tencent.com>
This can happen for HLS with AES-128 at the middle of m3u8, so old
protocol is https while new protocol is crypt+https.
And change the log level from ERROR to INFO when protocol/host/port
don't match. User should prepared for this function to fail and
retry with new connection.
---
libavformat/http.c | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/libavformat/http.c b/libavformat/http.c
index 65ea5d993c..f7b2a8a029 100644
--- a/libavformat/http.c
+++ b/libavformat/http.c
@@ -498,8 +498,13 @@ int ff_http_do_new_request2(URLContext *h, const char
*uri, AVDictionary **opts)
av_url_split(proto2, sizeof(proto2), NULL, 0,
hostname2, sizeof(hostname2), &port2,
NULL, 0, uri);
+ if (strcmp(proto1, proto2) != 0) {
+ av_log(h, AV_LOG_INFO, "Cannot reuse HTTP connection for different
protocol %s vs %s\n",
+ proto1, proto2);
+ return AVERROR(EINVAL);
+ }
if (port1 != port2 || strncmp(hostname1, hostname2, sizeof(hostname2)) !=
0) {
- av_log(h, AV_LOG_ERROR, "Cannot reuse HTTP connection for different
host: %s:%d != %s:%d\n",
+ av_log(h, AV_LOG_INFO, "Cannot reuse HTTP connection for different
host: %s:%d != %s:%d\n",
hostname1, port1,
hostname2, port2
);
--
2.46.0
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
