[FFmpeg-devel] [PATCH v2] avformat/hlsenc: Fix path handling for Windows
softworkz
ffmpegagent at gmail.com
Sat Jun 14 03:12:26 EEST 2025
From: softworkz <softworkz at hotmail.com>
Signed-off-by: softworkz <softworkz at hotmail.com>
---
avformat/hlsenc: Some HLS improvements
Includes only a single patch now:
* Fix path handling on Windows
Versions
========
V2
==
* Fix path handling for Windows;
* Keep using av_asprintf() with "%*s" to limit the length (as per
review by Marton - thanks!)
* Use int64:_t instead of int
* Removed logging patches
.
Published-As: https://github.com/ffstaging/FFmpeg/releases/tag/pr-ffstaging-97%2Fsoftworkz%2Fsubmit_hls_logging-v2
Fetch-It-Via: git fetch https://github.com/ffstaging/FFmpeg pr-ffstaging-97/softworkz/submit_hls_logging-v2
Pull-Request: https://github.com/ffstaging/FFmpeg/pull/97
Range-diff vs v1:
1: 58988164f2 < -: ---------- avformat/hlsenc: Log information about segments written
2: 1a4268d7e1 < -: ---------- avformat/hlsenc: Log InitFileComplete message
3: 99dfda2420 ! 1: 93e1a502be avformat/hlsenc: Fix path handling for Windows
@@ libavformat/hlsenc.c: static int hlsenc_io_close(AVFormatContext *s, AVIOContext
return ret;
}
-+static int get_last_separator_pos(const char *path)
++static int64_t get_last_separator_pos(const char *path)
+{
+ if (!path || *path == '\0')
+ return -1;
@@ libavformat/hlsenc.c: static int hls_rename_temp_file(AVFormatContext *s, AVForm
- size_t base_len = 0;
-
- if (!p) p = strrchr(master_url, '\\');
-+ int pos = get_last_separator_pos(master_url);
++ int64_t pos = get_last_separator_pos(master_url);
- if (p) {
- base_len = p - master_url;
@@ libavformat/hlsenc.c: static int hls_init(AVFormatContext *s)
- char tmp = *(++p);
- *p = '\0';
- vs->base_output_dirname = av_asprintf("%s%s", vs->m3u8_name,
-- vs->fmp4_init_filename);
-- *p = tmp;
-+ int pos = get_last_separator_pos(vs->m3u8_name);
++ int64_t pos = get_last_separator_pos(vs->m3u8_name);
+ if (pos >= 0) {
-+ AVBPrint buf;
-+ av_bprint_init(&buf, 0, AV_BPRINT_SIZE_UNLIMITED);
-+ av_bprint_append_data(&buf, vs->m3u8_name, pos + 1);
-+ av_bprintf(&buf, "%s", vs->fmp4_init_filename);
-+ av_bprint_finalize(&buf, &vs->base_output_dirname);
++ vs->base_output_dirname = av_asprintf("%.*s%s", pos + 1, vs->m3u8_name,
+ vs->fmp4_init_filename);
+- *p = tmp;
} else {
vs->base_output_dirname = av_strdup(vs->fmp4_init_filename);
}
libavformat/hlsenc.c | 38 ++++++++++++++++++++++++--------------
1 file changed, 24 insertions(+), 14 deletions(-)
diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
index a93d35ab75..965ed9c1c0 100644
--- a/libavformat/hlsenc.c
+++ b/libavformat/hlsenc.c
@@ -329,6 +329,23 @@ static int hlsenc_io_close(AVFormatContext *s, AVIOContext **pb, char *filename)
return ret;
}
+static int64_t get_last_separator_pos(const char *path)
+{
+ if (!path || *path == '\0')
+ return -1;
+
+ char *p = strrchr(path, '/');
+#if HAVE_DOS_PATHS
+ char *q = strrchr(path, '\\');
+ p = FFMAX(p, q);
+#endif
+
+ if (!p)
+ return -1;
+
+ return p - path;
+}
+
static void set_http_options(AVFormatContext *s, AVDictionary **options, HLSContext *c)
{
int http_base_proto = ff_is_http_proto(s->url);
@@ -1408,14 +1425,10 @@ static int hls_rename_temp_file(AVFormatContext *s, AVFormatContext *oc)
static const char* get_relative_url(const char *master_url, const char *media_url)
{
- const char *p = strrchr(master_url, '/');
- size_t base_len = 0;
-
- if (!p) p = strrchr(master_url, '\\');
+ int64_t pos = get_last_separator_pos(master_url);
- if (p) {
- base_len = p - master_url;
- if (av_strncasecmp(master_url, media_url, base_len)) {
+ if (pos >= 0) {
+ if (av_strncasecmp(master_url, media_url, pos)) {
av_log(NULL, AV_LOG_WARNING, "Unable to find relative url\n");
return NULL;
}
@@ -1423,7 +1436,7 @@ static const char* get_relative_url(const char *master_url, const char *media_ur
return media_url;
}
- return media_url + base_len + 1;
+ return media_url + pos + 1;
}
static int64_t get_stream_bit_rate(AVStream *stream)
@@ -3142,13 +3155,10 @@ static int hls_init(AVFormatContext *s)
vs->fmp4_init_filename = expanded;
}
- p = strrchr(vs->m3u8_name, '/');
- if (p) {
- char tmp = *(++p);
- *p = '\0';
- vs->base_output_dirname = av_asprintf("%s%s", vs->m3u8_name,
+ int64_t pos = get_last_separator_pos(vs->m3u8_name);
+ if (pos >= 0) {
+ vs->base_output_dirname = av_asprintf("%.*s%s", pos + 1, vs->m3u8_name,
vs->fmp4_init_filename);
- *p = tmp;
} else {
vs->base_output_dirname = av_strdup(vs->fmp4_init_filename);
}
base-commit: f3bf5356966b4ad9334034e720fa224c2d9464a6
--
ffmpeg-codebot
More information about the ffmpeg-devel
mailing list