Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 7 Mar 2003 18:19:33 +1100
From:      Tim Robbins <tjr@FreeBSD.org>
To:        Jeff Roberson <jroberson@chesapeake.net>
Cc:        Robert Watson <rwatson@FreeBSD.org>, src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/coda coda_vnops.c
Message-ID:  <20030307181933.C19473@dilbert.robbins.dropbear.id.au>
In-Reply-To: <20030307013326.W47958-100000@mail.chesapeake.net>; from jroberson@chesapeake.net on Fri, Mar 07, 2003 at 01:33:52AM -0500
References:  <Pine.NEB.3.96L.1030305112755.56689A-100000@fledge.watson.org> <20030307013326.W47958-100000@mail.chesapeake.net>

next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, Mar 07, 2003 at 01:33:52AM -0500, Jeff Roberson wrote:

> On Wed, 5 Mar 2003, Robert Watson wrote:
> 
> >
> > > tjr         2003/03/05 01:52:04 PST
> > >   Log:
> > >   Handle the case where a_uio->uio_td == NULL properly in coda_readlink().
> > >   This happens when called from lookup().
> >
> > Hmm.  An interesting question is whether this should actually be allowed
> > to happen.
> 
> Yeah, thats what I thought when I saw this too.  Any thoughts on this tjr?

I don't see why it can't use the thread from the struct componentname.
BTW, the call to VOP_READLINK with uio_td == NULL is in namei(), not
lookup(), the commit message was wrong.


Tim

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20030307181933.C19473>