File fix-spurious-warnings.patch of Package gptfdisk

Overview Repositories Revisions Requests Users Attributes Meta

File fix-spurious-warnings.patch of Package gptfdisk (Revision e21fc2be295a54d264df749a44a04116)

Currently displaying revision e21fc2be295a54d264df749a44a04116 , Show latest

Commit f063fe08e424c99f133df18bf9dce49c851bcb0a:
Fixed bug that caused spurious warnings about the partition table
header claiming an invalid size of partition entries when reading
some MBR disks.
See also https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=981231

--- a/gpt.cc
+++ b/gpt.cc
@@ -1042,11 +1042,19 @@
    *crcOk = CheckHeaderCRC(&tempHeader);
 
    if (tempHeader.sizeOfPartitionEntries != sizeof(GPTPart)) {
-       cerr << "Warning: Partition table header claims that the size of partition table\n";
-       cerr << "entries is " << tempHeader.sizeOfPartitionEntries << " bytes, but this program ";
-       cerr << " supports only " << sizeof(GPTPart) << "-byte entries.\n";
-       cerr << "Adjusting accordingly, but partition table may be garbage.\n";
-       tempHeader.sizeOfPartitionEntries = sizeof(GPTPart);
+      // Print the below warning only if the CRC is OK -- but correct the
+      // problem either way. The warning is printed only on a valid CRC
+      // because otherwise this warning will display inappropriately when
+      // reading MBR disks. If the CRC is invalid, then a warning about
+      // that will be shown later, so the user will still know that
+      // something is wrong.
+      if (*crcOk) {
+         cerr << "Warning: Partition table header claims that the size of partition table\n";
+         cerr << "entries is " << tempHeader.sizeOfPartitionEntries << " bytes, but this program ";
+         cerr << " supports only " << sizeof(GPTPart) << "-byte entries.\n";
+         cerr << "Adjusting accordingly, but partition table may be garbage.\n";
+      }
+      tempHeader.sizeOfPartitionEntries = sizeof(GPTPart);
    }
 
    if (allOK && (numParts != tempHeader.numParts) && *crcOk) {

Places

File fix-spurious-warnings.patch of Package gptfdisk (Revision e21fc2be295a54d264df749a44a04116)

Places