Hi,
apologies for the delay. Finally found time to investigate the random
failures -- turned out to be reading a variable that's uninitialized
after my original patch. The attached fix works for me.

-- 
Anton Khirnov
From 414522d5049a230e71a2c2fef45a6b525d6a9803 Mon Sep 17 00:00:00 2001
From: Anton Khirnov <an...@khirnov.net>
Date: Sun, 26 Oct 2014 18:29:48 +0100
Subject: [PATCH] Fix uninitialized read.

---
 xbmc/cores/dvdplayer/DVDCodecs/Video/VDPAU.cpp | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/xbmc/cores/dvdplayer/DVDCodecs/Video/VDPAU.cpp b/xbmc/cores/dvdplayer/DVDCodecs/Video/VDPAU.cpp
index 932985a..9d90a9c 100644
--- a/xbmc/cores/dvdplayer/DVDCodecs/Video/VDPAU.cpp
+++ b/xbmc/cores/dvdplayer/DVDCodecs/Video/VDPAU.cpp
@@ -1095,9 +1095,6 @@ int CDecoder::Decode(AVCodecContext *avctx, AVFrame *pFrame)
 
   CSingleLock lock(m_DecoderSection);
 
-  if (m_DecoderError && pFrame)
-    return VC_ERROR;
-
   if (!m_vdpauConfigured)
     return VC_ERROR;
 
-- 
2.0.0.rc2

Reply via email to