stored: don't touch devices that were not acquired

when one of acquire_device_for_read() or
acquire_device_for_append() in do_mac_run()
returned false, the code still touched the
devices eventually failing another job that
had the device currently acquired.
This patch makes sure do_mac_run() does not
touch any device it couldn't aquire.

core/src/stored/mac.cc

@@ -490,6 +490,7 @@ bool DoMacRun(JobControlRecord* jcr)
   char ec1[50];
   const char* Type;
   bool ok = true;
+  bool acquire_fail = false;
   BareosSocket* dir = jcr->dir_bsock;
   Device* dev = jcr->dcr->dev;
 
@@ -543,6 +544,7 @@ bool DoMacRun(JobControlRecord* jcr)
      */
     if (!AcquireDeviceForRead(jcr->read_dcr)) {
       ok = false;
+      acquire_fail = true;
       goto bail_out;
     }
 
@@ -661,6 +663,7 @@ bool DoMacRun(JobControlRecord* jcr)
     if (!AcquireDeviceForRead(jcr->read_dcr) ||
         !AcquireDeviceForAppend(jcr->dcr)) {
       ok = false;
+      acquire_fail = true;
       goto bail_out;
     }
 
@@ -699,7 +702,7 @@ bool DoMacRun(JobControlRecord* jcr)
 bail_out:
   if (!ok) { jcr->setJobStatus(JS_ErrorTerminated); }
 
-  if (!jcr->remote_replicate && jcr->dcr) {
+  if (!acquire_fail && !jcr->remote_replicate && jcr->dcr) {
     /*
      * Don't use time_t for job_elapsed as time_t can be 32 or 64 bits,
      *   and the subsequent Jmsg() editing will break