From owner-freebsd-hackers@FreeBSD.ORG  Tue Aug 10 23:51:16 2004
Return-Path: <owner-freebsd-hackers@FreeBSD.ORG>
Delivered-To: freebsd-hackers@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 8305116A4CE
	for <hackers@freebsd.org>; Tue, 10 Aug 2004 23:51:16 +0000 (GMT)
Received: from coverity.dreamhost.com (coverity.dreamhost.com [66.33.192.105])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 77E1843D2F
	for <hackers@freebsd.org>; Tue, 10 Aug 2004 23:51:16 +0000 (GMT)
	(envelope-from tedu@coverity.com)
Received: from coverity.com (dsl093-171-098.sfo4.dsl.speakeasy.net
	[66.93.171.98])
	by coverity.dreamhost.com (Postfix) with ESMTP id 34F2590888
	for <hackers@freebsd.org>; Tue, 10 Aug 2004 16:51:16 -0700 (PDT)
Message-ID: <41195E44.6070207@coverity.com>
Date: Tue, 10 Aug 2004 16:46:12 -0700
From: Ted Unangst <tedu@coverity.com>
User-Agent: Mozilla/5.0 (X11; U; FreeBSD i386; en-US; rv:1.6) Gecko/20040429
X-Accept-Language: en-us, en
MIME-Version: 1.0
To: hackers@freebsd.org
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit
X-Mailman-Approved-At: Wed, 11 Aug 2004 12:43:44 +0000
Subject: buglet in nfs_mountroot
X-BeenThere: freebsd-hackers@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: Technical Discussions relating to FreeBSD
	<freebsd-hackers.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-hackers>,
	<mailto:freebsd-hackers-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-hackers>
List-Post: <mailto:freebsd-hackers@freebsd.org>
List-Help: <mailto:freebsd-hackers-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-hackers>,
	<mailto:freebsd-hackers-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 10 Aug 2004 23:51:16 -0000

in freebsd 4.10 nfs/nfs_vfsops.c:nfs_mountroot, the handling of swap_mp 
is incorrect.  the code at line 498 appears useless, and furthermore 
uses swap_mp without initialization.