Bug #111

vn(4) minor cleanup

Added by csaba.henk about 8 years ago. Updated over 7 years ago.

Status:ClosedStart date:
Priority:LowDue date:
Assignee:-% Done:

0%

Category:-
Target version:-

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>

History

#1 Updated by dillon about 8 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

#2 Updated by trevorjkendall over 7 years ago

Patch commited

Also available in: Atom PDF