Ticket #13635: patch

cdrip.cpp

@@ -325 +325 @@
                                                   titleTrack));
                 }
 
-                if (!encoder->isValid())
+                if (!encoder)
+                {
+                    // This should never happen.
+                    QApplication::postEvent(
+                        m_parent,
+                        new RipStatusEvent(RipStatusEvent::kEncoderErrorEvent,
+                                           "Failed to create encoder"));
+                    LOG(VB_GENERAL, LOG_ERR, "MythMusic: No encoder, failing");
+                    RunEpilog();
+                    return;
+                }
+                else if (!encoder->isValid())
                 {
                     QApplication::postEvent(
                         m_parent,
@@ -340 +351 @@
                 }
             }
 
-            if (!encoder)
+            if (encoder)
             {
-                // This should never happen.
-                QApplication::postEvent(
-                    m_parent,
-                    new RipStatusEvent(RipStatusEvent::kEncoderErrorEvent,
-                                       "Failed to create encoder"));
-                LOG(VB_GENERAL, LOG_ERR, "MythMusic: No encoder, failing");
-                RunEpilog();
-                return;
+                ripTrack(m_cdDevice, encoder.get(), trackno + 1);
             }
-            ripTrack(m_cdDevice, encoder.get(), trackno + 1);
 
             if (isCancelled())
             {