Project

General

Profile

Actions

Bug #111

closed

vn(4) minor cleanup

Added by csaba.henk over 15 years ago. Updated about 15 years ago.

Status:
Closed
Priority:
Low
Assignee:
-
Category:
-
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:

Description

Stale comment and duplicate header inclusion removed.

Regards,
Csaba

diff r b2691ba1206a sys/dev/disk/vn/vn.c
--
a/sys/dev/disk/vn/vn.c Mon Mar 13 14:36:46 2006 0000
++ b/sys/dev/disk/vn/vn.c Tue Mar 14 14:03:28 2006 +0100
@ -48,17 +48,13 @ * Block/character interface to a vnode. Allows one to treat a file * as a disk (e.g. build a filesystem in it, mount it, etc.). *
- * NOTE 1: This uses the VOP_BMAP/VOP_STRATEGY interface to the vnode
- * instead of a simple VOP_RDWR. We do this to avoid distorting the
- * local buffer cache.
- *
- * NOTE 2: There is a security issue involved with this driver.
+ * NOTE 1: There is a security issue involved with this driver. * Once mounted all access to the contents of the "mapped" file via * the special file is controlled by the permissions on the special * file, the protection of the mapped file is ignored (effectively, * by using root credentials in all transactions). *
- * NOTE 3: Doesn't interact with leases, should it?
+ * NOTE 2: Doesn't interact with leases, should it?
*/

#include <sys/param.h>
@ -75,7 +71,6 @
#include <sys/disklabel.h>
#include <sys/diskslice.h>
#include <sys/stat.h>
-#include <sys/conf.h>
#include <sys/module.h>
#include <sys/vnioctl.h>
Actions #1

Updated by dillon over 15 years ago

:Stale comment and duplicate header inclusion removed.
:
:Regards,
:Csaba

I have incorporated this patch into the BUF/BIO patch set, so it
will get committed along with that this coming Friday.
-Matt
Actions #2

Updated by trevorjkendall over 15 years ago

Patch commited

Actions

Also available in: Atom PDF