OPENAFS-SA-2016-002 AFSStoreStatus information leak
[openafs.git] / src / afs / VNOPS / afs_vnop_symlink.c
index d626f30..f608b2b 100644 (file)
  */
 
 #include <afsconfig.h>
-#include "../afs/param.h"
+#include "afs/param.h"
 
-RCSID("$Header$");
 
-#include "../afs/sysincludes.h"        /* Standard vendor system headers */
-#include "../afs/afsincludes.h"        /* Afs-based standard headers */
-#include "../afs/afs_stats.h" /* statistics */
-#include "../afs/afs_cbqueue.h"
-#include "../afs/nfsclient.h"
-#include "../afs/afs_osidnlc.h"
+#include "afs/sysincludes.h"   /* Standard vendor system headers */
+#include "afsincludes.h"       /* Afs-based standard headers */
+#include "afs/afs_stats.h"     /* statistics */
+#include "afs/afs_cbqueue.h"
+#include "afs/nfsclient.h"
+#include "afs/afs_osidnlc.h"
 
 extern afs_rwlock_t afs_xvcache;
 extern afs_rwlock_t afs_xcbhash;
@@ -38,53 +37,73 @@ 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;
+
+    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);
+    afs_AdjustSize(tdc, len);
+    tdc->validPos = len;
+    tfile = afs_CFileOpen(&tdc->f.inode);
+    afs_CFileWrite(tfile, 0, aname, len);
+    afs_CFileClose(tfile);
+    ReleaseWriteLock(&tdc->lock);
+    return 0;
+}
 
 /* don't set CDirty in here because RPC is called synchronously */
-afs_symlink
-#ifdef AFS_OSF_ENV
-  (ndp, attrs, atargetName)
-    struct nameidata *ndp;
-    struct vattr *attrs;
-    register char *atargetName; {
-    register struct vcache *adp = VTOAFS(ndp->ni_dvp);
-    char *aname = ndp->ni_dent.d_name;
-    struct ucred *acred = ndp->ni_cred;
-#else  /* AFS_OSF_ENV */
-  (OSI_VC_ARG(adp), aname, attrs, atargetName, acred)
-    OSI_VC_DECL(adp);
-    char *atargetName;
-    char *aname;
-    struct vattr *attrs;
-    struct AFS_UCRED *acred; {
-#endif
-    afs_uint32 now;
-    struct vrequest treq;
-    afs_int32 code;
-    struct conn *tc;
+int 
+afs_symlink(OSI_VC_DECL(adp), char *aname, struct vattr *attrs, 
+           char *atargetName, struct vcache **tvcp, afs_ucred_t *acred)
+{
+    afs_uint32 now = 0;
+    struct vrequest *treq = NULL;
+    afs_int32 code = 0;
+    struct afs_conn *tc;
     struct VenusFid newFid;
     struct dcache *tdc;
     afs_size_t offset, len;
     afs_int32 alen;
-    struct server *hostp=0;
+    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 volume *volp = 0;
     struct afs_fakestat_state fakestate;
-    XSTATS_DECLS
-    OSI_VC_CONVERT(adp)
-    
+    struct rx_connection *rxconn;
+    XSTATS_DECLS;
+    OSI_VC_CONVERT(adp);
+
     AFS_STATCNT(afs_symlink);
     afs_Trace2(afs_iclSetp, CM_TRACE_SYMLINK, ICL_TYPE_POINTER, adp,
-                ICL_TYPE_STRING, aname);
+              ICL_TYPE_STRING, aname);
+
+    OutFidStatus = osi_AllocSmallSpace(sizeof(struct AFSFetchStatus));
+    OutDirStatus = osi_AllocSmallSpace(sizeof(struct AFSFetchStatus));
+    memset(&InStatus, 0, sizeof(InStatus));
 
-    if (code = afs_InitReq(&treq, acred))
+    if ((code = afs_CreateReq(&treq, acred)))
        goto done2;
 
     afs_InitFakeStat(&fakestate);
-    code = afs_EvalFakeStat(&adp, &fakestate, &treq);
+
+    AFS_DISCON_LOCK();
+    
+    code = afs_EvalFakeStat(&adp, &fakestate, treq);
     if (code)
        goto done;
 
@@ -97,79 +116,95 @@ afs_symlink
        code = afs_DynrootVOPSymlink(adp, acred, aname, atargetName);
        goto done;
     }
+    if (afs_IsDynrootMount(adp)) {
+       code = EROFS;
+       goto done;
+    }
 
-    code = afs_VerifyVCache(adp, &treq);
-    if (code) { 
-       code = afs_CheckCode(code, &treq, 30);
+    code = afs_VerifyVCache(adp, treq);
+    if (code) {
+       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 ) {
-        code = EROFS;
+    if (adp->f.states & CRO) {
+       code = EROFS;
        goto done;
     }
 
+    if (AFS_IS_DISCONNECTED && !AFS_IS_DISCON_RW) {
+        code = ENETDOWN;
+        goto done;
+    }
+    
     InStatus.Mask = AFS_SETMODTIME | AFS_SETMODE;
     InStatus.ClientModTime = osi_Time();
-    alen = strlen(atargetName);            /* we want it to include the null */
-    if (*atargetName == '#' || *atargetName == '%') {
+    alen = strlen(atargetName);        /* we want it to include the null */
+    if ( (*atargetName == '#' || *atargetName == '%') && alen > 1 && atargetName[alen-1] == '.') {
        InStatus.UnixModeBits = 0644;   /* mt pt: null from "." at end */
-        if (alen == 1) alen++; /* Empty string */
+       if (alen == 1)
+           alen++;             /* Empty string */
     } else {
        InStatus.UnixModeBits = 0755;
-       alen++;     /* add in the null */
+       alen++;                 /* add in the null */
+    }
+    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);
+    /* 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);
     }
-    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*/
-    ObtainWriteLock(&adp->lock,156);
-    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();
-#ifdef RX_ENABLE_LOCKS
-               AFS_GUNLOCK();
-#endif /* RX_ENABLE_LOCKS */
-               code = RXAFS_DFSSymlink(tc->id, (struct AFSFid *) &adp->fid.Fid, aname,
-                                    atargetName, &InStatus, (struct AFSFid *) &newFid.Fid,
-                                    &OutFidStatus, &OutDirStatus, &CallBack, &tsync);
-#ifdef RX_ENABLE_LOCKS
-               AFS_GLOCK();
-#endif /* RX_ENABLE_LOCKS */
-           } else {
-#ifdef RX_ENABLE_LOCKS
-               AFS_GUNLOCK();
-#endif /* RX_ENABLE_LOCKS */
-               code = RXAFS_Symlink(tc->id, (struct AFSFid *) &adp->fid.Fid, aname,
-                                    atargetName, &InStatus, (struct AFSFid *) &newFid.Fid,
-                                    &OutFidStatus, &OutDirStatus, &tsync);
-#ifdef RX_ENABLE_LOCKS
-               AFS_GLOCK();
-#endif /* RX_ENABLE_LOCKS */
-           }
-          XSTATS_END_TIME;
-       }
-       else code = -1;
-    } while 
-       (afs_Analyze(tc, code, &adp->fid, &treq,
-                   AFS_STATS_FS_RPCIDX_SYMLINK, SHARED_LOCK, (struct cell *)0));
 
-    UpgradeSToWLock(&afs_xvcache,40);
+    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);
+           ObtainWriteLock(&afs_xcbhash, 499);
+           afs_DequeueCallback(adp);
+           adp->f.states &= ~CStatd;
+           ReleaseWriteLock(&afs_xcbhash);
+           osi_dnlc_purgedp(adp);
        }
        ReleaseWriteLock(&adp->lock);
        ReleaseWriteLock(&afs_xvcache);
@@ -180,135 +215,113 @@ afs_symlink
        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 */
-       code = afs_dir_Create(&tdc->f.inode, aname, &newFid.Fid);
-       if (code) { 
+       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 && !AFS_IS_DISCON_RW) {
            ZapDCE(tdc);        /* surprise error -- use invalid value */
-           DZap(&tdc->f.inode);
+           DZap(tdc);
        }
     }
     if (tdc) {
        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
      * no one can get a pointer to the new cache entry until we release 
      * the xvcache lock. */
-    tvc = afs_NewVCache(&newFid, hostp, 1, WRITE_LOCK);
-    ObtainWriteLock(&tvc->lock,157);
+    tvc = afs_NewVCache(&newFid, hostp);
+    if (!tvc)
+    {
+       code = -2;
+       ReleaseWriteLock(&afs_xvcache);
+       goto done;
+    }
+    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 */
-        tvc->cbExpires = CallBack.ExpirationTime + now;
-        afs_QueueCallback(tvc, CBHash(CallBack.ExpirationTime), volp);
+       tvc->cbExpires = CallBack.ExpirationTime + now;
+       afs_QueueCallback(tvc, CBHash(CallBack.ExpirationTime), volp);
     } else {
        tvc->cbExpires = 0x7fffffff;    /* never expires, they can't change */
-     /* since it never expires, we don't have to queue the callback */
+       /* 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);
-       strncpy(tvc->linkData, atargetName, alen-1);
-       tvc->linkData[alen-1] = 0;
+       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, WRITE_LOCK);
+    if (tvcp)
+       *tvcp = tvc;
+    else
+       afs_PutVCache(tvc);
     code = 0;
-done:
+  done:
     afs_PutFakeStat(&fakestate);
-    if ( volp ) 
+    if (volp)
        afs_PutVolume(volp, READ_LOCK);
-    code = afs_CheckCode(code, &treq, 31);
-done2:
-#ifdef  AFS_OSF_ENV
-    AFS_RELE(ndp->ni_dvp);
-#endif  /* AFS_OSF_ENV */
+    AFS_DISCON_UNLOCK();
+    code = afs_CheckCode(code, treq, 31);
+    afs_DestroyReq(treq);
+  done2:
+    osi_FreeSmallSpace(OutFidStatus);
+    osi_FreeSmallSpace(OutDirStatus);
     return code;
 }
 
-afs_MemHandleLink(avc, areq)
-     register struct vcache *avc;
-     struct vrequest *areq;
-  {
-      register struct dcache *tdc;
-      register char *tp, *rbuf;
-      afs_size_t offset, len;
-      afs_int32 tlen, alen;
-      register afs_int32 code;
-
-      AFS_STATCNT(afs_MemHandleLink);
-      /* 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 755, we add a null to the end of */
-      if (!avc->linkData) {
-         void *addr;
-         tdc = afs_GetDCache(avc, (afs_size_t) 0, areq, &offset, &len, 0);
-         if (!tdc) {
-             return EIO;
-         }
-         /* otherwise we have the data loaded, go for it */
-         if (len > 1024) {
-             afs_PutDCache(tdc);
-             return EFAULT;
-         }
-         if (avc->m.Mode       & 0111) alen = len+1;   /* regular link */
-         else alen = len;                      /* mt point */
-         rbuf = (char *) osi_AllocLargeSpace(AFS_LRALLOCSIZ);
-         ObtainReadLock(&tdc->lock);
-          addr = afs_MemCacheOpen(tdc->f.inode);
-         tlen = len;
-          code = afs_MemReadBlk(addr, 0, rbuf, tlen);
-         afs_MemCacheClose(addr);
-         ReleaseReadLock(&tdc->lock);
-         afs_PutDCache(tdc);
-         rbuf[alen-1] = 0;
-         alen = strlen(rbuf) + 1;
-         tp = afs_osi_Alloc(alen); /* make room for terminating null */
-         memcpy(tp, rbuf, alen);
-         osi_FreeLargeSpace(rbuf);
-         if (code != len) {
-             afs_osi_Free(tp, alen);
-             return EIO;
-         }
-         avc->linkData = tp;
-      }
-      return 0;
-  }
-
-afs_UFSHandleLink(avc, areq)
-    register struct vcache *avc;
-    struct vrequest *areq; 
+int
+afs_MemHandleLink(struct vcache *avc, struct vrequest *areq)
 {
-    register struct dcache *tdc;
-    register char *tp, *rbuf;
-    char *tfile;
+    struct dcache *tdc;
+    char *tp, *rbuf;
     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 755,
-       we add a null to the end of */
-   AFS_STATCNT(afs_UFSHandleLink);
+    AFS_STATCNT(afs_MemHandleLink);
+    /* 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 755, we add a null to the end of */
     if (!avc->linkData) {
+       void *addr;
        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));
        if (!tdc) {
            return EIO;
        }
@@ -317,19 +330,85 @@ afs_UFSHandleLink(avc, areq)
            afs_PutDCache(tdc);
            return EFAULT;
        }
-       if (avc->m.Mode & 0111) alen = len+1;   /* regular link */
-       else alen = len;                        /* mt point */
-       rbuf = (char *) osi_AllocLargeSpace(AFS_LRALLOCSIZ);
+       if (avc->f.m.Mode & 0111)
+           alen = len + 1;     /* regular link */
+       else
+           alen = len;         /* mt point */
+       rbuf = osi_AllocLargeSpace(AFS_LRALLOCSIZ);
+       ObtainReadLock(&tdc->lock);
+       addr = afs_MemCacheOpen(&tdc->f.inode);
+       tlen = len;
+       code = afs_MemReadBlk(addr, 0, rbuf, tlen);
+       afs_MemCacheClose(addr);
+       ReleaseReadLock(&tdc->lock);
+       afs_PutDCache(tdc);
+       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) {
+           afs_osi_Free(tp, alen);
+           return EIO;
+       }
+       avc->linkData = tp;
+    }
+    return 0;
+}
+
+int
+afs_UFSHandleLink(struct vcache *avc, struct vrequest *areq)
+{
+    struct dcache *tdc;
+    char *tp, *rbuf;
+    void *tfile;
+    afs_size_t offset, len;
+    afs_int32 tlen, alen;
+    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
+     * 755, we add a null to the end of */
+    AFS_STATCNT(afs_UFSHandleLink);
+    if (!avc->linkData) {
+       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->f.m.Length));
+       if (!tdc) {
+           if (AFS_IS_DISCONNECTED)
+               return ENETDOWN;
+           else
+               return EIO;
+       }
+       /* otherwise we have the data loaded, go for it */
+       if (len > 1024) {
+           afs_PutDCache(tdc);
+           return EFAULT;
+       }
+       if (avc->f.m.Mode & 0111)
+           alen = len + 1;     /* regular link */
+       else
+           alen = len;         /* mt point */
+       rbuf = osi_AllocLargeSpace(AFS_LRALLOCSIZ);
        tlen = len;
        ObtainReadLock(&tdc->lock);
-       tfile = osi_UFSOpen (tdc->f.inode);
+       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);
        afs_PutDCache(tdc);
-       rbuf[alen-1] = '\0';
+       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) {
@@ -341,42 +420,49 @@ afs_UFSHandleLink(avc, areq)
     return 0;
 }
 
-afs_readlink(OSI_VC_ARG(avc), auio, acred)
-    OSI_VC_DECL(avc);
-    struct uio *auio;
-    struct AFS_UCRED *acred; 
+int
+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)
+    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)) return code;
+    if ((code = afs_CreateReq(&treq, acred)))
+       return code;
     afs_InitFakeStat(&fakestat);
-    code = afs_EvalFakeStat(&avc, &fakestat, &treq);
-    if (code) goto done;
-    code = afs_VerifyVCache(avc, &treq);
-    if (code) goto done;
+
+    AFS_DISCON_LOCK();
+    
+    code = afs_EvalFakeStat(&avc, &fakestat, treq);
+    if (code)
+       goto done;
+    code = afs_VerifyVCache(avc, treq);
+    if (code)
+       goto done;
     if (vType(avc) != VLNK) {
        code = EINVAL;
        goto done;
     }
-    ObtainWriteLock(&avc->lock,158);
-    code = afs_HandleLink(avc, &treq);
+    ObtainWriteLock(&avc->lock, 158);
+    code = afs_HandleLink(avc, treq);
     /* finally uiomove it to user-land */
     if (code == 0) {
        tp = avc->linkData;
-       if (tp) AFS_UIOMOVE(tp, strlen(tp), UIO_READ, auio, code);
+       if (tp)
+           AFS_UIOMOVE(tp, strlen(tp), UIO_READ, auio, code);
        else {
            code = EIO;
        }
     }
     ReleaseWriteLock(&avc->lock);
-done:
+  done:
     afs_PutFakeStat(&fakestat);
-    code = afs_CheckCode(code, &treq, 32);
+    AFS_DISCON_UNLOCK();
+    code = afs_CheckCode(code, treq, 32);
+    afs_DestroyReq(treq);
     return code;
 }