<div dir="ltr">From 2159cb01d9035950c0bb4ea19ef10ca2a806be8a Mon Sep 17 00:00:00 2001<br>From: Ankur Gupta <<a href="mailto:ankur.vision0@gmail.com">ankur.vision0@gmail.com</a>><br>Date: Mon, 7 Mar 2016 11:01:51 +0530<br>Subject: [PATCH] Fix crash caused by using Volume Slider when no media file is<br> loaded in VLMC<br>Fixed: A segmentation fault occurs when no media file is imported in the project and volume slider is clicked. <br>This happens because the object m_sourceRenderer is NULL while the volume slider is clicked in given situation.<br>P.S. This patch is my attempt at GSOC 2016. I am a student of Masters in Computer <br>Science at BITS Pilani, Hyderabad Campus. Kindly revert if I should do more exhaustive patches. <br>---<br> src/Renderer/ClipRenderer.cpp     | 6 +++++-<br> src/Renderer/WorkflowRenderer.cpp | 6 +++++-<br> 2 files changed, 10 insertions(+), 2 deletions(-)<br><br>diff --git a/src/Renderer/ClipRenderer.cpp b/src/Renderer/ClipRenderer.cpp<br>index aecdd0a..ec11e09 100644<br>--- a/src/Renderer/ClipRenderer.cpp<br>+++ b/src/Renderer/ClipRenderer.cpp<br>@@ -150,7 +150,11 @@ ClipRenderer::getVolume() const<br> void ClipRenderer::setVolume( int volume )<br> {<br>     //Returns 0 if the volume was set, -1 if it was out of range<br>-    return m_sourceRenderer->setVolume( volume );<br>+    //Handles the 'Clip Preview Window' VolSlider Crash when media file is not loaded<br>+    if(m_sourceRenderer != NULL)<br>+    {<br>+    m_sourceRenderer->setVolume( volume );<br>+    }<br> }<br> <br> void<br>diff --git a/src/Renderer/WorkflowRenderer.cpp b/src/Renderer/WorkflowRenderer.cpp<br>index 24a9b55..dd2f88a 100644<br>--- a/src/Renderer/WorkflowRenderer.cpp<br>+++ b/src/Renderer/WorkflowRenderer.cpp<br>@@ -277,7 +277,11 @@ WorkflowRenderer::getVolume() const<br> void WorkflowRenderer::setVolume( int volume )<br> {<br>     //Returns 0 if the volume was set, -1 if it was out of range<br>-    m_sourceRenderer->setVolume( volume );<br>+    //Handles the 'Project Preview Window' VolSlider Crash when media file is not loaded<br>+    if(m_sourceRenderer != NULL)<br>+    {<br>+    m_sourceRenderer->setVolume( volume );<br>+    }<br> }<br> <br> qint64<br>-- <br>1.9.1<br><br></div>