Commit 404db2b9 authored by Michael Niedermayer's avatar Michael Niedermayer

avformat/subtitles: Use size_t for len

string length could theoretically be larger than int
Reviewed-by: 's avatarClément Bœsch <u@pkh.me>
Signed-off-by: 's avatarMichael Niedermayer <michaelni@gmx.at>
(cherry picked from commit a633928d)
Signed-off-by: 's avatarMichael Niedermayer <michaelni@gmx.at>
parent 17ad9778
...@@ -24,7 +24,7 @@ ...@@ -24,7 +24,7 @@
#include "libavutil/avstring.h" #include "libavutil/avstring.h"
AVPacket *ff_subtitles_queue_insert(FFDemuxSubtitlesQueue *q, AVPacket *ff_subtitles_queue_insert(FFDemuxSubtitlesQueue *q,
const uint8_t *event, int len, int merge) const uint8_t *event, size_t len, int merge)
{ {
AVPacket *subs, *sub; AVPacket *subs, *sub;
...@@ -218,7 +218,7 @@ int ff_smil_extract_next_chunk(AVIOContext *pb, AVBPrint *buf, char *c) ...@@ -218,7 +218,7 @@ int ff_smil_extract_next_chunk(AVIOContext *pb, AVBPrint *buf, char *c)
const char *ff_smil_get_attr_ptr(const char *s, const char *attr) const char *ff_smil_get_attr_ptr(const char *s, const char *attr)
{ {
int in_quotes = 0; int in_quotes = 0;
const int len = strlen(attr); const size_t len = strlen(attr);
while (*s) { while (*s) {
while (*s) { while (*s) {
......
...@@ -47,7 +47,7 @@ typedef struct { ...@@ -47,7 +47,7 @@ typedef struct {
* previous one instead of adding a new entry, 0 otherwise * previous one instead of adding a new entry, 0 otherwise
*/ */
AVPacket *ff_subtitles_queue_insert(FFDemuxSubtitlesQueue *q, AVPacket *ff_subtitles_queue_insert(FFDemuxSubtitlesQueue *q,
const uint8_t *event, int len, int merge); const uint8_t *event, size_t len, int merge);
/** /**
* Set missing durations and sort subtitles by PTS, and then byte position. * Set missing durations and sort subtitles by PTS, and then byte position.
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment