[arch-commits] Commit in parted/repos (4 files)

Christian Hesse eworm at archlinux.org
Thu May 2 09:26:00 UTC 2019


    Date: Thursday, May 2, 2019 @ 09:25:59
  Author: eworm
Revision: 352375

archrelease: copy trunk to testing-x86_64

Added:
  parted/repos/testing-x86_64/
  parted/repos/testing-x86_64/0001-fat16.patch
    (from rev 352374, parted/trunk/0001-fat16.patch)
  parted/repos/testing-x86_64/0002-include.patch
    (from rev 352374, parted/trunk/0002-include.patch)
  parted/repos/testing-x86_64/PKGBUILD
    (from rev 352374, parted/trunk/PKGBUILD)

--------------------+
 0001-fat16.patch   |  186 +++++++++++++++++++++++++++++++++++++++++++++++++++
 0002-include.patch |   22 ++++++
 PKGBUILD           |   45 ++++++++++++
 3 files changed, 253 insertions(+)

Copied: parted/repos/testing-x86_64/0001-fat16.patch (from rev 352374, parted/trunk/0001-fat16.patch)
===================================================================
--- testing-x86_64/0001-fat16.patch	                        (rev 0)
+++ testing-x86_64/0001-fat16.patch	2019-05-02 09:25:59 UTC (rev 352375)
@@ -0,0 +1,186 @@
+From 3a4c152d38ce34481b0f4fda8aea4e71a8280d8f Mon Sep 17 00:00:00 2001
+From: Mike Fleetwood <mike.fleetwood at googlemail.com>
+Date: Sat, 27 Sep 2014 10:23:17 +0100
+Subject: [PATCH 1/3] lib-fs-resize: Prevent crash resizing FAT16 file systems
+
+Resizing FAT16 file system crashes in libparted/fs/r/fat/resize.c
+create_resize_context() because it was dereferencing NULL pointer
+fs_info->info_sector to copy the info_sector.
+
+Only FAT32 file systems have info_sector populated by fat_open() ->
+fat_info_sector_read().  FAT12 and FAT16 file systems don't have an
+info_sector so pointer fs_info->info_sector remains assigned NULL from
+fat_alloc().  When resizing a FAT file system create_resize_context()
+was always dereferencing fs_info->info_sector to memory copy the
+info_sector, hence it crashed for FAT12 and FAT16.
+
+Make create_resize_context() only copy the info_sector for FAT32 file
+systems.
+
+Reported by Christian Hesse in
+https://bugzilla.gnome.org/show_bug.cgi?id=735669
+---
+ NEWS                        |    4 ++++
+ libparted/fs/r/fat/resize.c |   12 +++++++++---
+ 2 files changed, 13 insertions(+), 3 deletions(-)
+
+diff --git a/NEWS b/NEWS
+index 297b0a5..da7db50 100644
+--- a/NEWS
++++ b/NEWS
+@@ -2,6 +2,10 @@ GNU parted NEWS                                    -*- outline -*-
+ 
+ * Noteworthy changes in release ?.? (????-??-??) [?]
+ 
++** Bug Fixes
++
++  libparted-fs-resize: Prevent crash resizing FAT16 file systems.
++
+ 
+ * Noteworthy changes in release 3.2 (2014-07-28) [stable]
+ 
+diff --git a/libparted/fs/r/fat/resize.c b/libparted/fs/r/fat/resize.c
+index 919acf0..bfe60a0 100644
+--- a/libparted/fs/r/fat/resize.c
++++ b/libparted/fs/r/fat/resize.c
+@@ -668,11 +668,17 @@ create_resize_context (PedFileSystem* fs, const PedGeometry* new_geom)
+ 
+ /* preserve boot code, etc. */
+ 	new_fs_info->boot_sector = ped_malloc (new_geom->dev->sector_size);
+-	new_fs_info->info_sector = ped_malloc (new_geom->dev->sector_size);
+ 	memcpy (new_fs_info->boot_sector, fs_info->boot_sector,
+ 		new_geom->dev->sector_size);
+-	memcpy (new_fs_info->info_sector, fs_info->info_sector,
+-		new_geom->dev->sector_size);
++	new_fs_info->info_sector = NULL;
++	if (fs_info->fat_type == FAT_TYPE_FAT32)
++	{
++		PED_ASSERT (fs_info->info_sector != NULL);
++		new_fs_info->info_sector =
++			ped_malloc (new_geom->dev->sector_size);
++		memcpy (new_fs_info->info_sector, fs_info->info_sector,
++			new_geom->dev->sector_size);
++	}
+ 
+ 	new_fs_info->logical_sector_size = fs_info->logical_sector_size;
+ 	new_fs_info->sector_count = new_geom->length;
+-- 
+1.7.1
+
+
+From 2b5a4805533557b1bcdb5f70537569383f1fe7e8 Mon Sep 17 00:00:00 2001
+From: Mike Fleetwood <mike.fleetwood at googlemail.com>
+Date: Sat, 27 Sep 2014 11:31:46 +0100
+Subject: [PATCH 2/3] tests: t3000-resize-fs.sh: Add FAT16 resizing test
+
+Add FAT16 resizing test so that we don't regress again.
+---
+ tests/t3000-resize-fs.sh |   16 +++++++++++++---
+ 1 files changed, 13 insertions(+), 3 deletions(-)
+
+diff --git a/tests/t3000-resize-fs.sh b/tests/t3000-resize-fs.sh
+index 8cab476..9084eb4 100755
+--- a/tests/t3000-resize-fs.sh
++++ b/tests/t3000-resize-fs.sh
+@@ -46,7 +46,7 @@ device_sectors_required=$(echo $default_end | sed 's/s$//')
+ # Ensure that $dev is large enough for this test
+ test $device_sectors_required -le $dev_n_sectors || fail=1
+ 
+-for fs_type in hfs+ fat32; do
++for fs_type in hfs+ fat32 fat16; do
+ 
+   # create an empty $fs_type partition, cylinder aligned, size > 256 MB
+   parted -a min -s $dev mkpart p1 $start $default_end > out 2>&1 || fail=1
+@@ -59,6 +59,7 @@ for fs_type in hfs+ fat32; do
+   wait_for_dev_to_appear_ ${dev}1
+ 
+   case $fs_type in
++    fat16) mkfs_cmd='mkfs.vfat -F 16'; fsck='fsck.vfat -v';;
+     fat32) mkfs_cmd='mkfs.vfat -F 32'; fsck='fsck.vfat -v';;
+     hfs*) mkfs_cmd='mkfs.hfs';         fsck=fsck.hfs;;
+     *) error "internal error: unhandled fs type: $fs_type";;
+@@ -70,8 +71,17 @@ for fs_type in hfs+ fat32; do
+   # NOTE: shrinking is the only type of resizing that works.
+   # resize that file system to be one cylinder (8MiB) smaller
+   fs-resize ${dev}1 0 $new_end > out 2>&1 || fail=1
+-  # expect no output
+-  compare /dev/null out || fail=1
++
++  # check for expected output
++  case $fs_type in
++    fat16) cat << EOF > exp || framework_failure
++Information: Would you like to use FAT32?  If you leave your file system as FAT16, then you will have no problems.  If you convert to FAT32, and MS Windows is installed on this partition, then you must re-install the MS Windows boot loader.  If you want to do this, you should consult the Parted manual (or your distribution's manual).  Also, converting to FAT32 will make the file system unreadable by MS DOS, MS Windows 95a, and MS Windows NT.
++EOF
++      ;;
++    fat32) cat /dev/null > exp || framework_failure;;     # expect no output
++    hfs*)  cat /dev/null > exp || framework_failure;;     # expect no output
++  esac
++  compare exp out || fail=1
+ 
+   # This is known to segfault with fsck.hfs from
+   # Fedora 16's hfsplus-tools-332.14-12.fc15.x86_64.
+From ca37fcb204f97964ff2c92ea0221367e798810bb Mon Sep 17 00:00:00 2001
+From: Mike Fleetwood <mike.fleetwood at googlemail.com>
+Date: Sun, 28 Sep 2014 11:54:45 +0100
+Subject: [PATCH 3/3] tests: t3000-resize-fs.sh: Add requirement on mkfs.vfat
+
+Add test skipping requirement on mkfs.vfat for the FAT32 and FAT16 file
+system resizing tests.  This matches existing test skipping requirement
+on mkfs.hfs for the hfs+ file system.
+
+* tests/t3000-resize-fs.sh: Also correct skip_test_ to skip_.
+* tests/t-lib-helpers.sh: Also update message for requirement of hfs.
+---
+ tests/t-lib-helpers.sh   |    8 +++++++-
+ tests/t3000-resize-fs.sh |    5 +++--
+ 2 files changed, 10 insertions(+), 3 deletions(-)
+
+diff --git a/tests/t-lib-helpers.sh b/tests/t-lib-helpers.sh
+index 4e83a05..c8684bb 100644
+--- a/tests/t-lib-helpers.sh
++++ b/tests/t-lib-helpers.sh
+@@ -20,7 +20,13 @@ require_acl_()
+ require_hfs_()
+ {
+   mkfs.hfs 2>&1 | grep '^usage:' \
+-    || skip_ "This test requires HFS support."
++    || skip_ "mkfs.hfs: command not found"
++}
++
++require_fat_()
++{
++  mkfs.vfat 2>&1 | grep '^Usage:' \
++    || skip_ "mkfs.vfat: command not found"
+ }
+ 
+ # Skip this test if we're not in SELinux "enforcing" mode.
+diff --git a/tests/t3000-resize-fs.sh b/tests/t3000-resize-fs.sh
+index 9084eb4..a79a307 100755
+--- a/tests/t3000-resize-fs.sh
++++ b/tests/t3000-resize-fs.sh
+@@ -18,7 +18,7 @@
+ 
+ . "${srcdir=.}/init.sh"; path_prepend_ ../parted .
+ require_hfs_
+-
++require_fat_
+ require_root_
+ require_scsi_debug_module_
+ require_512_byte_sector_size_
+@@ -31,7 +31,7 @@ default_end=546147s
+ 
+ # create memory-backed device
+ scsi_debug_setup_ dev_size_mb=550 > dev-name ||
+-  skip_test_ 'failed to create scsi_debug device'
++  skip_ 'failed to create scsi_debug device'
+ dev=$(cat dev-name)
+ 
+ fail=0
+@@ -47,6 +47,7 @@ device_sectors_required=$(echo $default_end | sed 's/s$//')
+ test $device_sectors_required -le $dev_n_sectors || fail=1
+ 
+ for fs_type in hfs+ fat32 fat16; do
++  echo "fs_type=$fs_type"
+ 
+   # create an empty $fs_type partition, cylinder aligned, size > 256 MB
+   parted -a min -s $dev mkpart p1 $start $default_end > out 2>&1 || fail=1

Copied: parted/repos/testing-x86_64/0002-include.patch (from rev 352374, parted/trunk/0002-include.patch)
===================================================================
--- testing-x86_64/0002-include.patch	                        (rev 0)
+++ testing-x86_64/0002-include.patch	2019-05-02 09:25:59 UTC (rev 352375)
@@ -0,0 +1,22 @@
+From ba5e0451b51c983e40afd123b6e0d3eddb55e610 Mon Sep 17 00:00:00 2001
+From: "Richard W.M. Jones" <rjones at redhat.com>
+Date: Sat, 24 Mar 2018 17:37:02 +0000
+Subject: linux: Include <sys/sysmacros.h> for major() macro.
+
+Since glibc 2.27 this header is required.
+---
+ libparted/arch/linux.c | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/libparted/arch/linux.c b/libparted/arch/linux.c
+index 31b98ab..7e86b51 100644
+--- a/libparted/arch/linux.c
++++ b/libparted/arch/linux.c
+@@ -41,6 +41,7 @@
+ #include <sys/utsname.h>        /* for uname() */
+ #include <scsi/scsi.h>
+ #include <assert.h>
++#include <sys/sysmacros.h>
+ #ifdef ENABLE_DEVICE_MAPPER
+ #include <libdevmapper.h>
+ #endif

Copied: parted/repos/testing-x86_64/PKGBUILD (from rev 352374, parted/trunk/PKGBUILD)
===================================================================
--- testing-x86_64/PKGBUILD	                        (rev 0)
+++ testing-x86_64/PKGBUILD	2019-05-02 09:25:59 UTC (rev 352375)
@@ -0,0 +1,45 @@
+# Maintainer: Giovanni Scafora <giovanni at archlinux.org>
+# Contributor: Aaron Griffin <aaron at archlinux.org>
+
+pkgname=parted
+pkgver=3.2
+pkgrel=9
+pkgdesc="A program for creating, destroying, resizing, checking and copying partitions"
+arch=('x86_64')
+license=('GPL3')
+url='https://www.gnu.org/software/parted/parted.html'
+depends=('device-mapper' 'libutil-linux')
+makedepends=('pkg-config')
+validpgpkeys=('1B49F933916A37A3F45A1812015F4DD4A70FB705') # Phillip Susi <psusi at ubuntu.com>
+source=("https://ftp.gnu.org/gnu/${pkgname}/${pkgname}-${pkgver}.tar.xz"{,.sig}
+        '0001-fat16.patch'
+        '0002-include.patch')
+sha256sums=('858b589c22297cacdf437f3baff6f04b333087521ab274f7ab677cb8c6bb78e4'
+            'SKIP'
+            '20f54040e58bbbef6ed043c8be98dda5696035349c3efad850972c938f08f84a'
+            '789d6ef8fcff389f9f74bd9e377aff9949ab5b317a80bd38cec39510bd40cef6')
+
+prepare() {
+  cd "${srcdir}/${pkgname}-${pkgver}"
+
+  patch -Np1 -i "${srcdir}/0001-fat16.patch"
+  patch -Np1 -i "${srcdir}/0002-include.patch"
+}
+
+build() {
+  cd "${srcdir}/${pkgname}-${pkgver}"
+
+  # Do *not* add --disable-debug, it prevents building fatresize
+  # https://github.com/ya-mouse/fatresize/issues/9
+  ./configure \
+    --prefix=/usr \
+    --sbindir=/usr/bin \
+    --disable-rpath
+  make
+}
+
+package() {
+  cd "${srcdir}/${pkgname}-${pkgver}"
+
+  make DESTDIR="${pkgdir}" install
+}


More information about the arch-commits mailing list