summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--libmpcodecs/vd_realvid.c5
-rw-r--r--libmpdemux/demux_real.c20
2 files changed, 19 insertions, 6 deletions
diff --git a/libmpcodecs/vd_realvid.c b/libmpcodecs/vd_realvid.c
index 6bee6c2aca..acdc38ee3f 100644
--- a/libmpcodecs/vd_realvid.c
+++ b/libmpcodecs/vd_realvid.c
@@ -253,7 +253,10 @@ static int init(sh_video_t *sh){
}
// setup rv30 codec (codec sub-type and image dimensions):
if((sh->format<=0x30335652) && (extrahdr[1]>=0x20200002)){
- uint32_t cmsg24[4]={sh->disp_w,sh->disp_h,((unsigned short *)extrahdr)[4],((unsigned short *)extrahdr)[5]};
+ // We could read nonsense data while filling this, but input is big enough so no sig11
+ uint32_t cmsg24[8]={sh->disp_w,sh->disp_h,((unsigned char *)extrahdr)[8]*4,((unsigned char *)extrahdr)[9]*4,
+ ((unsigned char *)extrahdr)[10]*4,((unsigned char *)extrahdr)[11]*4,
+ ((unsigned char *)extrahdr)[12]*4,((unsigned char *)extrahdr)[13]*4};
cmsg_data_t cmsg_data={0x24,1+((extrahdr[0]>>16)&7), &cmsg24[0]};
#ifdef USE_WIN32DLL
diff --git a/libmpdemux/demux_real.c b/libmpdemux/demux_real.c
index 86c16b89cd..111998b507 100644
--- a/libmpdemux/demux_real.c
+++ b/libmpdemux/demux_real.c
@@ -1447,8 +1447,8 @@ void demux_open_real(demuxer_t* demuxer)
mp_msg(MSGT_DEMUX,MSGL_V,"video fourcc: %.4s (%x)\n", (char *)&sh->format, sh->format);
/* emulate BITMAPINFOHEADER */
- sh->bih = malloc(sizeof(BITMAPINFOHEADER)+12);
- memset(sh->bih, 0, sizeof(BITMAPINFOHEADER)+12);
+ sh->bih = malloc(sizeof(BITMAPINFOHEADER)+16);
+ memset(sh->bih, 0, sizeof(BITMAPINFOHEADER)+16);
sh->bih->biSize = 48;
sh->disp_w = sh->bih->biWidth = stream_read_word(demuxer->stream);
sh->disp_h = sh->bih->biHeight = stream_read_word(demuxer->stream);
@@ -1517,9 +1517,19 @@ void demux_open_real(demuxer_t* demuxer)
}
if((sh->format<=0x30335652) && (tmp>=0x20200002)){
- // read secondary WxH for the cmsg24[] (see vd_realvid.c)
- ((unsigned short*)(sh->bih+1))[4]=4*(unsigned short)stream_read_char(demuxer->stream); //widht
- ((unsigned short*)(sh->bih+1))[5]=4*(unsigned short)stream_read_char(demuxer->stream); //height
+ // read data for the cmsg24[] (see vd_realvid.c)
+ unsigned int cnt = codec_data_size - (stream_tell(demuxer->stream) - codec_pos);
+ if (cnt < 2) {
+ mp_msg(MSGT_DEMUX, MSGL_ERR,"realvid: cmsg24 data too short (size %u)\n", cnt);
+ } else {
+ int ii;
+ if (cnt > 6) {
+ mp_msg(MSGT_DEMUX, MSGL_WARN,"realvid: cmsg24 data too big, please report (size %u)\n", cnt);
+ cnt = 6;
+ }
+ for (ii = 0; ii < cnt; ii++)
+ ((unsigned char*)(sh->bih+1))[8+ii]=(unsigned short)stream_read_char(demuxer->stream);
+ }
}
/* Select video stream with highest bitrate if multirate file*/