all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#35749] [PATCH] services: sddm: fix root login failure issue.
@ 2019-05-15 14:11 Reza Alizadeh Majd
  2019-05-15 14:34 ` [bug#35749] SDDM root login issue Reza Alizadeh Majd
                   ` (3 more replies)
  0 siblings, 4 replies; 9+ messages in thread
From: Reza Alizadeh Majd @ 2019-05-15 14:11 UTC (permalink / raw)
  To: 35749; +Cc: Reza Alizadeh Majd

* gnu/services/sddm.scm
     (sdm-pam-service): set uid from config.
     (sdm-autologin-pam-service): set uid from config.
     (sdm-pam-services): pass sddm-configuration to sddm-pam-service and
sddm-autologin-pam-service
---
 gnu/services/sddm.scm | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

diff --git a/gnu/services/sddm.scm b/gnu/services/sddm.scm
index b433c59e12..b0e6d40260 100644
--- a/gnu/services/sddm.scm
+++ b/gnu/services/sddm.scm
@@ -175,7 +175,7 @@ Relogin="              (if (sddm-configuration-relogin? config)
 (define (sddm-etc-service config)
   (list `("sddm.conf" ,(sddm-configuration-file config))))
 
-(define (sddm-pam-service)
+(define (sddm-pam-service config)
   "Return a PAM service for @command{sddm}."
   (pam-service
    (name "sddm")
@@ -190,7 +190,9 @@ Relogin="              (if (sddm-configuration-relogin? config)
      (pam-entry
       (control "required")
       (module "pam_succeed_if.so")
-      (arguments (list "uid >= 1000" "quiet")))
+      (arguments (list (string-append "uid >= "
+                                      (number->string (sddm-configuration-minimum-uid config)))
+                       "quiet")))
      ;; should be factored out into system-auth
      (pam-entry
       (control "required")
@@ -249,7 +251,7 @@ Relogin="              (if (sddm-configuration-relogin? config)
       (control "required")
       (module "pam_unix.so"))))))
 
-(define (sddm-autologin-pam-service)
+(define (sddm-autologin-pam-service config)
   "Return a PAM service for @command{sddm-autologin}"
   (pam-service
    (name "sddm-autologin")
@@ -261,7 +263,9 @@ Relogin="              (if (sddm-configuration-relogin? config)
      (pam-entry
       (control "required")
       (module "pam_succeed_if.so")
-      (arguments (list "uid >= 1000" "quiet")))
+      (arguments (list (string-append "uid >= "
+                                      (number->string (sddm-configuration-minimum-uid config)))
+                       "quiet")))
      (pam-entry
       (control "required")
       (module "pam_permit.so"))))
@@ -282,9 +286,9 @@ Relogin="              (if (sddm-configuration-relogin? config)
       (module "sddm"))))))
 
 (define (sddm-pam-services config)
-  (list (sddm-pam-service)
+  (list (sddm-pam-service config)
         (sddm-greeter-pam-service)
-        (sddm-autologin-pam-service)))
+        (sddm-autologin-pam-service config)))
 
 (define %sddm-accounts
   (list (user-group (name "sddm") (system? #t))
-- 
2.19.0

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [bug#35749] SDDM root login issue
  2019-05-15 14:11 [bug#35749] [PATCH] services: sddm: fix root login failure issue Reza Alizadeh Majd
@ 2019-05-15 14:34 ` Reza Alizadeh Majd
  2019-05-20 21:16 ` [bug#35749] [PATCH] services: sddm: fix root login failure issue Ludovic Courtès
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 9+ messages in thread
From: Reza Alizadeh Majd @ 2019-05-15 14:34 UTC (permalink / raw)
  To: 35749

There was an issue for root login using SDDM, that was related to hard-coded uid  in sddm-pam-service. 
this patch set's the uid based on `minimum-uid` option in `sddm-configuration`.

Best, 
Reza

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [bug#35749] [PATCH] services: sddm: fix root login failure issue.
  2019-05-15 14:11 [bug#35749] [PATCH] services: sddm: fix root login failure issue Reza Alizadeh Majd
  2019-05-15 14:34 ` [bug#35749] SDDM root login issue Reza Alizadeh Majd
@ 2019-05-20 21:16 ` Ludovic Courtès
  2019-05-21 11:26   ` Reza Alizadeh Majd
  2019-05-21 11:42 ` [bug#35749] [PATCH 1/2] " Reza Alizadeh Majd
  2019-10-15 19:38 ` bug#35749: [PATCH] services: sddm: fix root login failure issue Efraim Flashner
  3 siblings, 1 reply; 9+ messages in thread
From: Ludovic Courtès @ 2019-05-20 21:16 UTC (permalink / raw)
  To: Reza Alizadeh Majd; +Cc: 35749

Hi,

Reza Alizadeh Majd <r.majd@pantherx.org> skribis:

> * gnu/services/sddm.scm
>      (sdm-pam-service): set uid from config.
>      (sdm-autologin-pam-service): set uid from config.
>      (sdm-pam-services): pass sddm-configuration to sddm-pam-service and
> sddm-autologin-pam-service
> ---
>  gnu/services/sddm.scm | 16 ++++++++++------
>  1 file changed, 10 insertions(+), 6 deletions(-)

Could you send an updated patch that documents the ‘minimum-uid’ field
in doc/guix.texi, under ‘sddm-configuration’?

Apart from that it LGTM.

Thanks for fixing this!

Ludo’.

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [bug#35749] [PATCH] services: sddm: fix root login failure issue.
  2019-05-20 21:16 ` [bug#35749] [PATCH] services: sddm: fix root login failure issue Ludovic Courtès
@ 2019-05-21 11:26   ` Reza Alizadeh Majd
  2019-05-21 13:17     ` Ludovic Courtès
  0 siblings, 1 reply; 9+ messages in thread
From: Reza Alizadeh Majd @ 2019-05-21 11:26 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 35749

Hi, 

> Could you send an updated patch that documents the ‘minimum-uid’ field
> in doc/guix.texi, under ‘sddm-configuration’?

Is it possible to submit further changes to this patch, or I need to submit a new one with whole changes? 

Thanks, 
Reza

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [bug#35749] [PATCH 1/2] services: sddm: fix root login failure issue.
  2019-05-15 14:11 [bug#35749] [PATCH] services: sddm: fix root login failure issue Reza Alizadeh Majd
  2019-05-15 14:34 ` [bug#35749] SDDM root login issue Reza Alizadeh Majd
  2019-05-20 21:16 ` [bug#35749] [PATCH] services: sddm: fix root login failure issue Ludovic Courtès
@ 2019-05-21 11:42 ` Reza Alizadeh Majd
  2019-05-21 11:42   ` [bug#35749] [PATCH 2/2] doc: update minimum-uid document for sddm-configuration Reza Alizadeh Majd
  2019-10-15 19:38 ` bug#35749: [PATCH] services: sddm: fix root login failure issue Efraim Flashner
  3 siblings, 1 reply; 9+ messages in thread
From: Reza Alizadeh Majd @ 2019-05-21 11:42 UTC (permalink / raw)
  To: 35749; +Cc: Reza Alizadeh Majd

* gnu/services/sddm.scm
     (sdm-pam-service): set uid from config.
     (sdm-autologin-pam-service): set uid from config.
     (sdm-pam-services): pass sddm-configuration to sddm-pam-service and
sddm-autologin-pam-service
---
 gnu/services/sddm.scm | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

diff --git a/gnu/services/sddm.scm b/gnu/services/sddm.scm
index b433c59e12..b0e6d40260 100644
--- a/gnu/services/sddm.scm
+++ b/gnu/services/sddm.scm
@@ -175,7 +175,7 @@ Relogin="              (if (sddm-configuration-relogin? config)
 (define (sddm-etc-service config)
   (list `("sddm.conf" ,(sddm-configuration-file config))))
 
-(define (sddm-pam-service)
+(define (sddm-pam-service config)
   "Return a PAM service for @command{sddm}."
   (pam-service
    (name "sddm")
@@ -190,7 +190,9 @@ Relogin="              (if (sddm-configuration-relogin? config)
      (pam-entry
       (control "required")
       (module "pam_succeed_if.so")
-      (arguments (list "uid >= 1000" "quiet")))
+      (arguments (list (string-append "uid >= "
+                                      (number->string (sddm-configuration-minimum-uid config)))
+                       "quiet")))
      ;; should be factored out into system-auth
      (pam-entry
       (control "required")
@@ -249,7 +251,7 @@ Relogin="              (if (sddm-configuration-relogin? config)
       (control "required")
       (module "pam_unix.so"))))))
 
-(define (sddm-autologin-pam-service)
+(define (sddm-autologin-pam-service config)
   "Return a PAM service for @command{sddm-autologin}"
   (pam-service
    (name "sddm-autologin")
@@ -261,7 +263,9 @@ Relogin="              (if (sddm-configuration-relogin? config)
      (pam-entry
       (control "required")
       (module "pam_succeed_if.so")
-      (arguments (list "uid >= 1000" "quiet")))
+      (arguments (list (string-append "uid >= "
+                                      (number->string (sddm-configuration-minimum-uid config)))
+                       "quiet")))
      (pam-entry
       (control "required")
       (module "pam_permit.so"))))
@@ -282,9 +286,9 @@ Relogin="              (if (sddm-configuration-relogin? config)
       (module "sddm"))))))
 
 (define (sddm-pam-services config)
-  (list (sddm-pam-service)
+  (list (sddm-pam-service config)
         (sddm-greeter-pam-service)
-        (sddm-autologin-pam-service)))
+        (sddm-autologin-pam-service config)))
 
 (define %sddm-accounts
   (list (user-group (name "sddm") (system? #t))
-- 
2.19.0

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [bug#35749] [PATCH 2/2] doc: update minimum-uid document for sddm-configuration.
  2019-05-21 11:42 ` [bug#35749] [PATCH 1/2] " Reza Alizadeh Majd
@ 2019-05-21 11:42   ` Reza Alizadeh Majd
  0 siblings, 0 replies; 9+ messages in thread
From: Reza Alizadeh Majd @ 2019-05-21 11:42 UTC (permalink / raw)
  To: 35749; +Cc: Reza Alizadeh Majd

---
 doc/guix.texi | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/doc/guix.texi b/doc/guix.texi
index 27e0f72ccb..6bce474859 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -13609,7 +13609,8 @@ Directory to look for faces.
 Default PATH to use.
 
 @item @code{minimum-uid} (default 1000)
-Minimum UID to display in SDDM.
+Minimum UID to display in SDDM. This option also used in @code{sddm-pam-service} and 
+@code{sddm-autologin-pam-service} in order to set minimum UID which allows to login. 
 
 @item @code{maximum-uid} (default 2000)
 Maximum UID to display in SDDM
-- 
2.19.0

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [bug#35749] [PATCH] services: sddm: fix root login failure issue.
  2019-05-21 11:26   ` Reza Alizadeh Majd
@ 2019-05-21 13:17     ` Ludovic Courtès
  2019-05-21 13:21       ` Reza Alizadeh Majd
  0 siblings, 1 reply; 9+ messages in thread
From: Ludovic Courtès @ 2019-05-21 13:17 UTC (permalink / raw)
  To: Reza Alizadeh Majd; +Cc: 35749

Hi,

"Reza Alizadeh Majd" <r.majd@pantherx.org> skribis:

>> Could you send an updated patch that documents the ‘minimum-uid’ field
>> in doc/guix.texi, under ‘sddm-configuration’?
>
> Is it possible to submit further changes to this patch, or I need to submit a new one with whole changes? 

Please send us a new patch that includes all the changes.

Thanks in advance!

Ludo’.

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [bug#35749] [PATCH] services: sddm: fix root login failure issue.
  2019-05-21 13:17     ` Ludovic Courtès
@ 2019-05-21 13:21       ` Reza Alizadeh Majd
  0 siblings, 0 replies; 9+ messages in thread
From: Reza Alizadeh Majd @ 2019-05-21 13:21 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 35749

OK, sure

On Tue, May 21, 2019, at 5:47 PM, Ludovic Courtès wrote:
> Hi,
> 
> "Reza Alizadeh Majd" <r.majd@pantherx.org> skribis:
> 
> >> Could you send an updated patch that documents the ‘minimum-uid’ field
> >> in doc/guix.texi, under ‘sddm-configuration’?
> >
> > Is it possible to submit further changes to this patch, or I need to submit a new one with whole changes? 
> 
> Please send us a new patch that includes all the changes.
> 
> Thanks in advance!
> 
> Ludo’.
>

^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#35749: [PATCH] services: sddm: fix root login failure issue.
  2019-05-15 14:11 [bug#35749] [PATCH] services: sddm: fix root login failure issue Reza Alizadeh Majd
                   ` (2 preceding siblings ...)
  2019-05-21 11:42 ` [bug#35749] [PATCH 1/2] " Reza Alizadeh Majd
@ 2019-10-15 19:38 ` Efraim Flashner
  3 siblings, 0 replies; 9+ messages in thread
From: Efraim Flashner @ 2019-10-15 19:38 UTC (permalink / raw)
  To: Reza Alizadeh Majd; +Cc: ludo, 35749-done

[-- Attachment #1: Type: text/plain, Size: 278 bytes --]

This patch-set was pushed back in May. Closing the bug now :)


-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2019-10-15 19:39 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-05-15 14:11 [bug#35749] [PATCH] services: sddm: fix root login failure issue Reza Alizadeh Majd
2019-05-15 14:34 ` [bug#35749] SDDM root login issue Reza Alizadeh Majd
2019-05-20 21:16 ` [bug#35749] [PATCH] services: sddm: fix root login failure issue Ludovic Courtès
2019-05-21 11:26   ` Reza Alizadeh Majd
2019-05-21 13:17     ` Ludovic Courtès
2019-05-21 13:21       ` Reza Alizadeh Majd
2019-05-21 11:42 ` [bug#35749] [PATCH 1/2] " Reza Alizadeh Majd
2019-05-21 11:42   ` [bug#35749] [PATCH 2/2] doc: update minimum-uid document for sddm-configuration Reza Alizadeh Majd
2019-10-15 19:38 ` bug#35749: [PATCH] services: sddm: fix root login failure issue Efraim Flashner

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.