rt_neth_acp fix for lists changed during remote request when gdb lock is removed

src/exe/rt_neth_acp/src/rt_neth_acp.c

@@ -326,9 +326,15 @@ lockMountServers (
     vl = pool_Qsucc(NULL, gdbroot->pool, vl)
   ) {
     vp = pool_Qitem(vl, gdb_sVolume, l.own_ll);
+
+    if ( vl->self == vl->flink) {
+      /* Connection lost and volume removed from own list */
+      errh_Error("Volume not owned any more, %s", vp->g.name.orig);
+      return;
+    }
     if(!vp->l.flags.b.isConnected) {
       /* !!! Todo !!! How do we make this known ?  */
-      errh_Error("Volume not connected, %s", vp->g.name);
+      errh_Error("Volume not connected, %s", vp->g.name.orig);
       continue;
     }
 
@@ -349,7 +355,8 @@ lockMountServers (
       }
       msp->msor = pool_ItemReference(NULL, gdbroot->pool, op);
       op->l.flags.b.isMountServer = 1;
-      pool_QinsertPred(NULL, gdbroot->pool, &msp->nodms_ll, &np->nodms_lh);
+      if ( msp->nodms_ll.self == msp->nodms_ll.flink && msp->nodms_ll.self == msp->nodms_ll.blink)
+	pool_QinsertPred(NULL, gdbroot->pool, &msp->nodms_ll, &np->nodms_lh);
       
       if (0) errh_Info("Locking object %s", op->g.f.name.orig);
       cvolc_LockObject(&sts, op);

src/lib/rt/src/rt_cvolc.c

@@ -101,6 +101,8 @@ fetch (
     net_Free(NULL, rsp);
     pwr_Return(NULL, sts, lsts);
   }
+  if ( !vp->l.flags.b.isConnected)
+    pwr_Return(NULL, sts, GDH__CONNLOST);
 
   for (i = 0, gop = &rsp->g[0]; i < rsp->count; i++, gop++) {
     if ( vp->g.vid != gop->oid.vid) {

src/lib/rt/src/rt_cvolcm.c

@@ -65,7 +65,7 @@ cvolcm_AddClassVolume(
   const net_sGvolume	*vp
 )
 {
-  gdb_sCclassVolume	*ccvp;
+  gdb_sCclassVolume	*ccvp, *rp;
   gdb_sVolume		*cvp;
   
 
@@ -89,8 +89,11 @@ cvolcm_AddClassVolume(
     ccvp->equalClasses = time_Acomp(&ccvp->time, &t) == 0 ? 1 : 0;
   }
   
-  ccvp = hash_Insert(sts, gdbroot->ccvol_ht, ccvp);
-  if (ccvp == NULL) errh_Bugcheck(GDH__WEIRD, "adding cached class volume");
+  rp = hash_Insert(sts, gdbroot->ccvol_ht, ccvp);
+  if (rp == NULL) { /* This was previously a bugcheck but obviously can occur */
+    pool_Free(NULL, gdbroot->pool, ccvp);
+    return;
+  }
   
   pool_QinsertPred(NULL, gdbroot->pool, &ccvp->ccvol_ll, &np->ccvol_lh);