A small patch

Support specifying client working directory
Clean all compile warnings

Author: wh201906

README.md

@@ -72,6 +72,7 @@ Great thanks to him.
 + Support configuring environment variables by script  
 (Useful when the client requires specific environment variables)  
 + All functions are compatible with Iceman/RRG repo(tested on v4.9237)  
++ Support specifying client working directory
 + Fix some bugs
 
 ### V0.1.3

README/doc/README_zh_CN.md

@@ -71,6 +71,7 @@ release页面中有含客户端的GUI。这个GUI也可以搭配你自己的客
 + 可通过外部脚本配置环境变量  
 (在客户端需要配置环境变量时很有用)  
 + 全功能兼容冰人版（在v4.9237上测试通过）  
++ 支持指定客户端工作路径
 + 修复部分bug
 
 ### V0.1.3

common/pm3process.cpp

@@ -55,7 +55,11 @@ void PM3Process::connectPM3(const QString& path, const QString& port, const QStr
             result = result.left(result.indexOf("\r\n"));
             result = result.mid(3, result.lastIndexOf(" ") - 3);
             emit PM3StatedChanged(true, result);
-            setSerialListener(port, true);
+
+            // if the arguments don't contain <port>, then disable the port listener
+            // useful when using offline sniff
+            if(args.indexOf(port) != -1)
+                setSerialListener(port, true);
         }
         else
             kill();
@@ -132,5 +136,11 @@ void PM3Process::setProcEnv(const QStringList* env)
 {
 //    qDebug() << "passed Env List" << *env;
     this->setEnvironment(*env);
-//    qDebug() << "final Env List" << processEnvironment().toStringList();
+    //    qDebug() << "final Env List" << processEnvironment().toStringList();
+}
+
+void PM3Process::setWorkingDir(const QString& dir)
+{
+    // the working directory cannot be the default, or the client will failed to load the dll
+    this->setWorkingDirectory(dir);
 }

common/pm3process.h

@@ -9,6 +9,7 @@
 #include <QtSerialPort/QSerialPortInfo>
 #include <QtSerialPort/QSerialPort>
 #include <QProcessEnvironment>
+#include <QDir>
 
 #include "util.h"
 
@@ -27,6 +28,7 @@ public slots:
     qint64 write(QString data);
     void reconnectPM3();
     void setProcEnv(const QStringList* env);
+    void setWorkingDir(const QString& dir);
 private slots:
     void onTimeout();
     void onReadyRead();

lang/en_US.ts

@@ -102,16 +102,14 @@ QString Mifare::info(bool isRequiringOutput)
                 end = result.indexOf("\n", end);
                 return result.mid(begin, end - begin + 1);
             }
-            else
-                return "";
         }
         else
         {
             util->execCMD("hf 14a info");
             ui->funcTab->setCurrentIndex(Util::rawTabIndex);
-            return "";
         }
     }
+    return "";
 }
 
 void Mifare::chk()
@@ -629,6 +627,7 @@ bool Mifare::_writeblk(int blockId, KeyType keyType, const QString& key, const Q
             return true;
         }
     }
+    return false;
 }
 
 void Mifare::writeOne(TargetType targetType)

lang/zh_CN.qm

@@ -28,6 +28,7 @@ MainWindow::MainWindow(QWidget *parent):
     pm3 = new PM3Process(pm3Thread);
     pm3Thread->start();
     pm3state = false;
+    clientWorkingDir = new QDir;
 
     util = new Util(this);
     mifare = new Mifare(ui, util, this);
@@ -94,6 +95,7 @@ void MainWindow::on_PM3_connectButton_clicked()
     {
         QStringList args = ui->Set_Client_startArgsEdit->text().replace("<port>", port).split(' ');
         saveClientPath(ui->PM3_pathEdit->text());
+
         QProcess envSetProcess;
         QFileInfo envScriptPath(ui->Set_Client_envScriptEdit->text());
         if(envScriptPath.exists())
@@ -113,6 +115,15 @@ void MainWindow::on_PM3_connectButton_clicked()
         else
             clientEnv.clear();
         emit setProcEnv(&clientEnv);
+
+        clientWorkingDir->setPath(QApplication::applicationDirPath());
+        qDebug() << clientWorkingDir->absolutePath();
+        clientWorkingDir->mkpath(ui->Set_Client_workingDirEdit->text());
+        qDebug() << clientWorkingDir->absolutePath();
+        clientWorkingDir->cd(ui->Set_Client_workingDirEdit->text());
+        qDebug() << clientWorkingDir->absolutePath();
+        emit setWorkingDir(clientWorkingDir->absolutePath());
+
         emit connectPM3(ui->PM3_pathEdit->text(), port, args);
     }
 }
@@ -976,6 +987,7 @@ void MainWindow::uiInit()
 
     settings->beginGroup("Client_Env");
     ui->Set_Client_envScriptEdit->setText(settings->value("scriptPath").toString());
+    ui->Set_Client_workingDirEdit->setText(settings->value("workingDir", "../data").toString());
     settings->endGroup();
 
     ui->MF_RW_keyTypeBox->addItem("A", Mifare::KEY_A);
@@ -996,6 +1008,7 @@ void MainWindow::signalInit()
     connect(pm3, &PM3Process::PM3StatedChanged, util, &Util::setRunningState);
     connect(this, &MainWindow::killPM3, pm3, &PM3Process::kill);
     connect(this, &MainWindow::setProcEnv, pm3, &PM3Process::setProcEnv);
+    connect(this, &MainWindow::setWorkingDir, pm3, &PM3Process::setWorkingDir);
 
     connect(util, &Util::write, pm3, &PM3Process::write);
 
@@ -1160,3 +1173,10 @@ void MainWindow::on_Set_Client_envScriptEdit_editingFinished()
     settings->setValue("scriptPath", ui->Set_Client_envScriptEdit->text());
     settings->endGroup();
 }
+
+void MainWindow::on_Set_Client_saveWorkingDirButton_clicked()
+{
+    settings->beginGroup("Client_Env");
+    settings->setValue("workingDir", ui->Set_Client_workingDirEdit->text());
+    settings->endGroup();
+}

lang/zh_CN.ts

@@ -173,6 +173,8 @@ private slots:
 
     void on_Set_Client_envScriptEdit_editingFinished();
 
+    void on_Set_Client_saveWorkingDirButton_clicked();
+
 private:
     Ui::MainWindow* ui;
     QButtonGroup* MFCardTypeBtnGroup;
@@ -199,6 +201,7 @@ private slots:
     QTimer* portSearchTimer;
     QStringList portList;
     QStringList clientEnv;
+    QDir* clientWorkingDir;
 
     Mifare* mifare;
     Util* util;
@@ -217,5 +220,6 @@ private slots:
     void killPM3();
     void setSerialListener(const QString& name, bool state);
     void setProcEnv(const QStringList *env);
+    void setWorkingDir(const QString& dir);
 };
 #endif // MAINWINDOW_H