[parted] - Fix crash on ppc64 with GPT (#829960) (rwmj)

Brian C. Lane bcl at fedoraproject.org
Fri Jun 8 21:57:04 UTC 2012


commit f380659c9f97c07b9eb0b6c1b6551a117efe04c1
Author: Brian C. Lane <bcl at redhat.com>
Date:   Fri Jun 8 14:56:55 2012 -0700

    - Fix crash on ppc64 with GPT (#829960) (rwmj)

 ...rted-Fix-endian-error-with-FirstUsableLBA.patch |   54 ++++++++++++++++++++
 parted.spec                                        |    6 ++-
 2 files changed, 59 insertions(+), 1 deletions(-)
---
diff --git a/parted-3.1-libparted-Fix-endian-error-with-FirstUsableLBA.patch b/parted-3.1-libparted-Fix-endian-error-with-FirstUsableLBA.patch
new file mode 100644
index 0000000..f9365d2
--- /dev/null
+++ b/parted-3.1-libparted-Fix-endian-error-with-FirstUsableLBA.patch
@@ -0,0 +1,54 @@
+From 1190dc5b214ce6ba4cbeadbdd599ab3ebb2cbb13 Mon Sep 17 00:00:00 2001
+From: "Richard W.M. Jones" <rjones at redhat.com>
+Date: Fri, 8 Jun 2012 14:42:45 -0700
+Subject: [PATCH] libparted: Fix endian error with FirstUsableLBA (#829960)
+
+On big-endial systems (eg. ppc64) this would cause parted to
+crash.
+
+* NEWS: add new bugfix
+* libparted/labels/gpt.c (gpt_get_max_supported_partition_count):
+  fix endian issues with pth->FirstUsableLBA
+---
+ NEWS                   |    3 ++-
+ libparted/labels/gpt.c |    4 ++--
+ 2 files changed, 4 insertions(+), 3 deletions(-)
+
+diff --git a/NEWS b/NEWS
+index b0a0657..596ab37 100644
+--- a/NEWS
++++ b/NEWS
+@@ -1,9 +1,10 @@
+ GNU parted NEWS                                    -*- outline -*-
+ 
+-* Noteworthy changes in release 3.1-2 (2012-03-21) [Fedora]
++* Noteworthy changes in release 3.1-4 (2012-06-08) [Fedora]
+ 
+ ** Bug Fixes
+ 
++  libparted: Don't crash on big-endian systems when using GPT
+   libparted: Treat disks without a PMBR as msdos labeled disks
+   even if they have GPT partition tables.
+ 
+diff --git a/libparted/labels/gpt.c b/libparted/labels/gpt.c
+index ab2145e..6ca33c8 100644
+--- a/libparted/labels/gpt.c
++++ b/libparted/labels/gpt.c
+@@ -1787,12 +1787,12 @@ gpt_get_max_supported_partition_count (const PedDisk *disk, int *max_n)
+ 
+   if (!_header_is_valid (disk, pth, 1))
+     {
+-      pth->FirstUsableLBA = 34;
++      pth->FirstUsableLBA = PED_CPU_TO_LE64 (34);
+       pth->SizeOfPartitionEntry
+         = PED_CPU_TO_LE32 (sizeof (GuidPartitionEntry_t));
+     }
+ 
+-  *max_n = (disk->dev->sector_size * (pth->FirstUsableLBA - 2)
++  *max_n = (disk->dev->sector_size * (PED_LE64_TO_CPU (pth->FirstUsableLBA)-2)
+             / PED_LE32_TO_CPU (pth->SizeOfPartitionEntry));
+   pth_free (pth);
+   return true;
+-- 
+1.7.7.6
+
diff --git a/parted.spec b/parted.spec
index e897659..d4dd52c 100644
--- a/parted.spec
+++ b/parted.spec
@@ -4,7 +4,7 @@
 Summary: The GNU disk partition manipulation program
 Name:    parted
 Version: 3.1
-Release: 3%{?dist}
+Release: 4%{?dist}
 License: GPLv3+
 Group:   Applications/System
 URL:     http://www.gnu.org/software/parted
@@ -16,6 +16,7 @@ Source2: pubkey.jim.meyering
 Patch0: parted-3.0-libparted-copy-pmbr_boot-when-duplicating-GPT-disk.patch
 Patch1: parted-3.1-libparted-check-PMBR-before-GPT-partition-table-8052.patch
 Patch2: parted-3.1-tests-add-t0301-overwrite-gpt-pmbr.sh.patch
+Patch3: parted-3.1-libparted-Fix-endian-error-with-FirstUsableLBA.patch
 
 Buildroot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 BuildRequires: e2fsprogs-devel
@@ -151,6 +152,9 @@ fi
 
 
 %changelog
+* Fri Jun 08 2012 Brian C. Lane <bcl at redhat.com> 3.1-4
+- Fix crash on ppc64 with GPT (#829960) (rwmj)
+
 * Tue May 15 2012 Brian C. Lane <bcl at redhat.com> 3.1-3
 - Added Provides: bundled(gnulib) (#821782)
 


More information about the scm-commits mailing list