On Tue, 8 Jun 2021, hou...@uniontech.com wrote:
From: houlei <hou...@uniontech.com> Signed-off-by: houlei <hou...@uniontech.com> --- libavdevice/xv.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/libavdevice/xv.c b/libavdevice/xv.c index 24ba3179f6..7e15c178b6 100644 --- a/libavdevice/xv.c +++ b/libavdevice/xv.c @@ -296,10 +296,11 @@ static int write_picture(AVFormatContext *s, uint8_t *input_data[4], { XVContext *xv = s->priv_data; XvImage *img = xv->yuv_image; - uint8_t *data[3] = { + uint8_t *data[4] = { img->data + img->offsets[0], img->data + img->offsets[1], - img->data + img->offsets[2] + img->data + img->offsets[2], + img->data + img->offsets[3]
This does not look good, img->offsets[3] is probably not valid, because img->offsets and img->pitches are both arrays of img->num_planes, which is 3... So to be correct, img->pitches should also be copied to a 4 long array, and that should be passed to av_image_copy, but the 4th value can be left as 0 both for pitches and data.
Regards, Marton
}; /* Check messages. Window might get closed. */ -- 2.20.1 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".