Changeset 33484 in vbox for trunk

Timestamp:

Oct 27, 2010 8:44:01 AM (14 years ago)

Author:

vboxsync

Message:

Runtime: force zero terminated strings on file read

File:

• trunk/src/VBox/Runtime/common/misc/tar.cpp (modified) (1 diff)

trunk/src/VBox/Runtime/common/misc/tar.cpp

@@ -206 +206 @@
     rc = RTStrToUInt32Full(pRecord->h.chksum, 8, &sum);
     if (RT_SUCCESS(rc) && sum == check)
-        return VINF_SUCCESS;
-    return VERR_TAR_CHKSUM_MISMATCH;
+    {
+        /* Make sure the strings are zero terminated. */
+        pRecord->h.name[sizeof(pRecord->h.name) - 1]         = 0;
+        pRecord->h.linkname[sizeof(pRecord->h.linkname) - 1] = 0;
+        pRecord->h.magic[sizeof(pRecord->h.magic) - 1]       = 0;
+        pRecord->h.uname[sizeof(pRecord->h.uname) - 1]       = 0;
+        pRecord->h.gname[sizeof(pRecord->h.gname) - 1]       = 0;
+    }
+    else
+        rc = VERR_TAR_CHKSUM_MISMATCH;
+
+    return rc;
 }