virtualenv/FFmpeg
1
0
Fork 0
mirror of https://github.com/FFmpeg/FFmpeg.git synced 2024-12-23 12:43:46 +02:00
Code Issues Releases Activity

Revert flvdec.c part of r20836

Browse Source 
Log:
	Use AV_METADATA_DONT_STRDUP* / use av_malloced metadata instead of strduped
	arrays of fixed length.
	Code from ffmbc with changes to adapt to our metadata API.
Reason: memleak & fix is not trivial

Originally committed as revision 20866 to svn://svn.ffmpeg.org/ffmpeg/trunk
This commit is contained in:
Michael Niedermayer 2009-12-14 15:59:19 +00:00
parent 0f649d667e
commit cc38e063be
1 changed files with 21 additions and 25 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

46
libavformat/flvdec.c
View File

@ -104,25 +104,25 @@ static int flv_set_video_codec(AVFormatContext *s, AVStream *vstream, int flv_co
return 0; return 0;
} }
static int amf_get_string(ByteIOContext *ioc, char **buf) static int amf_get_string(ByteIOContext *ioc, char *buffer, int buffsize) {
{ int length = get_be16(ioc);
uint16_t len = get_be16(ioc); if(length >= buffsize) {
url_fskip(ioc, length);
return -1;
}
*buf = av_malloc(len+1); get_buffer(ioc, buffer, length);
if (!*buf)
return AVERROR_NOMEM;
get_buffer(ioc, *buf, len); buffer[length] = '\0';
(*buf)[len] = '\0';
return len; return length;
} }
static int amf_parse_object(AVFormatContext *s, AVStream *astream, AVStream *vstream, const char *key, int64_t max_pos, int depth) { static int amf_parse_object(AVFormatContext *s, AVStream *astream, AVStream *vstream, const char *key, int64_t max_pos, int depth) {
AVCodecContext *acodec, *vcodec; AVCodecContext *acodec, *vcodec;
ByteIOContext *ioc; ByteIOContext *ioc;
AMFDataType amf_type; AMFDataType amf_type;
char *str = NULL; char str_val[256];
double num_val; double num_val;
num_val = 0; num_val = 0;
@ -136,7 +136,7 @@ static int amf_parse_object(AVFormatContext *s, AVStream *astream, AVStream *vst
case AMF_DATA_TYPE_BOOL: case AMF_DATA_TYPE_BOOL:
num_val = get_byte(ioc); break; num_val = get_byte(ioc); break;
case AMF_DATA_TYPE_STRING: case AMF_DATA_TYPE_STRING:
if(amf_get_string(ioc, &str) < 0) if(amf_get_string(ioc, str_val, sizeof(str_val)) < 0)
return -1; return -1;
break; break;
case AMF_DATA_TYPE_OBJECT: { case AMF_DATA_TYPE_OBJECT: {
@ -157,11 +157,10 @@ static int amf_parse_object(AVFormatContext *s, AVStream *astream, AVStream *vst
break; //these take up no additional space break; //these take up no additional space
case AMF_DATA_TYPE_MIXEDARRAY: case AMF_DATA_TYPE_MIXEDARRAY:
url_fskip(ioc, 4); //skip 32-bit max array index url_fskip(ioc, 4); //skip 32-bit max array index
while(url_ftell(ioc) < max_pos - 2 && amf_get_string(ioc, &str) > 0) { while(url_ftell(ioc) < max_pos - 2 && amf_get_string(ioc, str_val, sizeof(str_val)) > 0) {
//this is the only case in which we would want a nested parse to not skip over the object //this is the only case in which we would want a nested parse to not skip over the object
if(amf_parse_object(s, astream, vstream, str, max_pos, depth + 1) < 0) if(amf_parse_object(s, astream, vstream, str_val, max_pos, depth + 1) < 0)
return -1; return -1;
av_freep(&str);
} }
if(get_byte(ioc) != AMF_END_OF_OBJECT) if(get_byte(ioc) != AMF_END_OF_OBJECT)
return -1; return -1;
@ -188,15 +187,16 @@ static int amf_parse_object(AVFormatContext *s, AVStream *astream, AVStream *vst
vcodec = vstream ? vstream->codec : NULL; vcodec = vstream ? vstream->codec : NULL;
if(amf_type == AMF_DATA_TYPE_BOOL) { if(amf_type == AMF_DATA_TYPE_BOOL) {
av_metadata_set2(&s->metadata, key, av_d2str(num_val), AV_METADATA_DONT_STRDUP_VAL); av_strlcpy(str_val, num_val > 0 ? "true" : "false", sizeof(str_val));
av_metadata_set(&s->metadata, key, str_val);
} else if(amf_type == AMF_DATA_TYPE_NUMBER) { } else if(amf_type == AMF_DATA_TYPE_NUMBER) {
av_metadata_set2(&s->metadata, key, av_d2str(num_val), AV_METADATA_DONT_STRDUP_VAL); snprintf(str_val, sizeof(str_val), "%.f", num_val);
av_metadata_set(&s->metadata, key, str_val);
if(!strcmp(key, "duration")) s->duration = num_val * AV_TIME_BASE; if(!strcmp(key, "duration")) s->duration = num_val * AV_TIME_BASE;
else if(!strcmp(key, "videodatarate") && vcodec && 0 <= (int)(num_val * 1024.0)) else if(!strcmp(key, "videodatarate") && vcodec && 0 <= (int)(num_val * 1024.0))
vcodec->bit_rate = num_val * 1024.0; vcodec->bit_rate = num_val * 1024.0;
} else if (amf_type == AMF_DATA_TYPE_STRING) } else if (amf_type == AMF_DATA_TYPE_STRING)
av_metadata_set2(&s->metadata, key, str, av_metadata_set(&s->metadata, key, str_val);
AV_METADATA_DONT_STRDUP_VAL);
} }
return 0; return 0;
@ -207,7 +207,7 @@ static int flv_read_metabody(AVFormatContext *s, int64_t next_pos) {
AVStream *stream, *astream, *vstream; AVStream *stream, *astream, *vstream;
ByteIOContext *ioc; ByteIOContext *ioc;
int i; int i;
char *buf = NULL; //only needs to hold the string "onMetaData". Anything longer is something we don't want. char buffer[11]; //only needs to hold the string "onMetaData". Anything longer is something we don't want.
astream = NULL; astream = NULL;
vstream = NULL; vstream = NULL;
@ -215,11 +215,8 @@ static int flv_read_metabody(AVFormatContext *s, int64_t next_pos) {
//first object needs to be "onMetaData" string //first object needs to be "onMetaData" string
type = get_byte(ioc); type = get_byte(ioc);
if (type != AMF_DATA_TYPE_STRING || if(type != AMF_DATA_TYPE_STRING || amf_get_string(ioc, buffer, sizeof(buffer)) < 0 || strcmp(buffer, "onMetaData"))
amf_get_string(ioc, &buf) < 0 || strcmp(buf, "onMetaData")) {
av_freep(&buf);
return -1; return -1;
}
//find the streams now so that amf_parse_object doesn't need to do the lookup every time it is called. //find the streams now so that amf_parse_object doesn't need to do the lookup every time it is called.
for(i = 0; i < s->nb_streams; i++) { for(i = 0; i < s->nb_streams; i++) {
@ -229,9 +226,8 @@ static int flv_read_metabody(AVFormatContext *s, int64_t next_pos) {
} }
//parse the second object (we want a mixed array) //parse the second object (we want a mixed array)
if(amf_parse_object(s, astream, vstream, buf, next_pos, 0) < 0) if(amf_parse_object(s, astream, vstream, buffer, next_pos, 0) < 0)
return -1; return -1;
av_freep(&buf);
return 0; return 0;
} }