Commit 5245adb9 authored by Martin Storsjö's avatar Martin Storsjö

rtpdec_h264: Check the available data length before reading

This makes sure the length is checked for STAP-A type packets.
Signed-off-by: 's avatarMartin Storsjö <martin@martin.st>
parent de26a4b6
...@@ -218,7 +218,7 @@ static int h264_handle_packet(AVFormatContext *ctx, ...@@ -218,7 +218,7 @@ static int h264_handle_packet(AVFormatContext *ctx,
const uint8_t *src= buf; const uint8_t *src= buf;
int src_len= len; int src_len= len;
do { while (src_len > 2) {
uint16_t nal_size = AV_RB16(src); // this going to be a problem if unaligned (can it be?) uint16_t nal_size = AV_RB16(src); // this going to be a problem if unaligned (can it be?)
// consume the length of the aggregate... // consume the length of the aggregate...
...@@ -252,7 +252,7 @@ static int h264_handle_packet(AVFormatContext *ctx, ...@@ -252,7 +252,7 @@ static int h264_handle_packet(AVFormatContext *ctx,
if (src_len < 0) if (src_len < 0)
av_log(ctx, AV_LOG_ERROR, av_log(ctx, AV_LOG_ERROR,
"Consumed more bytes than we got! (%d)\n", src_len); "Consumed more bytes than we got! (%d)\n", src_len);
} while (src_len > 2); // because there could be rtp padding.. }
if(pass==0) { if(pass==0) {
// now we know the total size of the packet (with the start sequences added) // now we know the total size of the packet (with the start sequences added)
......
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