From robot at translationproject.org  Sun Sep 14 06:32:03 2014
From: robot at translationproject.org (Translation Project Robot)
Date: Sun, 14 Sep 2014 08:32:03 +0200
Subject: [parted-devel] New Serbian PO file for 'parted' (version 3.1.90)
Message-ID: <E1XT3Lv-0003Ie-LM@duik.vrijschrift.org>

Hello, gentle maintainer.

This is a message from the Translation Project robot.

A revised PO file for textual domain 'parted' has been submitted
by the Serbian team of translators.  The file is available at:

    http://translationproject.org/latest/parted/sr.po

(This file, 'parted-3.1.90.sr.po', has just now been sent to you in
a separate email.)

All other PO files for your package are available in:

    http://translationproject.org/latest/parted/

Please consider including all of these in your next release, whether
official or a pretest.

Whenever you have a new distribution with a new version number ready,
containing a newer POT file, please send the URL of that distribution
tarball to the address below.  The tarball may be just a pretest or a
snapshot, it does not even have to compile.  It is just used by the
translators when they need some extra translation context.

The following HTML page has been updated:

    http://translationproject.org/domain/parted.html

If any question arises, please contact the translation coordinator.

Thank you for all your work,

                                The Translation Project robot, in the
                                name of your translation coordinator.
                                <coordinator at translationproject.org>



From mike.fleetwood at googlemail.com  Sun Sep 28 15:15:47 2014
From: mike.fleetwood at googlemail.com (Mike Fleetwood)
Date: Sun, 28 Sep 2014 16:15:47 +0100
Subject: [parted-devel] [PATCH 0/3] Prevent crash resizing FAT16 file systems
Message-ID: <1411917350-14275-1-git-send-email-mike.fleetwood@googlemail.com>

Hi,

Here's a fix for libparted-fs-resize crashing when it resizes FAT16 file
systems.  Adds resize FAT16 test to existing resize FS test case to
prevent it from happening again.  Also does a tidyup on the test case.
Original reporter was using GPared:

Bug 735669 - GParted crashes resizing fat16 file system
https://bugzilla.gnome.org/show_bug.cgi?id=735669

New test detects the failure and passes with the fix applied.  Tested
using 1) cd tests && sudo make check; and 2) GParted.

Thanks,
Mike

Mike Fleetwood (3):
  lib-fs-resize: Prevent crash resizing FAT16 file systems
  tests: t3000-resize-fs.sh: Add FAT16 resizing test
  tests: t3000-resize-fs.sh: Add requirement on mkfs.vfat

 NEWS                        |    4 ++++
 libparted/fs/r/fat/resize.c |   12 +++++++++---
 tests/t-lib-helpers.sh      |    8 +++++++-
 tests/t3000-resize-fs.sh    |   21 ++++++++++++++++-----
 4 files changed, 36 insertions(+), 9 deletions(-)



From mike.fleetwood at googlemail.com  Sun Sep 28 15:15:48 2014
From: mike.fleetwood at googlemail.com (Mike Fleetwood)
Date: Sun, 28 Sep 2014 16:15:48 +0100
Subject: [parted-devel] [PATCH 1/3] lib-fs-resize: Prevent crash resizing
	FAT16 file systems
In-Reply-To: <1411917350-14275-1-git-send-email-mike.fleetwood@googlemail.com>
References: <1411917350-14275-1-git-send-email-mike.fleetwood@googlemail.com>
Message-ID: <1411917350-14275-2-git-send-email-mike.fleetwood@googlemail.com>

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 mike.fleetwood at googlemail.com  Sun Sep 28 15:15:49 2014
From: mike.fleetwood at googlemail.com (Mike Fleetwood)
Date: Sun, 28 Sep 2014 16:15:49 +0100
Subject: [parted-devel] [PATCH 2/3] tests: t3000-resize-fs.sh: Add FAT16
	resizing test
In-Reply-To: <1411917350-14275-1-git-send-email-mike.fleetwood@googlemail.com>
References: <1411917350-14275-1-git-send-email-mike.fleetwood@googlemail.com>
Message-ID: <1411917350-14275-3-git-send-email-mike.fleetwood@googlemail.com>

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.
-- 
1.7.1



From mike.fleetwood at googlemail.com  Sun Sep 28 15:15:50 2014
From: mike.fleetwood at googlemail.com (Mike Fleetwood)
Date: Sun, 28 Sep 2014 16:15:50 +0100
Subject: [parted-devel] [PATCH 3/3] tests: t3000-resize-fs.sh: Add
	requirement on mkfs.vfat
In-Reply-To: <1411917350-14275-1-git-send-email-mike.fleetwood@googlemail.com>
References: <1411917350-14275-1-git-send-email-mike.fleetwood@googlemail.com>
Message-ID: <1411917350-14275-4-git-send-email-mike.fleetwood@googlemail.com>

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
-- 
1.7.1