aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Niedermayer <michaelni@gmx.at>2013-01-16 12:13:10 +0100
committerMichael Niedermayer <michaelni@gmx.at>2013-01-16 12:13:16 +0100
commit1459f34251c35e5ba5af38ab9ae25ec134dbc845 (patch)
tree5f5d7b8d1efb472240a405e829db278fecfde179
parent9ea65c65f70029896b1c23afd9925ae42c4d009f (diff)
parent977d4a3b8a2dbc2fb5e747c7072485016c9cdfaa (diff)
downloadffmpeg-1459f34251c35e5ba5af38ab9ae25ec134dbc845.tar.gz
Merge commit '977d4a3b8a2dbc2fb5e747c7072485016c9cdfaa'
* commit '977d4a3b8a2dbc2fb5e747c7072485016c9cdfaa': rtpdec_mpeg4: Check the return value from malloc srtp: Mark a few variables as uninitialized Merged-by: Michael Niedermayer <michaelni@gmx.at>
-rw-r--r--libavformat/rtpdec_mpeg4.c6
-rw-r--r--libavformat/srtp.c4
2 files changed, 7 insertions, 3 deletions
diff --git a/libavformat/rtpdec_mpeg4.c b/libavformat/rtpdec_mpeg4.c
index a3f60d348d..737d993b07 100644
--- a/libavformat/rtpdec_mpeg4.c
+++ b/libavformat/rtpdec_mpeg4.c
@@ -137,6 +137,8 @@ static int rtp_parse_mp4_au(PayloadContext *data, const uint8_t *buf)
if (!data->au_headers || data->au_headers_allocated < data->nb_au_headers) {
av_free(data->au_headers);
data->au_headers = av_malloc(sizeof(struct AUHeaders) * data->nb_au_headers);
+ if (!data->au_headers)
+ return AVERROR(ENOMEM);
data->au_headers_allocated = data->nb_au_headers;
}
@@ -162,6 +164,7 @@ static int aac_parse_packet(AVFormatContext *ctx, PayloadContext *data,
const uint8_t *buf, int len, uint16_t seq,
int flags)
{
+ int ret;
if (rtp_parse_mp4_au(data, buf))
return -1;
@@ -170,7 +173,8 @@ static int aac_parse_packet(AVFormatContext *ctx, PayloadContext *data,
/* XXX: Fixme we only handle the case where rtp_parse_mp4_au define
one au_header */
- av_new_packet(pkt, data->au_headers[0].size);
+ if ((ret = av_new_packet(pkt, data->au_headers[0].size)) < 0)
+ return ret;
memcpy(pkt->data, buf, data->au_headers[0].size);
pkt->stream_index = st->index;
diff --git a/libavformat/srtp.c b/libavformat/srtp.c
index b3c428ba0f..7d344b816d 100644
--- a/libavformat/srtp.c
+++ b/libavformat/srtp.c
@@ -121,8 +121,8 @@ int ff_srtp_decrypt(struct SRTPContext *s, uint8_t *buf, int *lenptr)
{
uint8_t iv[16] = { 0 }, hmac[20];
int len = *lenptr;
- int ext, seq_largest;
- uint32_t ssrc, roc;
+ int ext, av_uninit(seq_largest);
+ uint32_t ssrc, av_uninit(roc);
uint64_t index;
int rtcp;