From owner-cvs-all  Fri Jun 28 14:53:39 2002
Delivered-To: cvs-all@freebsd.org
Received: from mx1.FreeBSD.org (mx1.FreeBSD.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP
	id 465B637B400; Fri, 28 Jun 2002 14:53:25 -0700 (PDT)
Received: from freefall.freebsd.org (freefall.FreeBSD.org [216.136.204.21])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id A10D543E09; Fri, 28 Jun 2002 14:53:08 -0700 (PDT)
	(envelope-from jhb@FreeBSD.org)
Received: from freefall.freebsd.org (jhb@localhost [127.0.0.1])
	by freefall.freebsd.org (8.12.4/8.12.4) with ESMTP id g5SLr8JU023932;
	Fri, 28 Jun 2002 14:53:08 -0700 (PDT)
	(envelope-from jhb@freefall.freebsd.org)
Received: (from jhb@localhost)
	by freefall.freebsd.org (8.12.4/8.12.4/Submit) id g5SLr8OU023931;
	Fri, 28 Jun 2002 14:53:08 -0700 (PDT)
Message-Id: <200206282153.g5SLr8OU023931@freefall.freebsd.org>
From: John Baldwin <jhb@FreeBSD.org>
Date: Fri, 28 Jun 2002 14:53:08 -0700 (PDT)
To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject: cvs commit: src/sys/nfsclient nfs.h nfs_bio.c nfs_socket.c
         nfs_vnops.c
X-FreeBSD-CVS-Branch: HEAD
Sender: owner-cvs-all@FreeBSD.ORG
Precedence: bulk
List-ID: <cvs-all.FreeBSD.ORG>
List-Archive: <http://docs.freebsd.org/mail/> (Web Archive)
List-Help: <mailto:majordomo@FreeBSD.ORG?subject=help> (List Instructions)
List-Subscribe: <mailto:majordomo@FreeBSD.ORG?subject=subscribe%20cvs-all>
List-Unsubscribe: <mailto:majordomo@FreeBSD.ORG?subject=unsubscribe%20cvs-all>
X-Loop: FreeBSD.ORG

jhb         2002/06/28 14:53:08 PDT

  Modified files:
    sys/nfsclient        nfs.h nfs_bio.c nfs_socket.c nfs_vnops.c 
  Log:
  In namei(), we use a NULL thread for uio_td when doing a VOP_READLINK().
  nfs_readlink() calls nfs_bioread() which passes in uio_td as the thread
  argument to nfs_getcacheblk().  In nfs_getcacheblk() we dereference the
  thread pointer to get a process pointer to pass to nfs_sigintr().  This
  obviously results in a panic. :)
  
  Rather than change nfs_getcacheblk() to check if the thread pointer is
  NULL when calling nfs_sigintr() like other callers do, change
  nfs_sigintr() to take a thread as the last argument instead of a
  process so none of the callers have to care if the thread is NULL or not.
  
  Revision  Changes    Path
  1.70      +1 -1      src/sys/nfsclient/nfs.h
  1.108     +4 -4      src/sys/nfsclient/nfs_bio.c
  1.82      +8 -9      src/sys/nfsclient/nfs_socket.c
  1.183     +2 -2      src/sys/nfsclient/nfs_vnops.c

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