aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Niedermayer <michaelni@gmx.at>2015-06-08 23:05:01 +0200
committerMichael Niedermayer <michaelni@gmx.at>2015-06-08 23:05:01 +0200
commitd5a645625d876288021d8192e4e3199087a753fc (patch)
tree22a73c8dd0e3288b64f78ff3862aa7940b759434
parent3d6635749ab2f3507a73cb450038118ae561d74c (diff)
parent4733a12dd17a91d606e0079ff9bb48b9f419cbef (diff)
downloadffmpeg-d5a645625d876288021d8192e4e3199087a753fc.tar.gz
Merge commit '4733a12dd17a91d606e0079ff9bb48b9f419cbef'
* commit '4733a12dd17a91d606e0079ff9bb48b9f419cbef': rtpdec_asf: Check memory allocation and free memory on error Conflicts: libavformat/rtpdec_asf.c Merged-by: Michael Niedermayer <michaelni@gmx.at>
-rw-r--r--libavformat/rtpdec_asf.c13
1 files changed, 11 insertions, 2 deletions
diff --git a/libavformat/rtpdec_asf.c b/libavformat/rtpdec_asf.c
index 066bb0ed37..e59480c9e5 100644
--- a/libavformat/rtpdec_asf.c
+++ b/libavformat/rtpdec_asf.c
@@ -108,6 +108,8 @@ int ff_wms_parse_sdp_a_line(AVFormatContext *s, const char *p)
char *buf = av_mallocz(len);
AVInputFormat *iformat;
+ if (!buf)
+ return AVERROR(ENOMEM);
av_base64_decode(buf, p, len);
if (rtp_asf_fix_header(buf, len) < 0)
@@ -117,10 +119,15 @@ int ff_wms_parse_sdp_a_line(AVFormatContext *s, const char *p)
if (rt->asf_ctx) {
avformat_close_input(&rt->asf_ctx);
}
+
if (!(iformat = av_find_input_format("asf")))
return AVERROR_DEMUXER_NOT_FOUND;
- if (!(rt->asf_ctx = avformat_alloc_context()))
+
+ rt->asf_ctx = avformat_alloc_context();
+ if (!rt->asf_ctx) {
+ av_free(buf);
return AVERROR(ENOMEM);
+ }
rt->asf_ctx->pb = &pb;
av_dict_set(&opts, "no_resync_search", "1", 0);
@@ -131,8 +138,10 @@ int ff_wms_parse_sdp_a_line(AVFormatContext *s, const char *p)
ret = avformat_open_input(&rt->asf_ctx, "", iformat, &opts);
av_dict_free(&opts);
- if (ret < 0)
+ if (ret < 0) {
+ av_free(buf);
return ret;
+ }
av_dict_copy(&s->metadata, rt->asf_ctx->metadata, 0);
rt->asf_pb_pos = avio_tell(&pb);
av_free(buf);