From: pelikan <pelikan@openbsd.org>
Date: Thu, 31 Jul 2014 17:37:52 +0000 (+0000)
Subject: use proper on-disk inode size: no more, no less.
X-Git-Url: http://artulab.com/gitweb/?a=commitdiff_plain;h=d31d4a6efc77350e848422a4d0fc38384984d19d;p=openbsd

use proper on-disk inode size: no more, no less.

Reported by Roman Yakovlev, thanks!

"do it now" deraadt
---

diff --git a/sys/ufs/ext2fs/ext2fs_bswap.c b/sys/ufs/ext2fs/ext2fs_bswap.c
index 720310cee7f..c4f36ba3d86 100644
--- a/sys/ufs/ext2fs/ext2fs_bswap.c
+++ b/sys/ufs/ext2fs/ext2fs_bswap.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: ext2fs_bswap.c,v 1.7 2014/07/14 08:54:13 pelikan Exp $	*/
+/*	$OpenBSD: ext2fs_bswap.c,v 1.8 2014/07/31 17:37:52 pelikan Exp $	*/
 /*	$NetBSD: ext2fs_bswap.c,v 1.6 2000/07/24 00:23:10 mycroft Exp $	*/
 
 /*
@@ -110,7 +110,8 @@ e2fs_cg_bswap(struct ext2_gd *old, struct ext2_gd *new, int size)
 }
 
 void
-e2fs_i_bswap(struct ext2fs_dinode *old, struct ext2fs_dinode *new)
+e2fs_i_bswap(struct m_ext2fs *fs, struct ext2fs_dinode *old,
+    struct ext2fs_dinode *new)
 {
 	new->e2di_mode		=	swap16(old->e2di_mode);
 	new->e2di_uid_low	=	swap16(old->e2di_uid_low);
@@ -133,5 +134,9 @@ e2fs_i_bswap(struct ext2fs_dinode *old, struct ext2fs_dinode *new)
 	new->e2di_facl_hi	=	swap16(old->e2di_facl_hi);
 	memcpy(&new->e2di_blocks[0], &old->e2di_blocks[0],
 		(NDADDR+NIADDR) * sizeof(int));
+
+	if (EXT2_DINODE_SIZE(fs) <= EXT2_REV0_DINODE_SIZE)
+		return;
+	new->e2di_isize		=	swap16(old->e2di_isize);
 }
 #endif
diff --git a/sys/ufs/ext2fs/ext2fs_dinode.h b/sys/ufs/ext2fs/ext2fs_dinode.h
index 456e695812c..4357be5b9ef 100644
--- a/sys/ufs/ext2fs/ext2fs_dinode.h
+++ b/sys/ufs/ext2fs/ext2fs_dinode.h
@@ -1,4 +1,4 @@
-/*	$OpenBSD: ext2fs_dinode.h,v 1.16 2014/07/11 12:53:19 pelikan Exp $	*/
+/*	$OpenBSD: ext2fs_dinode.h,v 1.17 2014/07/31 17:37:52 pelikan Exp $	*/
 /*	$NetBSD: ext2fs_dinode.h,v 1.6 2000/01/26 16:21:33 bouyer Exp $	*/
 
 /*
@@ -154,10 +154,13 @@ struct ext2fs_dinode {
 
 /* e2fs needs byte swapping on big-endian systems */
 #if BYTE_ORDER == LITTLE_ENDIAN
-#	define e2fs_iload(old, new) memcpy((new),(old),sizeof(struct ext2fs_dinode))
-#	define e2fs_isave(old, new) memcpy((new),(old),sizeof(struct ext2fs_dinode))
+#	define e2fs_iload(fs, old, new)	\
+		memcpy((new),(old), MIN(EXT2_DINODE_SIZE(fs), sizeof(*new)))
+#	define e2fs_isave(fs, old, new) \
+		memcpy((new),(old), MIN(EXT2_DINODE_SIZE(fs), sizeof(*new)))
 #else
-void e2fs_i_bswap(struct ext2fs_dinode *, struct ext2fs_dinode *);
-#	define e2fs_iload(old, new) e2fs_i_bswap((old), (new))
-#	define e2fs_isave(old, new) e2fs_i_bswap((old), (new))
+struct m_ext2fs;
+void e2fs_i_bswap(struct m_ext2fs *, struct ext2fs_dinode *, struct ext2fs_dinode *);
+#	define e2fs_iload(fs, old, new) e2fs_i_bswap((fs), (old), (new))
+#	define e2fs_isave(fs, old, new) e2fs_i_bswap((fs), (old), (new))
 #endif
diff --git a/sys/ufs/ext2fs/ext2fs_inode.c b/sys/ufs/ext2fs/ext2fs_inode.c
index 6f883499d29..41d72bc6a31 100644
--- a/sys/ufs/ext2fs/ext2fs_inode.c
+++ b/sys/ufs/ext2fs/ext2fs_inode.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: ext2fs_inode.c,v 1.54 2014/07/13 16:59:35 pelikan Exp $	*/
+/*	$OpenBSD: ext2fs_inode.c,v 1.55 2014/07/31 17:37:52 pelikan Exp $	*/
 /*	$NetBSD: ext2fs_inode.c,v 1.24 2001/06/19 12:59:18 wiz Exp $	*/
 
 /*
@@ -189,7 +189,7 @@ ext2fs_update(struct inode *ip, int waitfor)
 	ip->i_e2fs_uid_high = ip->i_e2fs_uid >> 16;
 	ip->i_e2fs_gid_high = ip->i_e2fs_gid >> 16;
 
-	e2fs_isave(ip->i_e2din, (struct ext2fs_dinode *)cp);
+	e2fs_isave(fs, ip->i_e2din, (struct ext2fs_dinode *)cp);
 	if (waitfor)
 		return (bwrite(bp));
 	else {
diff --git a/sys/ufs/ext2fs/ext2fs_vfsops.c b/sys/ufs/ext2fs/ext2fs_vfsops.c
index 5db43958c55..e4f54fb5983 100644
--- a/sys/ufs/ext2fs/ext2fs_vfsops.c
+++ b/sys/ufs/ext2fs/ext2fs_vfsops.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: ext2fs_vfsops.c,v 1.81 2014/07/13 15:07:01 pelikan Exp $	*/
+/*	$OpenBSD: ext2fs_vfsops.c,v 1.82 2014/07/31 17:37:52 pelikan Exp $	*/
 /*	$NetBSD: ext2fs_vfsops.c,v 1.1 1997/06/11 09:34:07 bouyer Exp $	*/
 
 /*
@@ -365,7 +365,7 @@ ext2fs_reload_vnode(struct vnode *vp, void *args)
 	}
 	cp = (caddr_t)bp->b_data +
 	    (ino_to_fsbo(era->fs, ip->i_number) * EXT2_DINODE_SIZE(era->fs));
-	e2fs_iload((struct ext2fs_dinode *)cp, ip->i_e2din);
+	e2fs_iload(era->fs, (struct ext2fs_dinode *)cp, ip->i_e2din);
 	brelse(bp);
 	vput(vp);
 	return (0);
@@ -891,7 +891,7 @@ ext2fs_vget(struct mount *mp, ino_t ino, struct vnode **vpp)
 	    + EXT2_DINODE_SIZE(fs) * ino_to_fsbo(fs, ino));
 
 	ip->i_e2din = pool_get(&ext2fs_dinode_pool, PR_WAITOK);
-	e2fs_iload(dp, ip->i_e2din);
+	e2fs_iload(fs, dp, ip->i_e2din);
 	brelse(bp);
 
 	ip->i_effnlink = ip->i_e2fs_nlink;