From 8de0990e9f22d7f728f70d5a03f3e78fc958f658 Mon Sep 17 00:00:00 2001 From: Michael Niedermayer Date: Sat, 17 Jan 2015 16:21:36 +0100 Subject: [PATCH] avcodec/dvbsubdec: Check get_region() return value Fixes null pointer dereference with DEBUG enabled Signed-off-by: Michael Niedermayer --- libavcodec/dvbsubdec.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/libavcodec/dvbsubdec.c b/libavcodec/dvbsubdec.c index ce68425b5b..5dd362a274 100644 --- a/libavcodec/dvbsubdec.c +++ b/libavcodec/dvbsubdec.c @@ -1373,6 +1373,9 @@ static void save_display_set(DVBSubContext *ctx) for (display = ctx->display_list; display; display = display->next) { region = get_region(ctx, display->region_id); + if (!region) + return; + if (x_pos == -1) { x_pos = display->x_pos; y_pos = display->y_pos; @@ -1406,6 +1409,9 @@ static void save_display_set(DVBSubContext *ctx) for (display = ctx->display_list; display; display = display->next) { region = get_region(ctx, display->region_id); + if (!region) + return; + x_off = display->x_pos - x_pos; y_off = display->y_pos - y_pos; -- 2.11.0