afs: Avoid panics on failed return from afs_CFileOpen
[openafs.git] / src / afs / VNOPS / afs_vnop_symlink.c
index f306715..e88e09e 100644 (file)
@@ -21,8 +21,6 @@
 #include <afsconfig.h>
 #include "afs/param.h"
 
-RCSID
-    ("$Header$");
 
 #include "afs/sysincludes.h"   /* Standard vendor system headers */
 #include "afsincludes.h"       /* Afs-based standard headers */
@@ -39,15 +37,46 @@ extern afs_rwlock_t afs_xcbhash;
  * is just a performance hit.
  */
 
+static int
+afs_DisconCreateSymlink(struct vcache *avc, char *aname, 
+                       struct vrequest *areq) {
+    struct dcache *tdc;
+    struct osi_file *tfile;
+    afs_size_t offset, len;
+    int code = 0;
+
+    tdc = afs_GetDCache(avc, 0, areq, &offset, &len, 0);
+    if (!tdc) {
+       /* printf("afs_DisconCreateSymlink: can't get new dcache for symlink.\n"); */
+       return ENETDOWN;
+    }
+
+    len = strlen(aname);
+    avc->f.m.Length = len;
+
+    ObtainWriteLock(&tdc->lock, 720);
+    tfile = afs_CFileOpen(&tdc->f.inode);
+    if (!tfile) {
+       code = EIO;
+       goto done;
+    }
+    afs_AdjustSize(tdc, len);
+    tdc->validPos = len;
+    afs_CFileWrite(tfile, 0, aname, len);
+    afs_CFileClose(tfile);
+ done:
+    ReleaseWriteLock(&tdc->lock);
+    return code;
+}
 
 /* don't set CDirty in here because RPC is called synchronously */
 int 
 afs_symlink(OSI_VC_DECL(adp), char *aname, struct vattr *attrs, 
-           char *atargetName, struct AFS_UCRED *acred)
+           char *atargetName, struct vcache **tvcp, afs_ucred_t *acred)
 {
     afs_uint32 now = 0;
-    struct vrequest treq;
-    afs_int32 code;
+    struct vrequest *treq = NULL;
+    afs_int32 code = 0;
     struct afs_conn *tc;
     struct VenusFid newFid;
     struct dcache *tdc;
@@ -56,11 +85,12 @@ afs_symlink(OSI_VC_DECL(adp), char *aname, struct vattr *attrs,
     struct server *hostp = 0;
     struct vcache *tvc;
     struct AFSStoreStatus InStatus;
-    struct AFSFetchStatus OutFidStatus, OutDirStatus;
+    struct AFSFetchStatus *OutFidStatus, *OutDirStatus;
     struct AFSCallBack CallBack;
     struct AFSVolSync tsync;
     struct volume *volp = 0;
     struct afs_fakestat_state fakestate;
+    struct rx_connection *rxconn;
     XSTATS_DECLS;
     OSI_VC_CONVERT(adp);
 
@@ -68,14 +98,18 @@ afs_symlink(OSI_VC_DECL(adp), char *aname, struct vattr *attrs,
     afs_Trace2(afs_iclSetp, CM_TRACE_SYMLINK, ICL_TYPE_POINTER, adp,
               ICL_TYPE_STRING, aname);
 
-    if ((code = afs_InitReq(&treq, acred)))
+    OutFidStatus = osi_AllocSmallSpace(sizeof(struct AFSFetchStatus));
+    OutDirStatus = osi_AllocSmallSpace(sizeof(struct AFSFetchStatus));
+    memset(&InStatus, 0, sizeof(InStatus));
+
+    if ((code = afs_CreateReq(&treq, acred)))
        goto done2;
 
     afs_InitFakeStat(&fakestate);
 
     AFS_DISCON_LOCK();
     
-    code = afs_EvalFakeStat(&adp, &fakestate, &treq);
+    code = afs_EvalFakeStat(&adp, &fakestate, treq);
     if (code)
        goto done;
 
@@ -93,21 +127,21 @@ afs_symlink(OSI_VC_DECL(adp), char *aname, struct vattr *attrs,
        goto done;
     }
 
-    code = afs_VerifyVCache(adp, &treq);
+    code = afs_VerifyVCache(adp, treq);
     if (code) {
-       code = afs_CheckCode(code, &treq, 30);
+       code = afs_CheckCode(code, treq, 30);
        goto done;
     }
 
     /** If the volume is read-only, return error without making an RPC to the
       * fileserver
       */
-    if (adp->states & CRO) {
+    if (adp->f.states & CRO) {
        code = EROFS;
        goto done;
     }
 
-    if (AFS_IS_DISCONNECTED) {
+    if (AFS_IS_DISCONNECTED && !AFS_IS_DISCON_RW) {
         code = ENETDOWN;
         goto done;
     }
@@ -123,52 +157,56 @@ afs_symlink(OSI_VC_DECL(adp), char *aname, struct vattr *attrs,
        InStatus.UnixModeBits = 0755;
        alen++;                 /* add in the null */
     }
-    tdc = afs_GetDCache(adp, (afs_size_t) 0, &treq, &offset, &len, 1);
-    volp = afs_FindVolume(&adp->fid, READ_LOCK);       /*parent is also in same vol */
+    tdc = afs_GetDCache(adp, (afs_size_t) 0, treq, &offset, &len, 1);
+    volp = afs_FindVolume(&adp->f.fid, READ_LOCK);     /*parent is also in same vol */
     ObtainWriteLock(&adp->lock, 156);
     if (tdc)
        ObtainWriteLock(&tdc->lock, 636);
-    ObtainSharedLock(&afs_xvcache, 17);        /* prevent others from creating this entry */
-    /* XXX Pay attention to afs_xvcache around the whole thing!! XXX */
-    do {
-       tc = afs_Conn(&adp->fid, &treq, SHARED_LOCK);
-       if (tc) {
-           hostp = tc->srvr->server;
-           XSTATS_START_TIME(AFS_STATS_FS_RPCIDX_SYMLINK);
-           if (adp->states & CForeign) {
-               now = osi_Time();
-               RX_AFS_GUNLOCK();
-               code =
-                   RXAFS_DFSSymlink(tc->id, (struct AFSFid *)&adp->fid.Fid,
-                                    aname, atargetName, &InStatus,
-                                    (struct AFSFid *)&newFid.Fid,
-                                    &OutFidStatus, &OutDirStatus, &CallBack,
-                                    &tsync);
-               RX_AFS_GLOCK();
-           } else {
-               RX_AFS_GUNLOCK();
-               code =
-                   RXAFS_Symlink(tc->id, (struct AFSFid *)&adp->fid.Fid,
-                                 aname, atargetName, &InStatus,
-                                 (struct AFSFid *)&newFid.Fid, &OutFidStatus,
-                                 &OutDirStatus, &tsync);
-               RX_AFS_GLOCK();
-           }
-           XSTATS_END_TIME;
-       } else
-           code = -1;
-    } while (afs_Analyze
-            (tc, code, &adp->fid, &treq, AFS_STATS_FS_RPCIDX_SYMLINK,
-             SHARED_LOCK, NULL));
-
-    UpgradeSToWLock(&afs_xvcache, 40);
+    /* No further locks: if the SymLink succeeds, it does not matter what happens
+     * to our local copy of the directory. If somebody tampers with it in the meantime,
+     * the copy will be invalidated */
+    if (!AFS_IS_DISCON_RW) {
+       do {
+           tc = afs_Conn(&adp->f.fid, treq, SHARED_LOCK, &rxconn);
+           if (tc) {
+               hostp = tc->parent->srvr->server;
+               XSTATS_START_TIME(AFS_STATS_FS_RPCIDX_SYMLINK);
+               if (adp->f.states & CForeign) {
+                   now = osi_Time();
+                   RX_AFS_GUNLOCK();
+                   code = 
+                       RXAFS_DFSSymlink(rxconn,
+                                        (struct AFSFid *)&adp->f.fid.Fid,
+                                        aname, atargetName, &InStatus,
+                                        (struct AFSFid *)&newFid.Fid,
+                                        OutFidStatus, OutDirStatus,
+                                        &CallBack, &tsync);
+                   RX_AFS_GLOCK();
+               } else {
+                   RX_AFS_GUNLOCK();
+                   code =
+                       RXAFS_Symlink(rxconn, (struct AFSFid *)&adp->f.fid.Fid,
+                                     aname, atargetName, &InStatus,
+                                     (struct AFSFid *)&newFid.Fid, 
+                                     OutFidStatus, OutDirStatus, &tsync);
+                   RX_AFS_GLOCK();
+               }
+               XSTATS_END_TIME;
+           } else
+               code = -1;
+       } while (afs_Analyze
+                   (tc, rxconn, code, &adp->f.fid, treq, AFS_STATS_FS_RPCIDX_SYMLINK,
+                    SHARED_LOCK, NULL));
+    } else {
+       newFid.Cell = adp->f.fid.Cell;
+       newFid.Fid.Volume = adp->f.fid.Fid.Volume;
+       afs_GenFakeFid(&newFid, VREG, 0);
+    }
+
+    ObtainWriteLock(&afs_xvcache, 40);
     if (code) {
        if (code < 0) {
-           ObtainWriteLock(&afs_xcbhash, 499);
-           afs_DequeueCallback(adp);
-           adp->states &= ~CStatd;
-           ReleaseWriteLock(&afs_xcbhash);
-           osi_dnlc_purgedp(adp);
+           afs_StaleVCache(adp);
        }
        ReleaseWriteLock(&adp->lock);
        ReleaseWriteLock(&afs_xvcache);
@@ -179,12 +217,14 @@ afs_symlink(OSI_VC_DECL(adp), char *aname, struct vattr *attrs,
        goto done;
     }
     /* otherwise, we should see if we can make the change to the dir locally */
-    if (afs_LocalHero(adp, tdc, &OutDirStatus, 1)) {
+    if (AFS_IS_DISCON_RW || afs_LocalHero(adp, tdc, OutDirStatus, 1)) {
        /* we can do it locally */
        ObtainWriteLock(&afs_xdcache, 293);
+       /* If the following fails because the name has been created in the meantime, the
+        * directory is out-of-date - the file server knows best! */
        code = afs_dir_Create(tdc, aname, &newFid.Fid);
        ReleaseWriteLock(&afs_xdcache);
-       if (code) {
+       if (code && !AFS_IS_DISCON_RW) {
            ZapDCE(tdc);        /* surprise error -- use invalid value */
            DZap(tdc);
        }
@@ -193,8 +233,8 @@ afs_symlink(OSI_VC_DECL(adp), char *aname, struct vattr *attrs,
        ReleaseWriteLock(&tdc->lock);
        afs_PutDCache(tdc);
     }
-    newFid.Cell = adp->fid.Cell;
-    newFid.Fid.Volume = adp->fid.Fid.Volume;
+    newFid.Cell = adp->f.fid.Cell;
+    newFid.Fid.Volume = adp->f.fid.Fid.Volume;
     ReleaseWriteLock(&adp->lock);
 
     /* now we're done with parent dir, create the link's entry.  Note that
@@ -209,11 +249,11 @@ afs_symlink(OSI_VC_DECL(adp), char *aname, struct vattr *attrs,
     }
     ObtainWriteLock(&tvc->lock, 157);
     ObtainWriteLock(&afs_xcbhash, 500);
-    tvc->states |= CStatd;     /* have valid info */
-    tvc->states &= ~CBulkFetching;
+    tvc->f.states |= CStatd;   /* have valid info */
+    tvc->f.states &= ~CBulkFetching;
 
-    if (adp->states & CForeign) {
-       tvc->states |= CForeign;
+    if (adp->f.states & CForeign) {
+       tvc->f.states |= CForeign;
        /* We don't have to worry about losing the callback since we're doing it 
         * under the afs_xvcache lock actually, afs_NewVCache may drop the 
         * afs_xvcache lock, if it calls afs_FlushVCache */
@@ -224,35 +264,58 @@ afs_symlink(OSI_VC_DECL(adp), char *aname, struct vattr *attrs,
        /* since it never expires, we don't have to queue the callback */
     }
     ReleaseWriteLock(&afs_xcbhash);
-    afs_ProcessFS(tvc, &OutFidStatus, &treq);
+
+    if (AFS_IS_DISCON_RW) {
+       attrs->va_mode = InStatus.UnixModeBits;
+       afs_GenDisconStatus(adp, tvc, &newFid, attrs, treq, VLNK);
+       code = afs_DisconCreateSymlink(tvc, atargetName, treq);
+       if (code) {
+           /* XXX - When this goes wrong, we need to tidy up the changes we made to
+            * the parent, and get rid of the vcache we just created */
+           ReleaseWriteLock(&tvc->lock);
+           ReleaseWriteLock(&afs_xvcache);
+           afs_PutVCache(tvc);
+           goto done;
+       }
+       afs_DisconAddDirty(tvc, VDisconCreate, 0);
+    } else {
+       afs_ProcessFS(tvc, OutFidStatus, treq);
+    }
 
     if (!tvc->linkData) {
-       tvc->linkData = (char *)afs_osi_Alloc(alen);
+       tvc->linkData = afs_osi_Alloc(alen);
+       osi_Assert(tvc->linkData != NULL);
        strncpy(tvc->linkData, atargetName, alen - 1);
        tvc->linkData[alen - 1] = 0;
     }
     ReleaseWriteLock(&tvc->lock);
     ReleaseWriteLock(&afs_xvcache);
-    afs_PutVCache(tvc);
+    if (tvcp)
+       *tvcp = tvc;
+    else
+       afs_PutVCache(tvc);
     code = 0;
   done:
     afs_PutFakeStat(&fakestate);
     if (volp)
        afs_PutVolume(volp, READ_LOCK);
     AFS_DISCON_UNLOCK();
-    code = afs_CheckCode(code, &treq, 31);
+    code = afs_CheckCode(code, treq, 31);
+    afs_DestroyReq(treq);
   done2:
+    osi_FreeSmallSpace(OutFidStatus);
+    osi_FreeSmallSpace(OutDirStatus);
     return code;
 }
 
 int
-afs_MemHandleLink(register struct vcache *avc, struct vrequest *areq)
+afs_MemHandleLink(struct vcache *avc, struct vrequest *areq)
 {
-    register struct dcache *tdc;
-    register char *tp, *rbuf;
+    struct dcache *tdc;
+    char *tp, *rbuf;
     afs_size_t offset, len;
     afs_int32 tlen, alen;
-    register afs_int32 code;
+    afs_int32 code;
 
     AFS_STATCNT(afs_MemHandleLink);
     /* two different formats, one for links protected 644, have a "." at
@@ -269,13 +332,13 @@ afs_MemHandleLink(register struct vcache *avc, struct vrequest *areq)
            afs_PutDCache(tdc);
            return EFAULT;
        }
-       if (avc->m.Mode & 0111)
+       if (avc->f.m.Mode & 0111)
            alen = len + 1;     /* regular link */
        else
            alen = len;         /* mt point */
-       rbuf = (char *)osi_AllocLargeSpace(AFS_LRALLOCSIZ);
+       rbuf = osi_AllocLargeSpace(AFS_LRALLOCSIZ);
        ObtainReadLock(&tdc->lock);
-       addr = afs_MemCacheOpen(tdc->f.inode);
+       addr = afs_MemCacheOpen(&tdc->f.inode);
        tlen = len;
        code = afs_MemReadBlk(addr, 0, rbuf, tlen);
        afs_MemCacheClose(addr);
@@ -284,6 +347,7 @@ afs_MemHandleLink(register struct vcache *avc, struct vrequest *areq)
        rbuf[alen - 1] = 0;
        alen = strlen(rbuf) + 1;
        tp = afs_osi_Alloc(alen);       /* make room for terminating null */
+       osi_Assert(tp != NULL);
        memcpy(tp, rbuf, alen);
        osi_FreeLargeSpace(rbuf);
        if (code != len) {
@@ -296,14 +360,14 @@ afs_MemHandleLink(register struct vcache *avc, struct vrequest *areq)
 }
 
 int
-afs_UFSHandleLink(register struct vcache *avc, struct vrequest *areq)
+afs_UFSHandleLink(struct vcache *avc, struct vrequest *areq)
 {
-    register struct dcache *tdc;
-    register char *tp, *rbuf;
+    struct dcache *tdc;
+    char *tp, *rbuf;
     void *tfile;
     afs_size_t offset, len;
     afs_int32 tlen, alen;
-    register afs_int32 code;
+    afs_int32 code;
 
     /* two different formats, one for links protected 644, have a "." at the
      * end of the file name, which we turn into a null.  Others, protected
@@ -313,7 +377,7 @@ afs_UFSHandleLink(register struct vcache *avc, struct vrequest *areq)
        tdc = afs_GetDCache(avc, (afs_size_t) 0, areq, &offset, &len, 0);
        afs_Trace3(afs_iclSetp, CM_TRACE_UFSLINK, ICL_TYPE_POINTER, avc,
                   ICL_TYPE_POINTER, tdc, ICL_TYPE_OFFSET,
-                  ICL_HANDLE_OFFSET(avc->m.Length));
+                  ICL_HANDLE_OFFSET(avc->f.m.Length));
        if (!tdc) {
            if (AFS_IS_DISCONNECTED)
                return ENETDOWN;
@@ -325,18 +389,20 @@ afs_UFSHandleLink(register struct vcache *avc, struct vrequest *areq)
            afs_PutDCache(tdc);
            return EFAULT;
        }
-       if (avc->m.Mode & 0111)
+       if (avc->f.m.Mode & 0111)
            alen = len + 1;     /* regular link */
        else
            alen = len;         /* mt point */
-       rbuf = (char *)osi_AllocLargeSpace(AFS_LRALLOCSIZ);
+       rbuf = osi_AllocLargeSpace(AFS_LRALLOCSIZ);
        tlen = len;
        ObtainReadLock(&tdc->lock);
-#if defined(LINUX_USE_FH)
-       tfile = osi_UFSOpen_fh(&tdc->f.fh, tdc->f.fh_type);
-#else
-       tfile = osi_UFSOpen(tdc->f.inode);
-#endif
+       tfile = osi_UFSOpen(&tdc->f.inode);
+       if (!tfile) {
+           ReleaseReadLock(&tdc->lock);
+           afs_PutDCache(tdc);
+           osi_FreeLargeSpace(rbuf);
+           return EIO;
+       }
        code = afs_osi_Read(tfile, -1, rbuf, tlen);
        osi_UFSClose(tfile);
        ReleaseReadLock(&tdc->lock);
@@ -344,6 +410,7 @@ afs_UFSHandleLink(register struct vcache *avc, struct vrequest *areq)
        rbuf[alen - 1] = '\0';
        alen = strlen(rbuf) + 1;
        tp = afs_osi_Alloc(alen);       /* make room for terminating null */
+       osi_Assert(tp != NULL);
        memcpy(tp, rbuf, alen);
        osi_FreeLargeSpace(rbuf);
        if (code != tlen) {
@@ -356,26 +423,26 @@ afs_UFSHandleLink(register struct vcache *avc, struct vrequest *areq)
 }
 
 int
-afs_readlink(OSI_VC_DECL(avc), struct uio *auio, struct AFS_UCRED *acred)
+afs_readlink(OSI_VC_DECL(avc), struct uio *auio, afs_ucred_t *acred)
 {
-    register afs_int32 code;
-    struct vrequest treq;
-    register char *tp;
+    afs_int32 code;
+    struct vrequest *treq = NULL;
+    char *tp;
     struct afs_fakestat_state fakestat;
     OSI_VC_CONVERT(avc);
 
     AFS_STATCNT(afs_readlink);
     afs_Trace1(afs_iclSetp, CM_TRACE_READLINK, ICL_TYPE_POINTER, avc);
-    if ((code = afs_InitReq(&treq, acred)))
+    if ((code = afs_CreateReq(&treq, acred)))
        return code;
     afs_InitFakeStat(&fakestat);
 
     AFS_DISCON_LOCK();
     
-    code = afs_EvalFakeStat(&avc, &fakestat, &treq);
+    code = afs_EvalFakeStat(&avc, &fakestat, treq);
     if (code)
        goto done;
-    code = afs_VerifyVCache(avc, &treq);
+    code = afs_VerifyVCache(avc, treq);
     if (code)
        goto done;
     if (vType(avc) != VLNK) {
@@ -383,7 +450,7 @@ afs_readlink(OSI_VC_DECL(avc), struct uio *auio, struct AFS_UCRED *acred)
        goto done;
     }
     ObtainWriteLock(&avc->lock, 158);
-    code = afs_HandleLink(avc, &treq);
+    code = afs_HandleLink(avc, treq);
     /* finally uiomove it to user-land */
     if (code == 0) {
        tp = avc->linkData;
@@ -397,6 +464,7 @@ afs_readlink(OSI_VC_DECL(avc), struct uio *auio, struct AFS_UCRED *acred)
   done:
     afs_PutFakeStat(&fakestat);
     AFS_DISCON_UNLOCK();
-    code = afs_CheckCode(code, &treq, 32);
+    code = afs_CheckCode(code, treq, 32);
+    afs_DestroyReq(treq);
     return code;
 }