[vlc-devel] [PATCH 2/2] win32/common: make sure the values we're using are valid

Steve Lhomme robux4 at videolabs.io
Thu Jul 28 15:27:00 CEST 2016


---
 modules/video_output/win32/common.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/modules/video_output/win32/common.c b/modules/video_output/win32/common.c
index c3029a5..c15a8f9 100644
--- a/modules/video_output/win32/common.c
+++ b/modules/video_output/win32/common.c
@@ -452,6 +452,8 @@ int CommonUpdatePicture(picture_t *picture, picture_t **fallback,
     picture->p->p_pixels = data;
     picture->p->i_pitch  = pitch;
     picture->p->i_lines  = picture->format.i_height;
+    assert(picture->p->i_visible_pitch <= picture->p->i_pitch);
+    assert(picture->p->i_visible_lines <= picture->p->i_lines);
 
     /*  Fill chroma planes for biplanar YUV */
     if (picture->format.i_chroma == VLC_CODEC_NV12 ||
@@ -464,6 +466,8 @@ int CommonUpdatePicture(picture_t *picture, picture_t **fallback,
             p->p_pixels = o->p_pixels + o->i_lines * o->i_pitch;
             p->i_pitch  = pitch;
             p->i_lines  = picture->format.i_height;
+            assert(p->i_visible_pitch <= p->i_pitch);
+            assert(p->i_visible_lines <= p->i_lines);
         }
         /* The dx/d3d buffer is always allocated as NV12 */
         if (vlc_fourcc_AreUVPlanesSwapped(picture->format.i_chroma, VLC_CODEC_NV12)) {
-- 
2.8.2



More information about the vlc-devel mailing list