Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 2 Feb 2023 23:49:27 GMT
From:      Matthias Andree <mandree@FreeBSD.org>
To:        ports-committers@FreeBSD.org, dev-commits-ports-all@FreeBSD.org, dev-commits-ports-main@FreeBSD.org
Subject:   git: 594ba3e582e7 - main - sysutils/e2fsprogs: bugfix update to 1.46.6
Message-ID:  <202302022349.312NnR82049287@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by mandree:

URL: https://cgit.FreeBSD.org/ports/commit/?id=594ba3e582e75df1ad69e7b039dadb16e164e843

commit 594ba3e582e75df1ad69e7b039dadb16e164e843
Author:     Matthias Andree <mandree@FreeBSD.org>
AuthorDate: 2023-02-02 23:25:53 +0000
Commit:     Matthias Andree <mandree@FreeBSD.org>
CommitDate: 2023-02-02 23:49:24 +0000

    sysutils/e2fsprogs: bugfix update to 1.46.6
    
    ChangeLog: https://e2fsprogs.sourceforge.net/e2fsprogs-release.html#1.46.6
    
    Add patch to make m_offset reliable. Due to some race conditions on the
    pipeline between FreeBSD's yes a, which emits 8190-long writes on
    13-STABLE, and GNU dd (gdd), which it pipes its output into, gdd can
    sometimes receive short reads occasionally, and because the count=
    normally is the number of read(2) calls dispatched, this then leads to a
    shorter reference "file system" that we crcsum, and that causes false
    negative test results.  gdd warns that iflag=fullblock would avoid
    effects of short reads, but this warning is discarded in the test
    script.  Anyways, patch m_offset to use dd's and gdd's iflag=fullblock,
    which causes (g)dd to call read(2) again until the block is full for
    each nominal block, so we definitely read the desired output size 512 kB
    no matter what shorter blocks might travel through the pipeline.
    
    MFH:            2023Q1
---
 misc/e2fsprogs-libblkid/Makefile                   |  2 +-
 misc/e2fsprogs-libuuid/Makefile                    |  2 +-
 sysutils/e2fsprogs-core/Makefile                   |  4 +-
 sysutils/e2fsprogs-core/distinfo                   |  6 +--
 sysutils/e2fsprogs-core/files/patch-CVE-2022-1304  | 57 ----------------------
 sysutils/e2fsprogs-core/files/patch-ZZZ-g8535250a  | 26 ----------
 .../files/patch-tests_m__offset_script             | 11 +++++
 sysutils/e2fsprogs-core/pkg-plist                  |  1 +
 sysutils/e2fsprogs/Makefile                        |  4 +-
 9 files changed, 21 insertions(+), 92 deletions(-)

diff --git a/misc/e2fsprogs-libblkid/Makefile b/misc/e2fsprogs-libblkid/Makefile
index 40e8a7e25a15..2d50de7e985e 100644
--- a/misc/e2fsprogs-libblkid/Makefile
+++ b/misc/e2fsprogs-libblkid/Makefile
@@ -1,4 +1,4 @@
-PORTREVISION=	2
+PORTREVISION=	0
 CATEGORIES=	misc devel
 PKGNAMESUFFIX=	-libblkid
 
diff --git a/misc/e2fsprogs-libuuid/Makefile b/misc/e2fsprogs-libuuid/Makefile
index 98e76cf896a3..129c636e3acb 100644
--- a/misc/e2fsprogs-libuuid/Makefile
+++ b/misc/e2fsprogs-libuuid/Makefile
@@ -1,4 +1,4 @@
-PORTREVISION=	1
+PORTREVISION=	0
 CATEGORIES=	misc devel
 PKGNAMESUFFIX=	-libuuid
 
diff --git a/sysutils/e2fsprogs-core/Makefile b/sysutils/e2fsprogs-core/Makefile
index 484310867d13..16957df9fc08 100644
--- a/sysutils/e2fsprogs-core/Makefile
+++ b/sysutils/e2fsprogs-core/Makefile
@@ -1,6 +1,6 @@
 PORTNAME=	e2fsprogs
-PORTVERSION=	1.46.5
-PORTREVISION?=	5
+PORTVERSION=	1.46.6
+PORTREVISION?=	0
 CATEGORIES?=	sysutils
 MASTER_SITES=	KERNEL_ORG/linux/kernel/people/tytso/${PORTNAME}/v${PORTVERSION}
 
diff --git a/sysutils/e2fsprogs-core/distinfo b/sysutils/e2fsprogs-core/distinfo
index 0f7dd34cc74b..cf4610cb1968 100644
--- a/sysutils/e2fsprogs-core/distinfo
+++ b/sysutils/e2fsprogs-core/distinfo
@@ -1,3 +1,3 @@
-TIMESTAMP = 1643233594
-SHA256 (e2fsprogs-1.46.5.tar.xz) = 2f16c9176704cf645dc69d5b15ff704ae722d665df38b2ed3cfc249757d8d81e
-SIZE (e2fsprogs-1.46.5.tar.xz) = 7040672
+TIMESTAMP = 1675376753
+SHA256 (e2fsprogs-1.46.6.tar.xz) = a77517f19ff5e4e97ede63536566865dd5d48654e13fc145f5f2249ef7c4f4fc
+SIZE (e2fsprogs-1.46.6.tar.xz) = 7051160
diff --git a/sysutils/e2fsprogs-core/files/patch-CVE-2022-1304 b/sysutils/e2fsprogs-core/files/patch-CVE-2022-1304
deleted file mode 100644
index dec6e97b76be..000000000000
--- a/sysutils/e2fsprogs-core/files/patch-CVE-2022-1304
+++ /dev/null
@@ -1,57 +0,0 @@
-From:   Lukas Czerner <lczerner@redhat.com>
-To:     linux-ext4@vger.kernel.org
-Cc:     tytso@mit.edu, Nils Bars <nils_bars@t-online.de>
-Subject: [PATCH] e2fsprogs: add sanity check to extent manipulation
-Date:   Thu, 21 Apr 2022 19:31:48 +0200
-Message-Id: <20220421173148.20193-1-lczerner@redhat.com>
-List-ID: <linux-ext4.vger.kernel.org>
-X-Mailing-List: linux-ext4@vger.kernel.org
-
-It is possible to have a corrupted extent tree in such a way that a leaf
-node contains zero extents in it. Currently if that happens and we try
-to traverse the tree we can end up accessing wrong data, or possibly
-even uninitialized memory. Make sure we don't do that.
-
-Additionally make sure that we have a sane number of bytes passed to
-memmove() in ext2fs_extent_delete().
-
-Note that e2fsck is currently unable to spot and fix such corruption in
-pass1.
-
-Signed-off-by: Lukas Czerner <lczerner@redhat.com>
-Reported-by: Nils Bars <nils_bars@t-online.de>
-Addressess: https://bugzilla.redhat.com/show_bug.cgi?id=2068113
----
- lib/ext2fs/extent.c | 8 ++++++++
- 1 file changed, 8 insertions(+)
-
-diff --git a/lib/ext2fs/extent.c b/lib/ext2fs/extent.c
-index b324c7b0..1a206a16 100644
---- ./lib/ext2fs/extent.c
-+++ b/lib/ext2fs/extent.c
-@@ -495,6 +495,10 @@ retry:
- 			ext2fs_le16_to_cpu(eh->eh_entries);
- 		newpath->max_entries = ext2fs_le16_to_cpu(eh->eh_max);
- 
-+		/* Make sure there is at least one extent present */
-+		if (newpath->left <= 0)
-+			return EXT2_ET_EXTENT_NO_DOWN;
-+
- 		if (path->left > 0) {
- 			ix++;
- 			newpath->end_blk = ext2fs_le32_to_cpu(ix->ei_block);
-@@ -1630,6 +1634,10 @@ errcode_t ext2fs_extent_delete(ext2_extent_handle_t handle, int flags)
- 
- 	cp = path->curr;
- 
-+	/* Sanity check before memmove() */
-+	if (path->left < 0)
-+		return EXT2_ET_EXTENT_LEAF_BAD;
-+
- 	if (path->left) {
- 		memmove(cp, cp + sizeof(struct ext3_extent_idx),
- 			path->left * sizeof(struct ext3_extent_idx));
--- 
-2.35.1
-
-
diff --git a/sysutils/e2fsprogs-core/files/patch-ZZZ-g8535250a b/sysutils/e2fsprogs-core/files/patch-ZZZ-g8535250a
deleted file mode 100644
index 72820e4b67e6..000000000000
--- a/sysutils/e2fsprogs-core/files/patch-ZZZ-g8535250a
+++ /dev/null
@@ -1,26 +0,0 @@
-commit 8535250a8eeeb441cf466124cc420724935022eb
-Author: Theodore Ts'o <tytso@mit.edu>
-Date:   Mon Jan 3 22:45:37 2022 -0500
-
-    tests: support older versions of timeout in r_corrupt_fs
-    
-    Older versions of the timeout program in coreutils don't support the
-    -v option.  (This is apparently still in use in the GNU/FreeBSD Debain
-    port since coreutils hasn't built successfully since Coreutils version
-    8.28.)
-    
-    Signed-off-by: Theodore Ts'o <tytso@mit.edu>
-
-diff --git a/tests/r_corrupt_fs/script b/tests/r_corrupt_fs/script
-index 08af91ed..f6d3a89d 100644
---- ./tests/r_corrupt_fs/script
-+++ b/tests/r_corrupt_fs/script
-@@ -17,7 +17,7 @@ $MKE2FS -q -F -t ext4 -o Linux -b 1024 $TMPFILE 32M >> $OUT.new 2>&1
- echo debugfs -w -R \"set_bg 1 free_blocks_count 65536\" /tmp/foo.img >> $OUT.new
- $DEBUGFS -w -R "set_bg 1 free_blocks_count 65536" $TMPFILE > /dev/null 2>&1
- 
--if type timeout > /dev/null 2>&1 ; then
-+if timeout -v 1s true > /dev/null 2>&1 ; then
-    TIMEOUT="timeout -v 30s"
- else
-    TIMEOUT=
diff --git a/sysutils/e2fsprogs-core/files/patch-tests_m__offset_script b/sysutils/e2fsprogs-core/files/patch-tests_m__offset_script
new file mode 100644
index 000000000000..9cc4232eec3a
--- /dev/null
+++ b/sysutils/e2fsprogs-core/files/patch-tests_m__offset_script
@@ -0,0 +1,11 @@
+--- tests/m_offset/script.orig	2023-02-02 15:57:32 UTC
++++ tests/m_offset/script
+@@ -13,7 +13,7 @@ $MKE2FS -F -b 1024 -E offset=524288 "$TMPFILE" 1024 >>
+ # compute crc of the first and last 512 1k blocks
+ crc_first2=`$DD if="$TMPFILE" bs=1k count=512 2>/dev/null | $CRCSUM`
+ crc_last2=`$DD if="$TMPFILE" bs=1k count=512 skip=1536 2>/dev/null | $CRCSUM`
+-crc_exp=`yes a | $DD bs=1k count=512 2>/dev/null | $CRCSUM`
++crc_exp=`yes a | $DD bs=1k count=512 iflag=fullblock 2>/dev/null | $CRCSUM`
+ # a warning should be only emitted by the first mke2fs call
+ warning=`grep -c "offset specified without an explicit file system size." \
+ 	"$OUT"`
diff --git a/sysutils/e2fsprogs-core/pkg-plist b/sysutils/e2fsprogs-core/pkg-plist
index 2a9455717727..c2ca3765b12b 100644
--- a/sysutils/e2fsprogs-core/pkg-plist
+++ b/sysutils/e2fsprogs-core/pkg-plist
@@ -108,6 +108,7 @@ sbin/tune2fs
 %%NLS%%share/locale/es/LC_MESSAGES/e2fsprogs.mo
 %%NLS%%share/locale/fi/LC_MESSAGES/e2fsprogs.mo
 %%NLS%%share/locale/fr/LC_MESSAGES/e2fsprogs.mo
+%%NLS%%share/locale/fur/LC_MESSAGES/e2fsprogs.mo
 %%NLS%%share/locale/hu/LC_MESSAGES/e2fsprogs.mo
 %%NLS%%share/locale/id/LC_MESSAGES/e2fsprogs.mo
 %%NLS%%share/locale/it/LC_MESSAGES/e2fsprogs.mo
diff --git a/sysutils/e2fsprogs/Makefile b/sysutils/e2fsprogs/Makefile
index a59f478893cd..7b30953515d3 100644
--- a/sysutils/e2fsprogs/Makefile
+++ b/sysutils/e2fsprogs/Makefile
@@ -1,7 +1,7 @@
 PORTNAME=	e2fsprogs
 CATEGORIES=	sysutils
-PORTVERSION=	1.46.5
-PORTREVISION=	5
+PORTVERSION=	1.46.6
+PORTREVISION=	0
 BUILD_DEPENDS=	e2fsprogs-core>=${PORTVERSION}:sysutils/e2fsprogs-core
 RUN_DEPENDS=	${BUILD_DEPENDS}
 DISTFILES=	# empty



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202302022349.312NnR82049287>