Handle backupDate of zero

In older versions of OpenAFS (prior to 2001), the backupDate was
never set.  Try to provide somewhat more reasonable behavior in
this case, by using a different date in that case.

Reviewed-on: http://gerrit.openafs.org/11627
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
(cherry picked from commit 34e495d69a8831c57cac2ccf18898e63f02c7745)

Change-Id: Ia8250ed6e699d2ed8e2b5de7481c54ae65b0718c
Reviewed-on: http://gerrit.openafs.org/11826
Reviewed-by: Daria Phoebe Brashear <shadow@your-file-system.com>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>

diff --git a/src/volser/dumpstuff.c b/src/volser/dumpstuff.c
index a3e8c7ae53f3b84fca9d0c1a37fb715bbcaf9a48..915b70a378772f98f6107d1592e17be1f42827aa 100644 (file)
--- a/src/volser/dumpstuff.c
+++ b/src/volser/dumpstuff.c
@@ -1048,7 +1048,9 @@ DumpDumpHeader(struct iod *iodp, Volume * vp,
        dumpTimes[1] = V_creationDate(vp);      /* until clone was updated */
        break;
     case backupVolume:
-       dumpTimes[1] = V_backupDate(vp);        /* until backup was made */
+       /* until backup was made */
+       dumpTimes[1] = V_backupDate(vp) != 0 ? V_backupDate(vp) :
+                                              V_creationDate(vp);
        break;
     default:
        code = EINVAL;

diff --git a/src/volser/vol-dump.c b/src/volser/vol-dump.c
index fc5f62e54c9eb745a2278af704bb943373a16b52..0820e2544395182122d4513315b58d0549651a1e 100644 (file)
--- a/src/volser/vol-dump.c
+++ b/src/volser/vol-dump.c
@@ -418,7 +418,9 @@ DumpDumpHeader(int dumpfd, Volume * vp, afs_int32 fromtime)
        dumpTimes[1] = V_creationDate(vp);      /* until clone was updated */
        break;
     case backupVolume:
-       dumpTimes[1] = V_backupDate(vp);        /* until backup was made */
+       /* until backup was made */
+       dumpTimes[1] = V_backupDate(vp) != 0 ? V_backupDate(vp) :
+                                              V_creationDate(vp);
        break;
     default:
        code = EINVAL;