fix a few SIGSEGVs related to not checking the value of iface, if we

are capturing from a file (DEVTYPE_FILE)

1 files changed, 5 insertions, 3 deletions

diff --git a/noncore/net/wellenreiter/gui/wellenreiter.cpp b/noncore/net/wellenreiter/gui/wellenreiter.cpp
index 14f251a..405eda8 100644
--- a/noncore/net/wellenreiter/gui/wellenreiter.cpp
+++ b/noncore/net/wellenreiter/gui/wellenreiter.cpp
@@ -385,6 +385,8 @@ void Wellenreiter::startClicked()
     iface = static_cast<OWirelessNetworkInterface*>(net->interface( interface )); // fails if network is not wireless!
 
     // bring device UP
+    if ( cardtype != DEVTYPE_FILE )
+    {
     iface->setUp( true );
     if ( !iface->isUp() )
     {
@@ -392,7 +394,7 @@ void Wellenreiter::startClicked()
                             tr( "Can't bring interface '%1' up:\n" ).arg( iface->name() ) + strerror( errno ) );
         return;
     }
-
+    }
     // set monitor mode
     bool usePrism = configwindow->usePrismHeader();
 
@@ -454,8 +456,8 @@ void Wellenreiter::startClicked()
 
     if ( !pcap->isOpen() )
     {
-        QMessageBox::warning( this, "Wellenreiter II",
-                              tr( "Can't open packet capturer for '%1':\n" ).arg( iface->name() ) + QString(strerror( errno ) ));
+        QMessageBox::warning( this, "Wellenreiter II", tr( "Can't open packet capturer for\n'%1':\n" ).arg(
+                              cardtype == DEVTYPE_FILE ? (const char*) interface : iface->name() ) + QString(strerror( errno ) ));
         return;
     }