Commit Graph

16 Commits

Author SHA1 Message Date
Carsten Haitzler 92700403f7 e auth - allow empty password users to auth successfully 2021-02-27 18:18:34 +00:00
Carsten Haitzler 373b4c5ee1 e ckpasse - also back out another change i made to fix up pk 2020-05-27 09:24:24 +01:00
Carsten Haitzler fc89c4f78c e ckpasswd - clean some vars to be static and a fprintf status 2020-05-27 09:19:47 +01:00
Carsten Haitzler d8f1e63a75 back out undo of pk fix - back to bu5hm4ns version 2020-05-27 09:16:04 +01:00
Carsten Haitzler d2c0c1bf19 polkit - fix user id we're authing to and be explicit which one it is
@fix
2020-05-25 13:45:44 +01:00
Carsten Haitzler b16b2fac74 e auth - fix missing stderr output logs i missed last commit 2020-05-25 13:45:40 +01:00
Marcel Hollerbach 4475eb42a6 e_ckpasswd_main: fix polkit to auth root
there were the wrong pids passed.
This fixes calls like "systemctl start docker".
2020-05-25 14:39:25 +02:00
Carsten Haitzler e84dce7806 e auth - nicely prefix logs and clean up error reports and report more 2020-05-25 12:27:49 +01:00
Elyes HAOUAS f84d23ffd2 Remove duplicated includes
Remove duplicated includes

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11857
2020-05-19 13:06:21 -04:00
Carsten Haitzler bab2b503f6 setuid setup - move to single shared file to stop cnp of code 2020-05-01 18:13:23 +01:00
Carsten Haitzler 2ccfec3dad e ckpasswd - handle read errors on stdin
error path - not going to be a problem but in theory... it's there.

fix CID 1425211
2020-04-19 01:58:44 +01:00
Carsten Haitzler a0fc5a7c30 ckpasswd: remove warning of unused var 2019-10-08 08:48:30 +01:00
Carsten Haitzler 5f4697fd7d e - add polkit module to add an auth agent into e
new feature - polkit auth agent support partly in core (need to have
the pam setuid root auth tool respond via dbus) and partly a module
(the agent dbus protocol handling and setup as well as auth gui). this
took me a while even with all the docs to work out how polkit works...
it was really fussy and its data structs are an extra pain in the butt
to craft with eldbus, but i managed it. not everything is supported
but the core basics are there and this can be built on.

right now the gui is really basic, but does the job.
2019-10-08 01:38:54 +01:00
Al Poole f72aa5671d ckpasswd: fix build on OpenBSD (typo) 2018-03-09 16:37:04 +00:00
Carsten Haitzler 51cb454993 e auth - move all auth to child process only (e_ckpasswd).
this should fix T6211 ensuring no drivers can cause a segfault at exit
time. this also happens to remove the enlightenment_sys -z option for
openbsd and unifies all the passwd checking into the single
enlightenment_ckpasswd binary util (that has ifdefs for openbsd,
freebsd and linux pam in it).

this simplifies code removing a mess of auth being done in multiple
places, removes special fork vs run 1 exe or a different exe in
different cases making it more maintainable. yes - this requires
enlightenment_ckpasswd to be setuid root, but it already was when it
was built.

@fix
2018-02-26 19:01:46 +09:00
Conrad Meyer 90fe5a4d0d e_auth: Add suid helper for lokker own-pw checking on FreeBSD
Summary:
PAM on FreeBSD, unlike on Linux, does not allow users to check their own
password. Instead, we need a suid helper to do it for us. Add such a
helper on FreeBSD.

For now, it is limited to checking users in the local password database
(traditional Unix passwd file). This could and should be extended to use
PAM in a later patch.

Test Plan:
Tested empty pw, wrong pw, correct pw at lock screen; observed correct behavior
in each instance.

Reviewers: q66, zmike

Reviewed By: q66, zmike

Subscribers: cedric, seoz

Differential Revision: https://phab.enlightenment.org/D2355
2015-04-21 13:11:56 +01:00