From owner-cvs-all@FreeBSD.ORG Sun Dec 5 22:30:29 2004 Return-Path: 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 B352F16A514; Sun, 5 Dec 2004 22:30:29 +0000 (GMT) Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115]) by mx1.FreeBSD.org (Postfix) with ESMTP id A373943D53; Sun, 5 Dec 2004 22:30:29 +0000 (GMT) (envelope-from marcel@FreeBSD.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.13.1/8.13.1) with ESMTP id iB5MUTQv021928; Sun, 5 Dec 2004 22:30:29 GMT (envelope-from marcel@repoman.freebsd.org) Received: (from marcel@localhost) by repoman.freebsd.org (8.13.1/8.13.1/Submit) id iB5MUTZs021927; Sun, 5 Dec 2004 22:30:29 GMT (envelope-from marcel) Message-Id: <200412052230.iB5MUTZs021927@repoman.freebsd.org> From: Marcel Moolenaar Date: Sun, 5 Dec 2004 22:30:29 +0000 (UTC) To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org X-FreeBSD-CVS-Branch: HEAD Subject: cvs commit: src/sys/sys vnode.h src/sys/gnu/ext2fs ext2_vnops.c src/sys/nfsclient nfs_vnops.c src/sys/ufs/ufs ufs_vnops.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 05 Dec 2004 22:30:29 -0000 marcel 2004-12-05 22:30:29 UTC FreeBSD src repository Modified files: sys/sys vnode.h sys/gnu/ext2fs ext2_vnops.c sys/nfsclient nfs_vnops.c sys/ufs/ufs ufs_vnops.c Log: Fix null-pointer indirect function calls introduced in the previous commit. In the new world order, the transitive closure on the vector operations is not precomputed. As such, it's unsafe to actually use any of the function pointers in an indirect function call. They can be null, and we need to use the default vector in that case. This is mostly a quick fix for the four function pointers that are ed explicitly. A more generic or scalable solution is likely to see the light of day. No pathos on: current@ Revision Changes Path 1.93 +4 -4 src/sys/gnu/ext2fs/ext2_vnops.c 1.233 +3 -3 src/sys/nfsclient/nfs_vnops.c 1.259 +39 -0 src/sys/sys/vnode.h 1.253 +4 -4 src/sys/ufs/ufs/ufs_vnops.c