Merge pull request #70 from darkgeem/master

Fixed latest versions of proxmark3 not working.

README.md

@@ -82,6 +82,8 @@ open Proxmark3GUI.app
 
 > In order for the GUI to connect to the device in macOS, you'd need to tweak the settings a little bit
 
+Client Path must be path to pm3 console client like "/usr/local/bin/pm3/"
+
 ![macOS_settings](doc/README/macOS_settings.png)
 
 ***

src/common/pm3process.cpp

@@ -22,6 +22,7 @@ void PM3Process::connectPM3(const QString& path, const QStringList args)
     QString result;
     Util::ClientType clientType;
     setRequiringOutput(true);
+	QRegularExpression osPattern("(os:\\s+|OS\\.+\\s+)");
 
     // stash for reconnect
     currPath = path;
@@ -35,30 +36,35 @@ void PM3Process::connectPM3(const QString& path, const QStringList args)
         waitForReadyRead(10000);
         setRequiringOutput(false);
         result = *requiredOutput;
-        // Workaround for wayland system, e.g. Ubuntu 22.04
-        // The issue is that the warning is read and nothing else, hence the process is killed.
-        if(result.contains("Warning: Ignoring XDG_SESSION_TYPE=wayland on Gnome. Use QT_QPA_PLATFORM=wayland to run on Wayland anyway."))
-        {
-            setRequiringOutput(true);
-            readWaitForConnection(&result);
-        }
         if(result.contains("[=]"))
         {
             clientType = Util::CLIENTTYPE_ICEMAN;
             setRequiringOutput(true);
             write("hw version\n");
-            readWaitForConnection(&result);
+            for(int i = 0; i < 50; i++)
+            {
+                waitForReadyRead(200);
+                result += *requiredOutput;
+                // if(result.contains("os: "))
+                if(osPattern.match(result).hasMatch())
+                    break;
+            }
+            setRequiringOutput(false);
         }
         else
         {
             clientType = Util::CLIENTTYPE_OFFICIAL;
         }
-        if(result.contains("os: ")) // make sure the PM3 is connected
+        // if(result.contains("os: ")) // make sure the PM3 is connected
+		if(osPattern.match(result).hasMatch())
         {
             emit changeClientType(clientType);
-            result = result.mid(result.indexOf("os: "));
+            // result = result.mid(result.indexOf("os: "));
+			QRegularExpressionMatch osMatch = osPattern.match(result);
+			result = result.mid(osMatch.capturedStart());
             result = result.left(result.indexOf("\n"));
-            result = result.mid(4, result.indexOf(" ", 4) - 4);
+            // result = result.mid(4, result.indexOf(" ", 4) - 4);
+			result = result.mid(osMatch.capturedLength(), result.indexOf(" ", osMatch.capturedLength()) - osMatch.capturedLength());
             emit PM3StatedChanged(true, result);
         }
         else
@@ -72,18 +78,6 @@ void PM3Process::connectPM3(const QString& path, const QStringList args)
     setRequiringOutput(false);
 }
 
-void PM3Process::readWaitForConnection(QString *result)
-{
-    for(int i = 0; i < 50; i++)
-    {
-        waitForReadyRead(200);
-        (*result) += *requiredOutput;
-        if(result->contains("os: "))
-            break;
-    }
-    setRequiringOutput(false);
-}
-
 void PM3Process::reconnectPM3()
 {
     connectPM3(currPath, currArgs);

src/common/pm3process.h

@@ -38,7 +38,6 @@ private:
     bool isRequiringOutput;
     QString* requiredOutput; // It only works in this class now
     void setRequiringOutput(bool st);// It only works in this class now
-    void readWaitForConnection(QString *result);
     QTimer* serialListener;
     QSerialPortInfo* portInfo;
     QString currPath;