Changeset 89139 in vbox for trunk/src/VBox/Devices

Timestamp:

May 18, 2021 1:29:24 PM (4 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

144480

Message:

Audio/ValKit: Implemented unpacking and verification of packed audio test archives. bugref:10008

Location:

trunk/src/VBox/Devices/Audio

Files:

• AudioTest.cpp (modified) (6 diffs)
• AudioTest.h (modified) (2 diffs)

trunk/src/VBox/Devices/Audio/AudioTest.cpp

@@ -55 +55 @@
 /** The current test manifest version. */
 #define AUDIOTEST_MANIFEST_VER      1
+/** Audio test archive default suffix.
+ *  According to IPRT terminology this always contains the dot. */
+#define AUDIOTEST_ARCHIVE_SUFF_STR  ".tar.gz"
 
 /** Test manifest header name. */
@@ -779 +782 @@
         return VINF_SUCCESS;
 
+    if (!RTFileIsValid(pSet->f.hFile))
+        return VINF_SUCCESS;
+
     /* Update number of ran tests. */
     int rc = RTFileSeek(pSet->f.hFile, pSet->offTestCount, RTFILE_SEEK_BEGIN, NULL);
@@ -785 +791 @@
     AssertRCReturn(rc, rc);
 
-    if (RTFileIsValid(pSet->f.hFile))
-    {
-        RTFileClose(pSet->f.hFile);
-        pSet->f.hFile = NIL_RTFILE;
-    }
+    RTFileClose(pSet->f.hFile);
+    pSet->f.hFile = NIL_RTFILE;
 
     return rc;
@@ -1033 +1036 @@
 
     char szOutName[RT_ELEMENTS(AUDIOTEST_PATH_PREFIX_STR) + AUDIOTEST_TAG_MAX + 16];
-    if (RTStrPrintf2(szOutName, sizeof(szOutName), "%s-%s.tar.gz", AUDIOTEST_PATH_PREFIX_STR, pSet->szTag) <= 0)
+    if (RTStrPrintf2(szOutName, sizeof(szOutName), "%s-%s%s",
+                     AUDIOTEST_PATH_PREFIX_STR, pSet->szTag, AUDIOTEST_ARCHIVE_SUFF_STR) <= 0)
         AssertFailedReturn(VERR_BUFFER_OVERFLOW);
 
@@ -1066 +1070 @@
 
 /**
+ * Returns whether a test set archive is packed (as .tar.gz by default) or
+ * a plain directory.
+ *
+ * @returns \c true if packed (as .tar.gz), or \c false if not (directory).
+ * @param   pszPath             Path to return packed staus for.
+ */
+bool AudioTestSetIsPacked(const char *pszPath)
+{
+    /** @todo Improve this, good enough for now. */
+    return (RTStrIStr(pszPath, AUDIOTEST_ARCHIVE_SUFF_STR) != NULL);
+}
+
+/**
  * Unpacks a formerly packed audio test set.
  *
  * @returns VBox status code.
- * @param   pszFile             Test set file to unpack.
+ * @param   pszFile             Test set file to unpack. Must contain the absolute path.
  * @param   pszOutDir           Directory where to unpack the test set into.
  *                              If the directory does not exist it will be created.
@@ -1075 +1092 @@
 int AudioTestSetUnpack(const char *pszFile, const char *pszOutDir)
 {
-    RT_NOREF(pszFile, pszOutDir);
-
-    // RTZipTarCmd()
-
-    return VERR_NOT_IMPLEMENTED;
+    AssertReturn(pszFile && pszOutDir, VERR_INVALID_PARAMETER);
+
+    int rc = VINF_SUCCESS;
+
+    if (!RTDirExists(pszOutDir))
+    {
+        rc = RTDirCreateFullPath(pszOutDir, 0755);
+        if (RT_FAILURE(rc))
+            return rc;
+    }
+
+    const char *apszArgs[8];
+    unsigned    cArgs = 0;
+
+    apszArgs[cArgs++] = "AudioTest";
+    apszArgs[cArgs++] = "--extract";
+    apszArgs[cArgs++] = "--gunzip";
+    apszArgs[cArgs++] = "--directory";
+    apszArgs[cArgs++] = pszOutDir;
+    apszArgs[cArgs++] = "--file";
+    apszArgs[cArgs++] = pszFile;
+
+    RTEXITCODE rcExit = RTZipTarCmd(cArgs, (char **)apszArgs);
+    if (rcExit != RTEXITCODE_SUCCESS)
+        rc = VERR_GENERAL_FAILURE; /** @todo Fudge! */
+
+    return rc;
 }
 

trunk/src/VBox/Devices/Audio/AudioTest.h

@@ -27 +27 @@
 
 /** Maximum length in characters an audio test tag can have. */
-#define AUDIOTEST_TAG_MAX          64
+#define AUDIOTEST_TAG_MAX               64
 /** Maximum length in characters a single audio test error description can have. */
-#define AUDIOTEST_ERROR_DESC_MAX   128
+#define AUDIOTEST_ERROR_DESC_MAX        128
 /** Prefix for audio test (set) directories. */
-#define AUDIOTEST_PATH_PREFIX_STR "vkat"
+#define AUDIOTEST_PATH_PREFIX_STR       "vkat"
 
 /**
@@ -324 +324 @@
 int    AudioTestSetClose(PAUDIOTESTSET pSet);
 int    AudioTestSetWipe(PAUDIOTESTSET pSet);
+bool   AudioTestSetIsPacked(const char *pszPath);
 int    AudioTestSetPack(PAUDIOTESTSET pSet, const char *pszOutDir, char *pszFileName, size_t cbFileName);
 int    AudioTestSetUnpack(const char *pszFile, const char *pszOutDir);