From 37a0c9c404e7cb0fe9149b0472a08a9c72715239 Mon Sep 17 00:00:00 2001 From: Chris Burgener Date: Wed, 20 Jul 2016 16:46:53 -0400 Subject: [PATCH] Check for zero height when splitting video dump --- Source/Core/VideoCommon/AVIDump.cpp | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/Source/Core/VideoCommon/AVIDump.cpp b/Source/Core/VideoCommon/AVIDump.cpp index b18b77dea4..c0a2613ea5 100644 --- a/Source/Core/VideoCommon/AVIDump.cpp +++ b/Source/Core/VideoCommon/AVIDump.cpp @@ -296,7 +296,12 @@ void AVIDump::DoState() void AVIDump::CheckResolution(int width, int height) { - if (width != s_current_width || height != s_current_height) + // We check here to see if the requested width and height have changed since the last frame which + // was dumped, then create a new file accordingly. However, is it possible for the height + // (possibly width as well, but no examples known) to have a value of zero. This can occur as the + // VI is able to be set to a zero value for height/width to disable output. If this is the case, + // simply keep the last known resolution of the video for the added frame. + if ((width != s_current_width || height != s_current_height) && (width > 0 && height > 0)) { int temp_file_index = s_file_index; Stop();