[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    extragear/multimedia/amarok/src/engine/xine
From:       Andrew Turner <andrewturner512+kdebugs () googlemail ! com>
Date:       2006-10-15 23:41:09
Message-ID: 1160955669.994538.16647.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 595866 by aturner:

Sanity check - the function can still get called by App::applySettings (currently line 602) if the \
equalizer is meant to be enabled, but there isn't a stream. This can lead to xine_set_param being called \
with a null pointer.


 M  +3 -0      xine-engine.cpp  


--- trunk/extragear/multimedia/amarok/src/engine/xine/xine-engine.cpp #595865:595866
@@ -559,6 +559,9 @@
 void
 XineEngine::setEqualizerParameters( int preamp, const QValueList<int> &gains )
 {
+    if ( !m_stream )
+        return;
+
     m_equalizerGains = gains;
     m_intPreamp = preamp;
     QValueList<int>::ConstIterator it = gains.begin();


[prev in list] [next in list] [prev in thread] [next in thread]