Changeset 2986

Timestamp:

28/08/2011 16:37:31 (9 months ago)

Author:

chris

Message:

Fix incorrect "conversion failed" message when logging NULL VSS parameter.

Fix log level of VSS component type to TRACE instead of INFO.

Fix VSS path generation for absolute paths including drive letters.

File:

• box/trunk/bin/bbackupd/BackupDaemon.cpp (modified) (3 diffs)

box/trunk/bin/bbackupd/BackupDaemon.cpp

@@ -119 +119 @@
                 std::string WideStringToString(WCHAR *buf)
                 {
+                        if (buf == NULL)
+                        {
+                                return "(null)";
+                        }
+
                         char* pStr = ConvertFromWideString(buf, CP_UTF8);
                         
                         if(pStr == NULL)
                         {
-                                return "conversion failed";
+                                return "(conversion failed)";
                         }
                         
@@ -1251 +1256 @@
                         switch(pComponentInfo->type)
                         {
-                        case VSS_CT_UNDEFINED: BOX_INFO("VSS: type: undefined"); break;
-                        case VSS_CT_DATABASE:  BOX_INFO("VSS: type: database"); break;
-                        case VSS_CT_FILEGROUP: BOX_INFO("VSS: type: filegroup"); break;
+                        case VSS_CT_UNDEFINED: BOX_TRACE("VSS: type: undefined"); break;
+                        case VSS_CT_DATABASE:  BOX_TRACE("VSS: type: database"); break;
+                        case VSS_CT_FILEGROUP: BOX_TRACE("VSS: type: filegroup"); break;
                         default:
                                 BOX_WARNING("VSS: type: unknown (" << pComponentInfo->type << ")");
@@ -1335 +1340 @@
                                         rLocation.mSnapshotVolumeId = newVolumeId;
                                         rLocation.mIsSnapshotCreated = true;
+
+                                        // If the snapshot path starts with the volume root
+                                        // (drive letter), because the path is absolute (as
+                                        // it should be), then remove it so that the
+                                        // resulting snapshot path can be appended to the
+                                        // snapshot device object to make a real path,
+                                        // without a spurious drive letter in it.
+
+                                        if (path.substr(0, volumeRoot.length()) == volumeRoot)
+                                        {
+                                                path = path.substr(volumeRoot.length());
+                                        }
+
                                         rLocation.mSnapshotPath = path;
                                 }