New: util-linux/sfdisk 2.35.1 fails on sector-size header

https://bugs.busybox.net/show_bug.cgi?id=12841

Bug ID: 12841
Summary: util-linux/sfdisk 2.35.1 fails on sector-size header
Product: buildroot
Version: 2020.02.1
Hardware: All
OS: Linux
Status: NEW
Severity: normal
Priority: P5
Component: Other
Assignee: unassigned at buildroot.uclibc.org
Reporter: sebaroth at gmx.de
CC: buildroot at uclibc.org
Target Milestone: ---

Hi,

using sfdisk to print out and re-deploy the partition layout fails in version 2.35 and 2.35.1:

# sfdisk -v
sfdisk from util-linux 2.35.1
# sfdisk --dump /dev/sda > sda.partitions
# sfdisk /dev/sda < sda.partitions
...

...

Some more details can be found in this bugreport:
https://gitlab.alpinelinux.org/alpine/aports/issues/11200 (I know Alpine is unrelated but it's got all the details in there)

Seems like this is fixed upstream already:
https://github.com/karelzak/util-linux/commit/d8c68b52cc939a16f04ec976648a37f5f5de718c

I am wondering if buildroot wants to add a patch to fix this while waiting for the next util-linux version to be released. Suggested fix:

@@ -837,6 +837,7 @@
                   || strcmp(name, "grain") == 0
                   || strcmp(name, "first-lba") == 0
                   || strcmp(name, "last-lba") == 0
+                  || strcmp(name, "sector-size") == 0
                   || strcmp(name, "table-length") == 0) {
                ;                                       /* whatever is possible

*/

        } else

Thanks in advance!

Sebastian

You are receiving this mail because:
You are on the CC list for the bug.

https://bugs.busybox.net/show_bug.cgi?id=12841

--- Comment #1 from Fabrice Fontaine <[email protected]> --- Hi,

Instead of adding a custom patch, I retrieved the upstream patch:
https://patchwork.ozlabs.org/project/buildroot/patch/20200507224150.242211-2-fontaine.fabrice at gmail.com

Can you confirm that the patch above fixes your issue?

Best Regards,

Fabrice

You are receiving this mail because:
You are on the CC list for the bug.

https://bugs.busybox.net/show_bug.cgi?id=12841

Thomas Petazzoni <[email protected]> changed:

What |Removed |Added
----------------------------------------------------------------------------

         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #2 from Thomas Petazzoni <[email protected]> --- Thanks! Fixed by Fabrice in
https://git.buildroot.org/buildroot/commit/?id=af787a4cf5a19c3411d83ddbd8840b9486b6fcad

You are receiving this mail because:
You are on the CC list for the bug.

https://bugs.busybox.net/show_bug.cgi?id=12841

--- Comment #3 from SR <[email protected]> ---
Hi,

testing with the patch you provided it seems all fine too. Probably a good idea to use the upstream patch.

Thanks for adding this!

Sebastian

You are receiving this mail because:
You are on the CC list for the bug.