Changeset 2401

Timestamp:

05/12/2008 20:56:00 (15 months ago)

Author:

chris

Message:

Catch exceptions from BackupClientFileAttributes::ReadAttributes?
during backup, and report which file caused them.

If we cannot read the attributes of a directory to find its inode
number, assume that it hasn't been renamed.

Reformat "Storing uploaded file ID" log line to shorten source code
lines.

Change "Storing uploaded file ID" and "Found conflicting parent ID"
messages not to print the path, which duplicates data earlier in the
same message.

Fix typo in comment.

Files:

• box/trunk/bin/bbackupd/BackupClientDirectoryRecord.cpp (modified) (8 diffs)

box/trunk/bin/bbackupd/BackupClientDirectoryRecord.cpp

@@ -1040 +1040 @@
                         if(!rContext.StorageLimitExceeded())
                         {
-                                // Update store
-                                BackupClientFileAttributes attr;
-                                attr.ReadAttributes(filename.c_str(), false /* put mod times in the attributes, please */);
-                                MemBlockStream attrStream(attr);
-                                connection.QuerySetReplacementFileAttributes(mObjectID, attributesHash, storeFilename, attrStream);
-                                fileSynced = true;
+                                try
+                                {
+                                        // Update store
+                                        BackupClientFileAttributes attr;
+                                        attr.ReadAttributes(filename.c_str(), false /* put mod times in the attributes, please */);
+                                        MemBlockStream attrStream(attr);
+                                        connection.QuerySetReplacementFileAttributes(mObjectID, attributesHash, storeFilename, attrStream);
+                                        fileSynced = true;
+                                }
+                                catch (BoxException &e)
+                                {
+                                        BOX_ERROR("Failed to read or store "
+                                                "file attributes for '" <<
+                                                filename << "', will try "
+                                                "again later");
+                                }
                         }
                 }
@@ -1087 +1097 @@
                         {
                                 // Use this one
-                                BOX_TRACE("Storing uploaded file ID " << inodeNum << " (" << filename << ") in ID map as object " << latestObjectID << " with parent " << mObjectID << " (" << rLocalPath << ")");
+                                BOX_TRACE("Storing uploaded file ID " <<
+                                        inodeNum << " (" << filename << ") "
+                                        "in ID map as object " <<
+                                        latestObjectID << " with parent " <<
+                                        mObjectID);
                                 idMap.AddToMap(inodeNum, latestObjectID, mObjectID /* containing directory */);
                         }
@@ -1101 +1115 @@
                                         if (dirid != mObjectID)
                                         {
-                                                BOX_WARNING("Found conflicting parent ID for file ID " << inodeNum << " (" << filename << "): expected " << mObjectID << " (" << rLocalPath << ") but found " << dirid << " (same directory used in two different locations?)");
+                                                BOX_WARNING("Found conflicting parent ID for file ID " << inodeNum << " (" << filename << "): expected " << mObjectID << " but found " << dirid << " (same directory used in two different locations?)");
                                         }
 
@@ -1111 +1125 @@
                                         // not indicate anything wrong.
                                         // Run the release version for real life use, where this check is not made.
-                                        BOX_TRACE("Storing found file ID " << inodeNum << " (" << filename << ") in ID map as object " << latestObjectID << " with parent " << mObjectID << " (" << rLocalPath << ")");
-                                        idMap.AddToMap(inodeNum, objid, mObjectID /* containing directory */);                          
+                                        BOX_TRACE("Storing found file ID " <<
+                                                inodeNum << " (" << filename <<
+                                                ") in ID map as object " <<
+                                                objid << " with parent " <<
+                                                mObjectID);
+                                        idMap.AddToMap(inodeNum, objid,
+                                                mObjectID /* containing directory */);
                                 }
                         }
@@ -1173 +1192 @@
 
                 // Flag for having created directory, so can optimise the
-                // recusive call not to read it again, because we know
+                // recursive call not to read it again, because we know
                 // it's empty.
                 bool haveJustCreatedDirOnServer = false;
@@ -1179 +1198 @@
                 // Next, see if it's in the list of sub directories
                 BackupClientDirectoryRecord *psubDirRecord = 0;
-                std::map<std::string, BackupClientDirectoryRecord *>::iterator e(mSubDirectories.find(*d));
+                std::map<std::string, BackupClientDirectoryRecord *>::iterator
+                        e(mSubDirectories.find(*d));
+
                 if(e != mSubDirectories.end())
                 {
@@ -1219 +1240 @@
                                 InodeRefType inodeNum = 0;
                                 BackupClientFileAttributes attr;
-                                attr.ReadAttributes(dirname.c_str(), true /* directories have zero mod times */,
-                                        0 /* not interested in mod time */, &attrModTime, 0 /* not file size */,
-                                        &inodeNum);
+                                bool failedToReadAttributes = false;
+
+                                try
+                                {
+                                        attr.ReadAttributes(dirname.c_str(),
+                                                true /* directories have zero mod times */,
+                                                0 /* not interested in mod time */,
+                                                &attrModTime, 0 /* not file size */,
+                                                &inodeNum);
+                                }
+                                catch (BoxException &e)
+                                {
+                                        BOX_WARNING("Failed to read attributes "
+                                                "of directory, cannot check "
+                                                "for rename, assuming new: '"
+                                                << dirname << "'");
+                                        failedToReadAttributes = true;
+                                }
 
                                 // Check to see if the directory been renamed
@@ -1229 +1265 @@
                                 const BackupClientInodeToIDMap &idMap(
                                         rContext.GetCurrentIDMap());
-                                if(idMap.Lookup(inodeNum, renameObjectID, renameInDirectory))
+
+                                if(!failedToReadAttributes && idMap.Lookup(inodeNum,
+                                        renameObjectID, renameInDirectory))
                                 {
                                         // Look up on the server to get the name, to build the local filename