Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package yast2-users for openSUSE:Factory 
checked in at 2021-07-17 23:36:27
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/yast2-users (Old)
 and      /work/SRC/openSUSE:Factory/.yast2-users.new.2632 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "yast2-users"

Sat Jul 17 23:36:27 2021 rev:243 rq:906453 version:4.4.3

Changes:
--------
--- /work/SRC/openSUSE:Factory/yast2-users/yast2-users.changes  2021-07-04 
22:10:16.405446927 +0200
+++ /work/SRC/openSUSE:Factory/.yast2-users.new.2632/yast2-users.changes        
2021-07-17 23:36:49.877880656 +0200
@@ -1,0 +2,7 @@
+Wed Jul 14 14:18:32 UTC 2021 - Jos?? Iv??n L??pez Gonz??lez <jlo...@suse.com>
+
+- Show warning when reading system settings (e.g., login.defs)
+  fails (bsc#1177183).
+- 4.4.3
+
+-------------------------------------------------------------------

Old:
----
  yast2-users-4.4.2.tar.bz2

New:
----
  yast2-users-4.4.3.tar.bz2

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

Other differences:
------------------
++++++ yast2-users.spec ++++++
--- /var/tmp/diff_new_pack.7DuUBX/_old  2021-07-17 23:36:50.613874983 +0200
+++ /var/tmp/diff_new_pack.7DuUBX/_new  2021-07-17 23:36:50.617874952 +0200
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-users
-Version:        4.4.2
+Version:        4.4.3
 Release:        0
 Summary:        YaST2 - User and Group Configuration
 License:        GPL-2.0-only
@@ -49,7 +49,8 @@
 # CFA::Nsswitch
 Requires:       yast2-pam >= 4.3.0
 
-Requires:       yast2-security
+# Security::SafeRead
+Requires:       yast2-security >= 4.4.1
 
 # y2usernote, y2useritem
 Requires:       yast2-perl-bindings >= 2.18.0

++++++ yast2-users-4.4.2.tar.bz2 -> yast2-users-4.4.3.tar.bz2 ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-users-4.4.2/package/yast2-users.changes 
new/yast2-users-4.4.3/package/yast2-users.changes
--- old/yast2-users-4.4.2/package/yast2-users.changes   2021-06-28 
09:29:00.000000000 +0200
+++ new/yast2-users-4.4.3/package/yast2-users.changes   2021-07-15 
12:59:28.000000000 +0200
@@ -1,4 +1,11 @@
 -------------------------------------------------------------------
+Wed Jul 14 14:18:32 UTC 2021 - Jos?? Iv??n L??pez Gonz??lez <jlo...@suse.com>
+
+- Show warning when reading system settings (e.g., login.defs)
+  fails (bsc#1177183).
+- 4.4.3
+
+-------------------------------------------------------------------
 Fri Jun 25 14:25:25 UTC 2021 - Ancor Gonzalez Sosa <an...@suse.com>
 
 - Rely on useradd, groupadd and other shadow tools when creating
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-users-4.4.2/package/yast2-users.spec 
new/yast2-users-4.4.3/package/yast2-users.spec
--- old/yast2-users-4.4.2/package/yast2-users.spec      2021-06-28 
09:29:00.000000000 +0200
+++ new/yast2-users-4.4.3/package/yast2-users.spec      2021-07-15 
12:59:28.000000000 +0200
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-users
-Version:        4.4.2
+Version:        4.4.3
 Release:        0
 Summary:        YaST2 - User and Group Configuration
 License:        GPL-2.0-only
@@ -49,7 +49,8 @@
 # CFA::Nsswitch
 Requires:       yast2-pam >= 4.3.0
 
-Requires:       yast2-security
+# Security::SafeRead
+Requires:       yast2-security >= 4.4.1
 
 # y2usernote, y2useritem
 Requires:       yast2-perl-bindings >= 2.18.0
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-users-4.4.2/src/modules/Users.pm 
new/yast2-users-4.4.3/src/modules/Users.pm
--- old/yast2-users-4.4.2/src/modules/Users.pm  2021-06-28 09:29:00.000000000 
+0200
+++ new/yast2-users-4.4.3/src/modules/Users.pm  2021-07-15 12:59:28.000000000 
+0200
@@ -172,8 +172,8 @@
 my $pass_max_days              = "99999";
 
 # password encryption method
-my $encryption_method          = "des";
-my $group_encryption_method    = "des";
+my $encryption_method          = "sha512";
+my $group_encryption_method    = "sha512";
 
 # User/group names must match the following regex expression. (/etc/login.defs)
 my $character_class            = 
"[[:alpha:]_][[:alnum:]_.-]*[[:alnum:]_.\$-]\\?";
@@ -1321,11 +1321,13 @@
     my $self           = shift;
     my $force          = shift;
 
-    return if ($system_defaults_read && !$force);
+    if ($system_defaults_read && !$force) {
+       return Security->read_error || "";
+    }
 
     if (! Security->GetModified ()) {
        my $progress_orig = Progress->set (0);
-       Security->Read ();
+       Security->SafeRead ();
        Progress->set ($progress_orig);
     }
     $security_modified                 = $security_modified || 
Security->GetModified ();
@@ -1336,9 +1338,9 @@
     $pass_max_days     = $security{"PASS_MAX_DAYS"}    || $pass_max_days;
 
     # command to call before/after adding/deleting user
-    $useradd_cmd       = $security{"USERADD_CMD"};
-    $userdel_precmd    = $security{"USERDEL_PRECMD"};
-    $userdel_postcmd   = $security{"USERDEL_POSTCMD"};
+    $useradd_cmd       = $security{"USERADD_CMD"}      || $useradd_cmd;
+    $userdel_precmd    = $security{"USERDEL_PRECMD"}   || $userdel_precmd;
+    $userdel_postcmd   = $security{"USERDEL_POSTCMD"}  || $userdel_postcmd;
 
     # command to call after adding group
     $groupadd_cmd       = ShadowConfig->fetch("GROUPADD_CMD") || "";
@@ -1370,6 +1372,8 @@
 
     UsersCache->InitConstants (\%security);
     $system_defaults_read      = 1;
+
+    return Security->read_error || "";
 }
 
 ##------------------------------------
@@ -1627,7 +1631,13 @@
     # default system settings
     if ($use_gui) { Progress->NextStage (); }
 
-    $self->ReadSystemDefaults (1);
+    $error_msg = $self->ReadSystemDefaults (1);
+
+    if ($use_gui && $error_msg ne "") {
+       my $message = __("It was not possible to read some system settings. 
Some default values will be used. \n\nSee Details for more information.");
+
+       Popup->WarningDetails ($message, $error_msg);
+    }
 
     $self->ReadAllShells();
 

Reply via email to