Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package s390-tools for openSUSE:Factory 
checked in at 2021-03-10 08:49:32
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/s390-tools (Old)
 and      /work/SRC/openSUSE:Factory/.s390-tools.new.2378 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "s390-tools"

Wed Mar 10 08:49:32 2021 rev:37 rq:877835 version:2.15.1

Changes:
--------
--- /work/SRC/openSUSE:Factory/s390-tools/s390-tools.changes    2021-02-22 
14:58:09.765773977 +0100
+++ /work/SRC/openSUSE:Factory/.s390-tools.new.2378/s390-tools.changes  
2021-03-10 08:49:44.510423664 +0100
@@ -1,0 +2,31 @@
+Mon Mar  8 20:52:57 UTC 2021 - Mark Post <mp...@suse.com>
+
+- Added s390-tools-sles15sp3-zipl-fix-4k-scsi-ipl.patch (bsc#1183039)
+  IPL is not working when bootloader is installed on a SCSI disk
+  with 4k physical blocksize without using a devicemapper target.
+- Added s390-tools-sles15sp3-remove-no-pie-link-arguments.patch so
+  that we can build on systems with binutils 2.36 and higher. Without
+  this, the build fails because that version of the ld command
+  doesn't recognize "-no-pie" an acceptable argument.
+- Updated the mkdump.pl script to make the name of the temporary
+  configuration file in /tmp/ unpredictable. (bsc#1182876)
+
+-------------------------------------------------------------------
+Sat Feb 27 23:07:46 UTC 2021 - Mark Post <mp...@suse.com>
+
+- Updated detach_disks.sh to not use predictable temporary file
+  names. (bsc#1182777, CVE-2021-25316)
+- Made for the following changes for bsc#1182816:
+  * Updated s390-tools-sles15sp3-Format-devices-in-parallel.patch to
+    change all instances of the progress bar from printing "#" to
+    printing the process number of the DASD volume being formatted.
+  * Updated s390-tools-sles15sp3-Implement-Y-yast_mode.patch to print
+    out the numbder of cylinders in a device, as was done in the
+    prior versions of this patch, but was missed when modifying it
+    for SLES15 SP3.
+  * Added s390-tools-sles15sp3-check-return-code-from-util_file_read_l.patch
+    because otherwise a warning message from dasdfmt is being generated
+    that could interfere with YaST properly processing the progress
+    of the formatting process.
+
+-------------------------------------------------------------------

New:
----
  s390-tools-sles15sp3-check-return-code-from-util_file_read_l.patch
  s390-tools-sles15sp3-remove-no-pie-link-arguments.patch
  s390-tools-sles15sp3-zipl-fix-4k-scsi-ipl.patch

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ s390-tools.spec ++++++
--- /var/tmp/diff_new_pack.vbi383/_old  2021-03-10 08:49:45.490424676 +0100
+++ /var/tmp/diff_new_pack.vbi383/_new  2021-03-10 08:49:45.494424679 +0100
@@ -98,6 +98,7 @@
 Patch9:         
s390-tools-sles15sp3-03-genprotimg-require-argument-for-ramdisk-and-parmfile.patch
 Patch10:        
s390-tools-sles15sp3-04-genprotimg-add-host-key-document-verification-suppor.patch
 Patch11:        s390-tools-sles15sp3-zkey-Fix-APQN-property-names.patch
+Patch12:        s390-tools-sles15sp3-zipl-fix-4k-scsi-ipl.patch
 
 # SUSE patches
 Patch900:       s390-tools-sles12-zipl_boot_msg.patch
@@ -112,6 +113,8 @@
 Patch909:       59-dasd.rules-wait_for.patch
 Patch910:       
s390-tools-sles12-fdasd-skip-partition-check-and-BLKRRPART-ioctl.patch
 Patch911:       
s390-tools-sles15sp1-11-zdev-Do-not-call-zipl-on-initrd-update.patch
+Patch912:       
s390-tools-sles15sp3-check-return-code-from-util_file_read_l.patch
+Patch913:       s390-tools-sles15sp3-remove-no-pie-link-arguments.patch
 
 BuildRequires:  curl-devel
 BuildRequires:  dracut

++++++ detach_disks.sh ++++++
--- /var/tmp/diff_new_pack.vbi383/_old  2021-03-10 08:49:45.678424870 +0100
+++ /var/tmp/diff_new_pack.vbi383/_new  2021-03-10 08:49:45.682424873 +0100
@@ -1,11 +1,10 @@
 #!/bin/sh
 
-COOKIE=$(mcookie)
-DASDFILE=/tmp/dasd.list.${COOKIE}
-DETFILE=/tmp/detach.disks.${COOKIE}
-KEEPFILE=/tmp/keep.disks.${COOKIE}
-NICFILE=/tmp/nic.list.${COOKIE}
-FAILFILE=/tmp/error.${COOKIE}
+DASDFILE=/tmp/dasd.list.$(mcookie)
+DETFILE=/tmp/detach.disks.$(mcookie)
+KEEPFILE=/tmp/keep.disks.$(mcookie)
+NICFILE=/tmp/nic.list.$(mcookie)
+FAILFILE=/tmp/error.$(mcookie)
 
 function expand_RANGE(){
 local RANGE=${1}

++++++ mkdump.pl ++++++
--- /var/tmp/diff_new_pack.vbi383/_old  2021-03-10 08:49:45.754424948 +0100
+++ /var/tmp/diff_new_pack.vbi383/_new  2021-03-10 08:49:45.754424948 +0100
@@ -43,7 +43,7 @@
 my $ZGETDUMP = "/sbin/zgetdump";
 
 # temporary DASD device configuration file for Zipl
-my $MDPATH = "/tmp/mvdump.conf";
+my $MDPATH = "/tmp/mvdump.conf.".`mcookie`;
 # zFCP dump dir, without a leading '/'
 my $ZFCP_DUMP_DIR = "mydumps";
 
@@ -386,11 +386,9 @@
        prepare_dasd(@devices);
 
        # create zipl device configuration file
-       my $md_path = "/tmp/mvdump.conf";
-       
        # don't create files in debug mode
        unless ($OPT_DEBUG) {
-               open(my $file, ">", $md_path) or exit_with("Unable to access 
$md_path: $!.", 15);
+               open(my $file, ">", $MDPATH) or exit_with("Unable to access 
$MDPATH: $!.", 15);
                for my $device (@devices) {
                        print{$file}("${device}1\n");
                }
@@ -398,7 +396,7 @@
        }
 
        print("Creating dump record.\n");
-       run_cmd("${ZIPL} -V -n -M $md_path");
+       run_cmd("${ZIPL} -V -n -M $MDPATH");
 
        cleanup();
 }

++++++ s390-tools-sles15sp3-Format-devices-in-parallel.patch ++++++
--- /var/tmp/diff_new_pack.vbi383/_old  2021-03-10 08:49:45.878425076 +0100
+++ /var/tmp/diff_new_pack.vbi383/_new  2021-03-10 08:49:45.878425076 +0100
@@ -101,6 +101,15 @@
                        for (i = barlength + 1; i <= 33; i++)
                                printf("-");
                        printf("|%3d%%", p_new);
+@@ -311,7 +318,7 @@
+       }
+ 
+       if (g.print_hashmarks && (cyl / g.hashstep - hashcount) != 0) {
+-              printf("#");
++              printf("%d|", g.procnum);
+               fflush(stdout);
+               hashcount++;
+       }
 @@ -1560,7 +1568,11 @@
        char *reqsize_param_str = NULL;
        char *hashstep_str      = NULL;

++++++ s390-tools-sles15sp3-Implement-Y-yast_mode.patch ++++++
--- /var/tmp/diff_new_pack.vbi383/_old  2021-03-10 08:49:45.886425085 +0100
+++ /var/tmp/diff_new_pack.vbi383/_new  2021-03-10 08:49:45.886425085 +0100
@@ -104,6 +104,15 @@
                        printf("ok\n");
                }
        }
+@@ -1548,6 +1556,8 @@
+               error("%s", str);
+ 
+       set_geo(&cylinders, &heads);
++      if (g.yast_mode)
++              printf("%d\n", cylinders);
+       set_label(&vlabel, &format_params, cylinders);
+ 
+       if (g.check)
 @@ -1693,6 +1701,10 @@ int main(int argc, char *argv[])
                case OPT_NODISCARD:
                        g.no_discard = 1;

++++++ s390-tools-sles15sp3-check-return-code-from-util_file_read_l.patch ++++++
--- a/libdasd/dasd_sys.c        2020-10-28 10:31:59.000000000 -0400
+++ b/libdasd/dasd_sys.c        2021-02-27 17:41:04.937023501 -0500
@@ -218,7 +218,8 @@
                return 0;
 
        path = util_path_sysfs("bus/ccw/devices/%s/host_access_count", busid);
-       util_file_read_l(&value, 10, path);
+       if (util_file_read_l(&value, 10, path) == -1)
+               value=0;
        free(path);
 
        return value;
++++++ s390-tools-sles15sp3-remove-no-pie-link-arguments.patch ++++++
--- s390-tools-2.15.1/common.mak        2020-10-28 10:31:59.000000000 -0400
+++ s390-tools-2.15.1/common.mak        2021-03-01 11:16:20.285597140 -0500
@@ -224,8 +224,8 @@
 
 ifneq ($(shell $(CC_SILENT) -dumpspecs 2>/dev/null | grep -e '[^f]no-pie'),)
        NO_PIE_CFLAGS := -fno-pie
-       NO_PIE_LINKFLAGS := -no-pie
-       NO_PIE_LDFLAGS := -no-pie
+       NO_PIE_LINKFLAGS :=
+       NO_PIE_LDFLAGS :=
 else
        NO_PIE_CFLAGS :=
        NO_PIE_LINKFLAGS :=
++++++ s390-tools-sles15sp3-zipl-fix-4k-scsi-ipl.patch ++++++
Subject: [PATCH] [BZ 191696] zipl: fix reading 4k disk's geometry
From: Stefan Haberland <s...@linux.ibm.com>

Description:   zipl: fix IPL on 4K SCSI disks
Symptom:       IPL is not working when bootloader is installed
               on a SCSI disk with 4k physical blocksize without using
               a devicemapper target.
Problem:       The getgeo ioctl that is used reports the partition
               start in multiple of 512 byte sectors but the blocksize
               that is used is the physical blocksize of 4k and
               therefore the calculated partition start does not
               match. For device mapper targets the zipl-helper script
               is used that adjusts the partition start correctly to
               the physical blocksize.
Solution:      Adjust the partition start depending on the physical
               blocksize. 
Reproduction:  Use zipl to install a bootloader on a SCSI disk with 4k
               physical blocksize and without usage of a device mapper
               target. 
Upstream-ID:   4a3957fab5696cc410c5b495956859a424e3552a
Problem-ID:    191696

Upstream-Description:

              zipl: fix reading 4k disk's geometry

              On 4k SCSI disks zipl stores wrong values to 
'scsi_mbr.program_table_pointer',
              which makes system unbootable.
              This happens in 'zipl/src/disk.c:656':

              ```
              /* Convert file system block to physical */
              *physical = mapped * phy_per_fs + subblock;
              /* Add partition start */
              *physical += info->geo.start;

              ```

              So 'hd_geometry.start' should be adjusted before being used.

              Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1918723
              Closes: https://github.com/ibm-s390-linux/s390-tools/pull/107
              Signed-off-by: Nikita Dubrovskii <nik...@linux.ibm.com>
              Reviewed-by: Stefan Haberland <s...@linux.ibm.com>
              Signed-off-by: Jan Hoeppner <hoepp...@linux.ibm.com>


Signed-off-by: Stefan Haberland <s...@linux.ibm.com>
---
 zipl/src/disk.c |    3 +++
 1 file changed, 3 insertions(+)

--- a/zipl/src/disk.c
+++ b/zipl/src/disk.c
@@ -444,6 +444,9 @@ type_determined:
        }
        /* Convert device size to size in physical blocks */
        data->phy_blocks = devsize / (data->phy_block_size / 512);
+       /* Adjust start on SCSI according to block_size. device-mapper devices 
are skipped */
+       if (data->type == disk_type_scsi && target->targetbase == NULL)
+               data->geo.start = data->geo.start / (data->phy_block_size / 
512);
        if (data->partnum != 0)
                data->partition = stats.st_rdev;
        /* Try to get device name */

Reply via email to