Hello community, here is the log from the commit of package open-iscsi for openSUSE:Factory checked in at 2020-04-05 20:49:23 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/open-iscsi (Old) and /work/SRC/openSUSE:Factory/.open-iscsi.new.3248 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "open-iscsi" Sun Apr 5 20:49:23 2020 rev:93 rq:791030 version:unknown Changes: -------- --- /work/SRC/openSUSE:Factory/open-iscsi/open-iscsi.changes 2020-03-31 17:31:42.608241682 +0200 +++ /work/SRC/openSUSE:Factory/.open-iscsi.new.3248/open-iscsi.changes 2020-04-05 20:49:23.720944405 +0200 @@ -1,0 +2,10 @@ +Wed Apr 1 18:38:30 UTC 2020 - Lee Duncan <[email protected]> + +- Update with two upstream commits: + * Fix issue where "iscsi-iname -p" core dumps. (found upstream) + * Fix iscsi.service so it handles restarts better (bsc#1163499) + * Add Wants=remote-fs-pre.target for sequencing. (bsc#1158536) + updating: + * open-iscsi-SUSE-latest.diff.bz2 + +------------------------------------------------------------------- ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ open-iscsi-SUSE-latest.diff.bz2 ++++++ --- /var/tmp/diff_new_pack.ogHEW2/_old 2020-04-05 20:49:24.192944857 +0200 +++ /var/tmp/diff_new_pack.ogHEW2/_new 2020-04-05 20:49:24.192944857 +0200 @@ -0,0 +1,99 @@ +diff --git a/etc/systemd/iscsi.service b/etc/systemd/iscsi.service +index e4758881cb9c..1c286d1fdeda 100644 +--- a/etc/systemd/iscsi.service ++++ b/etc/systemd/iscsi.service +@@ -3,7 +3,7 @@ Description=Login and scanning of iSCSI devices + Documentation=man:iscsiadm(8) man:iscsid(8) + Before=remote-fs.target + After=network.target network-online.target iscsid.service +-Requires=iscsid.service ++Requires=iscsid.socket + ConditionPathExists=/etc/iscsi/initiatorname.iscsi + + [Service] +@@ -11,7 +11,7 @@ Type=oneshot + ExecStart=/sbin/iscsiadm -m node --loginall=automatic + ExecStop=/sbin/iscsiadm -m node --logoutall=automatic + ExecStop=/sbin/iscsiadm -m node --logoutall=manual +-SuccessExitStatus=21 ++SuccessExitStatus=21 15 + RemainAfterExit=true + + [Install] +diff --git a/etc/systemd/iscsid.service b/etc/systemd/iscsid.service +index 4fef168163ca..648ceeaa0f07 100644 +--- a/etc/systemd/iscsid.service ++++ b/etc/systemd/iscsid.service +@@ -4,6 +4,7 @@ Documentation=man:iscsid(8) man:iscsiuio(8) man:iscsiadm(8) + DefaultDependencies=no + After=network.target iscsiuio.service + Before=remote-fs-pre.target ++Wants=remote-fs-pre.target + + [Service] + Type=notify +diff --git a/etc/systemd/iscsiuio.service b/etc/systemd/iscsiuio.service +index e4d9fd0f916a..923e01978cf4 100644 +--- a/etc/systemd/iscsiuio.service ++++ b/etc/systemd/iscsiuio.service +@@ -7,6 +7,7 @@ Requires=iscsid.service + BindTo=iscsid.service + After=network.target + Before=remote-fs-pre.target iscsid.service ++Wants=remote-fs-pre.target + + [Service] + Type=notify +diff --git a/utils/Makefile b/utils/Makefile +index f65f1e79008c..aed3bb0a694f 100644 +--- a/utils/Makefile ++++ b/utils/Makefile +@@ -1,7 +1,7 @@ + # This Makefile will work only with GNU make. + + CFLAGS ?= -O2 -fno-inline -g +-CFLAGS += -Wall -Wstrict-prototypes ++CFLAGS += -Wall -Wextra -Wstrict-prototypes + PROGRAMS = iscsi-iname + + all: $(PROGRAMS) +diff --git a/utils/iscsi-iname.c b/utils/iscsi-iname.c +index da850dca0088..0f587e1e0d36 100644 +--- a/utils/iscsi-iname.c ++++ b/utils/iscsi-iname.c +@@ -40,6 +40,13 @@ + * a seperator and 12 characters (6 random bytes in hex representation) */ + #define PREFIX_MAX_LEN 210 + ++static void usage(void) ++{ ++ fprintf(stderr, "Usage: iscsi-iname [-h | --help | -p <prefix>]\n"); ++ fprintf(stderr, "where <prefix> has max length of %d\n", ++ PREFIX_MAX_LEN); ++} ++ + int + main(int argc, char *argv[]) + { +@@ -68,15 +75,17 @@ main(int argc, char *argv[]) + "on every invocation.\n"); + exit(0); + } else if ( strcmp(prefix, "-p") == 0 ) { ++ if (argc != 3) { ++ usage(); ++ exit(1); ++ } + prefix = argv[2]; + if (strnlen(prefix, PREFIX_MAX_LEN + 1) > PREFIX_MAX_LEN) { +- printf("Error: Prefix cannot exceed %d " +- "characters.\n", PREFIX_MAX_LEN); ++ usage(); + exit(1); + } + } else { +- printf("\nUsage: iscsi-iname [-h | --help | " +- "-p <prefix>]\n"); ++ usage(); + exit(0); + } + } else {
