* Copyright date update
[BackupPC.git] / bin / BackupPC_restore
index c01f12c..856a0a5 100755 (executable)
@@ -1,17 +1,17 @@
-#!/bin/perl -T
+#!/usr/bin/perl
 #============================================================= -*-perl-*-
 #
 # BackupPC_restore: Restore files to a client.
 #
 # DESCRIPTION
 #
-#   Usage: BackupPC_restore <hostIP> <host> <reqFileName>
+#   Usage: BackupPC_restore <hostIP> <client> <reqFileName>
 #
 # AUTHOR
 #   Craig Barratt  <cbarratt@users.sourceforge.net>
 #
 # COPYRIGHT
-#   Copyright (C) 2001  Craig Barratt
+#   Copyright (C) 2001-2009  Craig Barratt
 #
 #   This program is free software; you can redistribute it and/or modify
 #   it under the terms of the GNU General Public License as published by
 #
 #========================================================================
 #
-# Version 1.6.0_CVS, released 10 Dec 2002.
+# Version 3.2.0beta0, released 5 April 2009.
 #
 # See http://backuppc.sourceforge.net.
 #
 #========================================================================
 
 use strict;
+no  utf8;
 use lib "/usr/local/BackupPC/lib";
 use BackupPC::Lib;
 use BackupPC::FileZIO;
-use BackupPC::Xfer::Smb;
-use BackupPC::Xfer::Tar;
-use BackupPC::Xfer::Rsync;
+use BackupPC::Xfer;
+use Socket;
 
 use File::Path;
 use Getopt::Std;
@@ -58,35 +58,28 @@ my $BinDir = $bpc->BinDir();
 my %Conf   = $bpc->Conf();
 my $NeedPostCmd;
 
-my($hostIP, $host, $reqFileName);
+my($hostIP, $host, $client, $reqFileName, %stat);
 
 $bpc->ChildInit();
 
 if ( @ARGV != 3 ) {
-    print("usage: $0 <hostIP> <host> <reqFileName>\n");
+    print("usage: $0 <hostIP> <client> <reqFileName>\n");
     exit(1);
 }
 $hostIP = $1 if ( $ARGV[0] =~ /(.+)/ );
-$host   = $1 if ( $ARGV[1] =~ /(.+)/ );
+$client = $1 if ( $ARGV[1] =~ /(.+)/ );
 if ( $ARGV[2] !~ /^([\w.]+)$/ ) {
     print("$0: bad reqFileName (arg #3): $ARGV[2]\n");
     exit(1);
 }
 $reqFileName = $1;
 
-my $Hosts = $bpc->HostInfoRead();
+my $startTime = time();
 
-#
-# Re-read config file, so we can include the PC-specific config
-#
-if ( defined(my $error = $bpc->ConfigRead($host)) ) {
-    print("Can't read PC's config file: $error\n");
-    exit(1);
-}
-%Conf = $bpc->Conf();
+my $Hosts = $bpc->HostInfoRead($client);
 
-my $Dir     = "$TopDir/pc/$host";
-my $xferPid = -1;
+my $Dir     = "$TopDir/pc/$client";
+my @xferPid = ();
 my $tarPid  = -1;
 
 #
@@ -95,67 +88,136 @@ my $tarPid  = -1;
 $SIG{INT}  = \&catch_signal;
 $SIG{ALRM} = \&catch_signal;
 $SIG{TERM} = \&catch_signal;
+$SIG{PIPE} = \&catch_signal;
+$SIG{STOP} = \&catch_signal;
+$SIG{TSTP} = \&catch_signal;
+$SIG{TTIN} = \&catch_signal;
+my $Pid = $$;
+
+mkpath($Dir, 0, 0777) if ( !-d $Dir );
+if ( !-f "$Dir/LOCK" ) {
+    open(LOCK, ">", "$Dir/LOCK") && close(LOCK);
+}
+
+my($sec,$min,$hour,$mday,$mon,$year,$wday,$yday,$isdst) = localtime(time);
+my $logPath = sprintf("$Dir/LOG.%02d%04d", $mon + 1, $year + 1900);
+
+if ( !-f $logPath ) {
+    #
+    # Compress and prune old log files
+    #
+    my $lastLog = $Conf{MaxOldPerPCLogFiles} - 1;
+    foreach my $file ( $bpc->sortedPCLogFiles($client) ) {
+        if ( $lastLog <= 0 ) {
+            unlink($file);
+            next;
+        }
+        $lastLog--;
+        next if ( $file =~ /\.z$/ || !$Conf{CompressLevel} );
+        BackupPC::FileZIO->compressCopy($file,
+                                        "$file.z",
+                                        undef,
+                                        $Conf{CompressLevel}, 1);
+    }
+}
+open(LOG, ">>", $logPath);
+select(LOG); $| = 1; select(STDOUT);
 
 #
 # Read the request file
 #
 if ( !(my $ret = do "$Dir/$reqFileName") ) {
-   die "couldn't parse $Dir/$reqFileName: $@" if $@;
-   die "couldn't do $Dir/$reqFileName: $!"    unless defined $ret;
-   die "couldn't run $Dir/$reqFileName";
+    my $err;
+    if ( $@ ) {
+        $err = "couldn't parse $Dir/$reqFileName: $@";
+    } elsif ( !defined($ret) ) {
+       $err = "couldn't do $Dir/$reqFileName: $!";
+    } else {
+       $err = "couldn't run $Dir/$reqFileName";
+    }
+    $stat{hostError} = $err;
+    exit(RestoreCleanup($client));
 }
 
+#
+# Re-read config file, so we can include the PC-specific config
+#
+if ( defined(my $error = $bpc->ConfigRead($client)) ) {
+    $stat{hostError} = "Can't read PC's config file: $error";
+    exit(RestoreCleanup($client));
+}
+%Conf = $bpc->Conf();
+
 #
 # Make sure we eventually timeout if there is no activity from
 # the data transport program.
 #
 alarm($Conf{ClientTimeout});
 
-mkpath($Dir, 0, 0777) if ( !-d $Dir );
-if ( !-f "$Dir/LOCK" ) {
-    open(LOCK, ">$Dir/LOCK") && close(LOCK);
+#
+# See if the host name is aliased
+#
+if ( $Conf{ClientNameAlias} ne "" ) {
+    $host = $Conf{ClientNameAlias};
+} else {
+    $host = $client;
+}
+
+#
+# Find its IP address
+#
+if ( $hostIP !~ /\d+\.\d+\.\d+\.\d+/ ) {
+    if ( !defined(gethostbyname($host)) ) {
+       #
+       # Ok, NS doesn't know about it.  Maybe it is a NetBios name
+       # instead.
+       #
+       if ( !defined($hostIP = $bpc->NetBiosHostIPFind($host)) ) {
+           $stat{hostError} = "Can't find host $host";
+           exit(RestoreCleanup($client));
+       }
+    } else {
+       $hostIP = $host;
+    }
 }
-open(LOG, ">>$Dir/LOG");
-select(LOG); $| = 1; select(STDOUT);
 
 #
 # Check if $host is alive
 #
 my $delay = $bpc->CheckHostAlive($hostIP);
 if ( $delay < 0 ) {
-    print(LOG $bpc->timeStamp, "no ping response\n");
-    print("no ping response\n");
-    exit(1);
+    $stat{hostError} = "no ping response from $host ($hostIP)";
+    exit(RestoreCleanup($client));
 } elsif ( $delay > $Conf{PingMaxMsec} ) {
-    printf(LOG "%sping too slow: %.4gmsec\n", $bpc->timeStamp, $delay);
-    printf("ping too slow: %.4gmsec (threshold is %gmsec)\n",
+    $stat{hostError} = sprintf("ping too slow: %.4gmsec (max is %gmsec)\n",
                     $delay, $Conf{PingMaxMsec});
-    exit(1);
+    exit(RestoreCleanup($client));
 }
 
 #
 # Make sure it is really the machine we expect
 #
 if ( (my $errMsg = CorrectHostCheck($hostIP, $host)) ) {
-    print(LOG $bpc->timeStamp, "restore failed: $errMsg\n");
-    print("restore failed: $errMsg\n");
-    exit(1);
+    $stat{hostError} = $errMsg;
+    exit(RestoreCleanup($client));
 }
 
 #
 # Setup file extension for compression and open RestoreLOG output file
 #
-$Conf{CompressLevel} = 0 if ( !BackupPC::FileZIO->compOk );
+if ( $Conf{CompressLevel} && !BackupPC::FileZIO->compOk ) {
+    $stat{hostError} = "Compress:Zlib not found";
+    exit(RestoreCleanup($client));
+}
 my $fileExt = $Conf{CompressLevel} > 0 ? ".z" : "";
 my $RestoreLOG = BackupPC::FileZIO->open("$Dir/RestoreLOG$fileExt", 1,
                                      $Conf{CompressLevel});
-my $startTime = time();
-
 my $tarCreateFileCnt = 0;
 my $tarCreateByteCnt = 0;
+my $tarCreateDirCnt  = 0;
 my $tarCreateErrCnt  = 1;      # assume not ok until we learn otherwise
 my $tarCreateErr;
-my($logMsg, %stat, $xfer);
+my($logMsg, $xfer);
 
 $stat{xferOK} = $stat{hostAbort} = undef;
 $stat{hostError} = $stat{lastOutputLine} = undef;
@@ -165,41 +227,41 @@ local(*RH, *WH);
 # Run an optional pre-restore command
 #
 UserCommandRun("RestorePreUserCmd");
+if ( $? && $Conf{UserCmdCheckStatus} ) {
+    $stat{hostError} = "RestorePreUserCmd returned error status $?";
+    exit(RestoreCleanup($client));
+}
 $NeedPostCmd = 1;
 
-if ( $Conf{XferMethod} eq "tar" ) {
-    #
-    # Use tar (eg: tar/ssh) as the transport program.
-    #
-    $xfer = BackupPC::Xfer::Tar->new($bpc);
-} elsif ( $Conf{XferMethod} eq "rsync" || $Conf{XferMethod} eq "rsyncd" ) {
-    #
-    # Use rsync as the transport program.
-    #
-    if ( !defined($xfer = BackupPC::Xfer::Rsync->new($bpc)) ) {
-        my $errStr = BackupPC::Xfer::Rsync->errStr;
-        print(LOG $bpc->timeStamp, "restore failed: $errStr\n");
-        print("restore failed: $errStr\n");
-       UserCommandRun("RestorePostUserCmd") if ( $NeedPostCmd );
-        exit(1);
-    }
-} else {
-    #
-    # Default is to use smbclient (smb) as the transport program.
-    #
-    $xfer = BackupPC::Xfer::Smb->new($bpc);
+$xfer = BackupPC::Xfer::create($Conf{XferMethod}, $bpc);
+if ( !defined($xfer) ) {
+    my $errStr = BackupPC::Xfer::errStr();
+    UserCommandRun("RestorePostUserCmd") if ( $NeedPostCmd );
+    $stat{hostError} = $errStr;
+    exit(RestoreCleanup($client));
 }
+
 my $useTar = $xfer->useTar;
 
 if ( $useTar ) {
     #
-    # Create a pipe to connect BackupPC_tarCreate to the transport program
-    # (smbclient, tar, etc).
+    # Create a socketpair to connect BackupPC_tarCreate to the transport
+    # program (smbclient, tar, etc).
     # WH is the write handle for writing, provided to BackupPC_tarCreate
     # and RH is the other end of the pipe for reading provided to the
     # transport program.
     #
-    pipe(RH, WH);
+    if ( socketpair(RH, WH, AF_UNIX, SOCK_STREAM, PF_UNSPEC) ) {
+       shutdown(RH, 1);    # no writing to this socket
+       shutdown(WH, 0);    # no reading from this socket
+       setsockopt(RH, SOL_SOCKET, SO_RCVBUF, 8 * 65536);
+       setsockopt(WH, SOL_SOCKET, SO_SNDBUF, 8 * 65536);
+    } else {
+       #
+       # Default to pipe() if socketpair() doesn't work.
+       #
+       pipe(RH, WH);
+    }
 }
 
 #
@@ -207,6 +269,7 @@ if ( $useTar ) {
 #
 my @Backups = $bpc->BackupInfoRead($RestoreReq{hostSrc});
 my $xferArgs = {
+    client       => $client,
     host         => $host,
     hostIP       => $hostIP,
     type         => "restore",
@@ -215,6 +278,7 @@ my $xferArgs = {
     pipeWH       => *WH,
     XferLOG      => $RestoreLOG,
     XferMethod   => $Conf{XferMethod},
+    logLevel     => $Conf{XferLogLevel},
     bkupSrcHost  => $RestoreReq{hostSrc},
     bkupSrcShare => $RestoreReq{shareSrc},
     bkupSrcNum   => $RestoreReq{num},
@@ -222,15 +286,15 @@ my $xferArgs = {
     pathHdrSrc   => $RestoreReq{pathHdrSrc},
     pathHdrDest  => $RestoreReq{pathHdrDest},
     fileList     => $RestoreReq{fileList},
+    pidHandler   => \&pidHandler,
 };
 
 $xfer->args($xferArgs);
 
 if ( !defined($logMsg = $xfer->start()) ) {
-    print(LOG $bpc->timeStamp, "xfer start failed: ", $xfer->errStr, "\n");
-    print($xfer->errStr, "\n");
     UserCommandRun("RestorePostUserCmd") if ( $NeedPostCmd );
-    exit(1);
+    $stat{hostError} = "xfer start failed: ", $xfer->errStr;
+    exit(RestoreCleanup($client));
 }
 
 if ( $useTar ) {
@@ -261,17 +325,18 @@ if ( $useTar ) {
             @tarPathOpts,
             @{$RestoreReq{fileList}},
     );
-    my $logMsg = "Running: $BinDir/BackupPC_tarCreate "
-                     . join(" ", @tarArgs) . "\n";
-    $RestoreLOG->write(\$logMsg);
+    my $runMsg = "Running: "
+              . $bpc->execCmd2ShellCmd("$BinDir/BackupPC_tarCreate", @tarArgs)
+              . "\n";
+    $RestoreLOG->write(\$runMsg);
     if ( !defined($tarPid = open(TAR, "-|")) ) {
-       print(LOG $bpc->timeStamp, "can't fork to run tar\n");
-       print("can't fork to run tar\n");
        close(WH);
        # FIX: need to cleanup xfer
        UserCommandRun("RestorePostUserCmd") if ( $NeedPostCmd );
-       exit(0);
+       $stat{hostError} = "Can't fork to run tar";
+       exit(RestoreCleanup($client));
     }
+    binmode(TAR);
     if ( !$tarPid ) {
        #
        # This is the tarCreate child.  Clone STDERR to STDOUT,
@@ -282,6 +347,7 @@ if ( $useTar ) {
        open(STDERR, ">&STDOUT");
        close(STDOUT);
        open(STDOUT, ">&WH");
+       alarm(0);
        exec("$BinDir/BackupPC_tarCreate", @tarArgs);
        print(LOG $bpc->timeStamp, "can't exec $BinDir/BackupPC_tarCreate\n");
        # FIX: need to cleanup xfer
@@ -293,9 +359,12 @@ if ( $useTar ) {
     #
     close(WH);
 
-    $xferPid = $xfer->xferPid;
-    print(LOG $bpc->timeStamp, $logMsg, " (tarPid=$tarPid, xferPid=$xferPid)\n");
-    print("started restore, tarPid=$tarPid, xferPid=$xferPid\n");
+    @xferPid = $xfer->xferPid;
+
+    print(LOG $bpc->timeStamp, $logMsg, "\n");
+    print("started_restore\n");
+
+    pidHandler(@xferPid);
 
     #
     # Parse the output of the transfer program and BackupPC_tarCreate
@@ -329,6 +398,7 @@ if ( $useTar ) {
            if ( /^Done: (\d+) files, (\d+) bytes, (\d+) dirs, (\d+) specials, (\d+) errors/ ) {
                $tarCreateFileCnt = $1;
                $tarCreateByteCnt = $2;
+               $tarCreateDirCnt  = $3;
                $tarCreateErrCnt  = $5;
            }
        }
@@ -353,8 +423,8 @@ if ( $useTar ) {
     #
     # otherwise the xfer module does everything for us
     #
-    print(LOG $bpc->timeStamp, "Starting restore (tarPid=-1, xferPid=-1)\n");
-    print("started restore, tarPid=-1, xferPid=-1\n");
+    print(LOG $bpc->timeStamp, $logMsg . "\n");
+    print("started_restore\n");
     ($tarCreateFileCnt, $tarCreateByteCnt,
        $tarCreateErrCnt, $tarCreateErr) = $xfer->run();
 }
@@ -378,138 +448,187 @@ foreach my $k ( (keys(%stat), keys(%$newStat)) ) {
     }
 }
 
-$stat{xferOK} = 0 if ( $stat{hostError} || $stat{hostAbort} || $tarCreateErr );
+exit(RestoreCleanup($client));
+
+###########################################################################
+# Subroutines
+###########################################################################
+
+sub CorrectHostCheck
+{
+    my($hostIP, $host) = @_;
+    return if ( $hostIP eq $host && !$Conf{FixedIPNetBiosNameCheck}
+               || $Conf{NmbLookupCmd} eq "" );
+    my($netBiosHost, $netBiosUser) = $bpc->NetBiosInfoGet($hostIP);
+    return "host $host has mismatching netbios name $netBiosHost"
+            if ( lc($netBiosHost) ne lc(substr($host, 0, 15)) );
+    return;
+}
+
+sub catch_signal
+{
+    my $signame = shift;
 
-if ( !$stat{xferOK} ) {
     #
-    # kill off the tranfer program, first nicely then forcefully
+    # Children quit quietly on ALRM
     #
-    kill(2, $xferPid) if ( $xferPid > 0 );
-    sleep(1);
-    kill(9, $xferPid) if ( $xferPid > 0 );
+    exit(1) if ( $Pid != $$ && $signame eq "ALRM" );
+
     #
-    # kill off the tar process, first nicely then forcefully
+    # Ignore signals in children
     #
-    kill(2, $tarPid) if ( $tarPid > 0 );
-    sleep(1);
-    kill(9, $tarPid) if ( $tarPid > 0 );
-}
+    return if ( $Pid != $$ );
 
-my $lastNum  = -1;
-my @Restores;
-
-#
-# Do one last check to make sure it is still the machine we expect.
-#
-if ( $stat{xferOK} && (my $errMsg = CorrectHostCheck($hostIP, $host)) ) {
-    $stat{hostError} = $errMsg;
+    #
+    # Note: needs to be tested for each kind of XferMethod
+    #
+    print(LOG $bpc->timeStamp, "cleaning up after signal $signame\n");
+    $SIG{$signame} = 'IGNORE';
+    $RestoreLOG->write(\"exiting after signal $signame\n");
     $stat{xferOK} = 0;
+    if ( $signame eq "INT" ) {
+       $stat{hostError} = "aborted by user (signal=$signame)";
+    } else {
+       $stat{hostError} = "aborted by signal=$signame";
+    }
+    exit(RestoreCleanup($client));
 }
-@Restores = $bpc->RestoreInfoRead($host);
-for ( my $i = 0 ; $i < @Restores ; $i++ ) {
-    $lastNum = $Restores[$i]{num} if ( $lastNum < $Restores[$i]{num} );
-}
-$lastNum++;
 
 #
-# Run an optional post-restore command
+# Cleanup and update the restore status
 #
-UserCommandRun("RestorePostUserCmd") if ( $NeedPostCmd );
+sub RestoreCleanup
+{
+    my($client) = @_;
 
-$RestoreLOG->close();
-rename("$Dir/RestoreLOG$fileExt", "$Dir/RestoreLOG.$lastNum$fileExt");
-rename("$Dir/$reqFileName", "$Dir/RestoreInfo.$lastNum");
-my $endTime = time();
+    $stat{xferOK} = 0 if ( $stat{hostError} || $stat{hostAbort}
+                       || $tarCreateErr );
+
+    if ( !$stat{xferOK} ) {
+       #
+       # kill off the tranfer program, first nicely then forcefully
+       #
+       if ( @xferPid ) {
+           kill($bpc->sigName2num("INT"), @xferPid);
+           sleep(1);
+           kill($bpc->sigName2num("KILL"), @xferPid);
+       }
+       #
+       # kill off the tar process, first nicely then forcefully
+       #
+       if ( $tarPid > 0 ) {
+           kill($bpc->sigName2num("INT"), $tarPid);
+           sleep(1);
+           kill($bpc->sigName2num("KILL"), $tarPid);
+       }
+    }
+
+    my $lastNum  = -1;
+    my @Restores;
 
-#
-# If the restore failed, clean up
-#
-if ( !$stat{xferOK} ) {
     #
-    # wait a short while and see if the system is still alive
+    # Do one last check to make sure it is still the machine we expect.
     #
-    $stat{hostError} ||= $tarCreateErr if ( $tarCreateErr ne "" );
-    $stat{hostError} = $stat{lastOutputLine} if ( $stat{hostError} eq "" );
-    if ( $stat{hostError} ) {
-        print(LOG $bpc->timeStamp,
-                  "Got fatal error during xfer ($stat{hostError})\n");
-    }
-    sleep(2);
-    if ( $bpc->CheckHostAlive($hostIP) < 0 ) {
-        $stat{hostAbort} = 1;
+    if ( $stat{xferOK} && (my $errMsg = CorrectHostCheck($hostIP, $host)) ) {
+       $stat{hostError} = $errMsg;
+       $stat{xferOK} = 0;
     }
-    if ( $stat{hostAbort} ) {
-        $stat{hostError} = "lost network connection during restore";
+    @Restores = $bpc->RestoreInfoRead($client);
+    for ( my $i = 0 ; $i < @Restores ; $i++ ) {
+       $lastNum = $Restores[$i]{num} if ( $lastNum < $Restores[$i]{num} );
     }
-}
-
-#
-# Add the new restore information to the restore file
-#
-@Restores = $bpc->RestoreInfoRead($host);
-my $i = @Restores;
-$Restores[$i]{num}           = $lastNum;
-$Restores[$i]{startTime}     = $startTime;
-$Restores[$i]{endTime}       = $endTime;
-$Restores[$i]{result}        = $stat{xferOK} ? "ok" : "failed";
-$Restores[$i]{errorMsg}      = $stat{hostError};
-$Restores[$i]{nFiles}        = $tarCreateFileCnt;
-$Restores[$i]{size}          = $tarCreateByteCnt;
-$Restores[$i]{tarCreateErrs} = $tarCreateErrCnt;
-$Restores[$i]{xferErrs}      = $stat{xferErrCnt} || 0;
-
-while ( @Restores > $Conf{RestoreInfoKeepCnt} ) {
-    my $num = $Restores[0]{num};
-    unlink("$Dir/RestoreLOG.$num.z");
-    unlink("$Dir/RestoreLOG.$num");
-    unlink("$Dir/RestoreInfo.$num");
-    shift(@Restores);
-}
-$bpc->RestoreInfoWrite($host, @Restores);
+    $lastNum++;
 
-if ( !$stat{xferOK} ) {
-    print(LOG $bpc->timeStamp, "Restore aborted ($stat{hostError})\n");
-    print("restore failed: $stat{hostError}\n");
-} else {
-    print("restore complete\n");
-}
+    #
+    # Run an optional post-restore command
+    #
+    if ( $NeedPostCmd ) {
+        UserCommandRun("RestorePostUserCmd");
+        if ( $? && $Conf{UserCmdCheckStatus} ) {
+            $stat{hostError} = "RestorePostUserCmd returned error status $?";
+            $stat{xferOK} = 0;
+        }
+    }
 
-###########################################################################
-# Subroutines
-###########################################################################
+    rename("$Dir/RestoreLOG$fileExt", "$Dir/RestoreLOG.$lastNum$fileExt");
+    rename("$Dir/$reqFileName", "$Dir/RestoreInfo.$lastNum");
+    my $endTime = time();
 
-sub CorrectHostCheck
-{
-    my($hostIP, $host) = @_;
-    return if ( $hostIP eq $host && !$Conf{FixedIPNetBiosNameCheck} );
-    my($netBiosHost, $netBiosUser) = $bpc->NetBiosInfoGet($hostIP);
-    return "host $host has mismatching netbios name $netBiosHost"
-            if ( $netBiosHost ne $host );
-    return;
-}
+    #
+    # If the restore failed, clean up
+    #
+    if ( !$stat{xferOK} ) {
+       #
+       # wait a short while and see if the system is still alive
+       #
+       $stat{hostError} ||= $tarCreateErr if ( $tarCreateErr ne "" );
+       $stat{hostError} = $stat{lastOutputLine} if ( $stat{hostError} eq "" );
+       sleep(2);
+       if ( $bpc->CheckHostAlive($hostIP) < 0 ) {
+           $stat{hostAbort} = 1;
+       }
+       if ( $stat{hostAbort} && $stat{hostError} eq "" ) {
+           $stat{hostError} = "lost network connection during restore";
+       }
+       $RestoreLOG->write(\"restore failed: $stat{hostError}\n")
+                                           if ( defined($RestoreLOG) );
+    }
 
-sub catch_signal
-{
-    my $signame = shift;
+    $RestoreLOG->close() if ( defined($RestoreLOG) );
 
     #
-    # Note: needs to be tested for each kind of XferMethod
+    # Add the new restore information to the restore file
     #
-    print(LOG $bpc->timeStamp, "cleaning up after signal $signame\n");
-    if ( $xferPid > 0 ) {
-        if ( kill(2, $xferPid) <= 0 ) {
-            sleep(1);
-            kill(9, $xferPid);
-        }
+    @Restores = $bpc->RestoreInfoRead($client);
+    my $i = @Restores;
+    $Restores[$i]{num}           = $lastNum;
+    $Restores[$i]{startTime}     = $startTime;
+    $Restores[$i]{endTime}       = $endTime;
+    $Restores[$i]{result}        = $stat{xferOK} ? "ok" : "failed";
+    $Restores[$i]{errorMsg}      = $stat{hostError};
+    $Restores[$i]{nFiles}        = $tarCreateFileCnt;
+    $Restores[$i]{size}          = $tarCreateByteCnt;
+    $Restores[$i]{tarCreateErrs} = $tarCreateErrCnt;
+    $Restores[$i]{xferErrs}      = $stat{xferErrCnt} || 0;
+
+    while ( @Restores > $Conf{RestoreInfoKeepCnt} ) {
+       my $num = $Restores[0]{num};
+       unlink("$Dir/RestoreLOG.$num.z");
+       unlink("$Dir/RestoreLOG.$num");
+       unlink("$Dir/RestoreInfo.$num");
+       shift(@Restores);
     }
-    if ( $tarPid > 0 ) {
-        if ( kill(2, $tarPid) <= 0 ) {
-            sleep(1);
-            kill(9, $tarPid);
-        }
+    $bpc->RestoreInfoWrite($client, @Restores);
+
+    if ( !$stat{xferOK} ) {
+       print(LOG $bpc->timeStamp, "restore failed ($stat{hostError})\n");
+       print("restore failed: $stat{hostError}\n");
+       return 1;
+    } else {
+        $stat{xferErrCnt} ||= 0;
+       print(LOG $bpc->timeStamp, "restore $lastNum complete"
+                . " ($tarCreateFileCnt files, $tarCreateByteCnt bytes,"
+                . " $tarCreateDirCnt dirs, $stat{xferErrCnt} xferErrs)\n");
+       print("restore complete\n");
+       return;
     }
-    $stat{xferOK} = 0;
-    $stat{hostError} = "aborted by signal $signame";
+}
+
+#
+# The Xfer method might tell us from time to time about processes
+# it forks.  We tell BackupPC about this (for status displays) and
+# keep track of the pids in case we cancel the backup
+#
+sub pidHandler
+{
+    @xferPid = @_;
+    @xferPid = grep(/./, @xferPid);
+    return if ( !@xferPid && $tarPid < 0 );
+    my @pids = @xferPid;
+    push(@pids, $tarPid) if ( $tarPid > 0 );
+    my $str = join(",", @pids);
+    $RestoreLOG->write(\"Xfer PIDs are now $str\n") if ( defined($RestoreLOG) );
+    print("xferPids $str\n");
 }
 
 #
@@ -517,20 +636,25 @@ sub catch_signal
 #
 sub UserCommandRun
 {
-    my($type) = @_;
+    my($cmdType) = @_;
 
-    return if ( !defined($Conf{$type}) );
+    return if ( !defined($Conf{$cmdType}) );
     my $vars = {
         xfer         => $xfer,
+        client       => $client,
         host         => $host,
         hostIP       => $hostIP,
         share        => $RestoreReq{shareDest},
         XferMethod   => $Conf{XferMethod},
+        sshPath      => $Conf{SshPath},
         LOG          => *LOG,
+       user         => $Hosts->{$client}{user},
+       moreUsers    => $Hosts->{$client}{moreUsers},
         XferLOG      => $RestoreLOG,
         stat         => \%stat,
-        xferOK       => $stat{xferOK},
-       type         => $type,
+        xferOK       => $stat{xferOK} || 0,
+       hostError    => $stat{hostError},
+       type         => "restore",
        bkupSrcHost  => $RestoreReq{hostSrc},
        bkupSrcShare => $RestoreReq{shareSrc},
        bkupSrcNum   => $RestoreReq{num},
@@ -538,9 +662,10 @@ sub UserCommandRun
        pathHdrSrc   => $RestoreReq{pathHdrSrc},
        pathHdrDest  => $RestoreReq{pathHdrDest},
        fileList     => $RestoreReq{fileList},
+       cmdType      => $cmdType,
     };
-    my $cmd = $bpc->cmdVarSubstitute($Conf{$type}, $vars);
-    $RestoreLOG->write(\"Executing $type: @$cmd\n");
+    my $cmd = $bpc->cmdVarSubstitute($Conf{$cmdType}, $vars);
+    $RestoreLOG->write(\"Executing $cmdType: @$cmd\n");
     #
     # Run the user's command, dumping the stdout/stderr into the
     # Xfer log file.  Also supply the optional $vars and %Conf in