Bug 249445

Summary: sysutils/accountsservice: Update to 0.6.55
Product: Ports & Packages Reporter: Olivier Duchateau <duchateau.olivier>
Component: Individual Port(s)Assignee: freebsd-ports-bugs (Nobody) <ports-bugs>
Status: Open ---    
Severity: Affects Only Me CC: 0mp, bpurgar, desktop, duchateau.olivier, grahamperrin, greg, huanghwh, np, rm, tcberner, timp87, woodsb02
Priority: --- Flags: 0mp: maintainer-feedback-
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Patch to update sysutils/accountsservice none

Description Olivier Duchateau 2020-09-19 09:09:30 UTC
Created attachment 218068 [details]
Patch to update sysutils/accountsservice

- Update to 0.6.55 (big 4 years jump!)
- Switch to the Meson build system
- Update pkg-descr, Freedesktop.org migrated from cgit to gitlab
- Adjust dependencies
- Remove useless pkg-install script, feature already in pkg-plist

CC'ed the desktop team

Note: tested with GLib 2.66.0
Comment 1 Olivier Duchateau 2020-09-19 09:11:31 UTC
Tested with GLib 2.66.0 and GObject introspection 1.66.0 (both the latest releases).
Comment 2 Greg V 2020-11-12 13:52:21 UTC
Ooh, this also fixes the userdel thing (the previous patches forgot to touch deletion), nice.
Comment 3 Tobias C. Berner freebsd_committer 2021-05-24 15:05:14 UTC
Thanks for taking this 0mp :)


mfg Tobias
Comment 4 commit-hook freebsd_committer 2021-05-25 08:28:54 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=ffb1311e56725702208e807cfc63c8163c6b4a52

commit ffb1311e56725702208e807cfc63c8163c6b4a52
Author:     Olivier Duchateau <olivierd@FreeBSD.org>
AuthorDate: 2021-05-24 15:38:38 +0000
Commit:     Mateusz Piotrowski <0mp@FreeBSD.org>
CommitDate: 2021-05-25 08:28:07 +0000

    sysutils/accountsservice: Update to 0.6.55

    This update also fixes the userdel functionality.

    PR:             249445
    Reviewed by:    0mp
    Approved by:    maintainer timeout
    MFH:            2021Q2
    Security:       75aae50b-9e3c-11eb-9bc3-8c164582fbac
    Security:       CVE-2018-14036

 sysutils/accountsservice/Makefile                  |  30 +--
 sysutils/accountsservice/distinfo                  |   6 +-
 .../accountsservice/files/patch-configure (gone)   |  21 --
 .../accountsservice/files/patch-meson.build (new)  |  16 ++
 .../files/patch-meson__post__install.py (new)      |  13 +
 sysutils/accountsservice/files/patch-src_daemon.c  | 282 ++++++++++++++++++++-
 .../patch-src_libaccountsservice_act-user.c (new)  |  11 +
 .../files/patch-src_meson.build (new)              |  13 +
 sysutils/accountsservice/files/patch-src_user.c    | 190 ++++++++++----
 .../accountsservice/files/patch-src_user.h (new)   |  27 ++
 .../files/patch-src_wtmp-helper.h (new)            |  10 +
 sysutils/accountsservice/pkg-descr                 |   6 +-
 sysutils/accountsservice/pkg-install (gone)        |   9 -
 sysutils/accountsservice/pkg-plist                 |  51 ++--
 14 files changed, 540 insertions(+), 145 deletions(-)
Comment 5 commit-hook freebsd_committer 2021-05-25 08:32:56 UTC
A commit in branch 2021Q2 references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=6a456af02d7efda292eb0c52887c1a1964478589

commit 6a456af02d7efda292eb0c52887c1a1964478589
Author:     Olivier Duchateau <olivierd@FreeBSD.org>
AuthorDate: 2021-05-24 15:38:38 +0000
Commit:     Mateusz Piotrowski <0mp@FreeBSD.org>
CommitDate: 2021-05-25 08:31:57 +0000

    sysutils/accountsservice: Update to 0.6.55

    This update also fixes the userdel functionality.

    PR:             249445
    Reviewed by:    0mp
    Approved by:    maintainer timeout
    MFH:            2021Q2
    Security:       75aae50b-9e3c-11eb-9bc3-8c164582fbac
    Security:       CVE-2018-14036

    (cherry picked from commit ffb1311e56725702208e807cfc63c8163c6b4a52)

 sysutils/accountsservice/Makefile                  |  30 +--
 sysutils/accountsservice/distinfo                  |   6 +-
 .../accountsservice/files/patch-configure (gone)   |  21 --
 .../accountsservice/files/patch-meson.build (new)  |  16 ++
 .../files/patch-meson__post__install.py (new)      |  13 +
 sysutils/accountsservice/files/patch-src_daemon.c  | 282 ++++++++++++++++++++-
 .../patch-src_libaccountsservice_act-user.c (new)  |  11 +
 .../files/patch-src_meson.build (new)              |  13 +
 sysutils/accountsservice/files/patch-src_user.c    | 190 ++++++++++----
 .../accountsservice/files/patch-src_user.h (new)   |  27 ++
 .../files/patch-src_wtmp-helper.h (new)            |  10 +
 sysutils/accountsservice/pkg-descr                 |   6 +-
 sysutils/accountsservice/pkg-install (gone)        |   9 -
 sysutils/accountsservice/pkg-plist                 |  51 ++--
 14 files changed, 540 insertions(+), 145 deletions(-)
Comment 6 Ruslan Makhmatkhanov freebsd_committer 2021-06-06 09:03:52 UTC
Reopen. After update to 0.6.55 there is no users list at gdm login screen. It just shows "not listed" string. But I able to click there and type username manually. At 14-CURRENT I'm able to get to desktop after that, but there is similar user report for 13.x at freebsd-ports@ ML, and reporter has a problem with getting the desktop after manually typing username. But if I Lock the screen (Windows+L or via menu), I can't get back to the desktop without restarting gdm and supplying user credentials again, because Lock login screen lacks username field and providing just password isn't enough.  

Reverting to 0.6.42 makes users list available again and fixes Lock screen dialog. Both for 14-CURRENT and 13.x.
Comment 7 Ruslan Makhmatkhanov freebsd_committer 2021-06-06 09:22:59 UTC
Here is original report: 
https://lists.freebsd.org/archives/freebsd-ports/2021-June/000094.html
Comment 8 Olivier Duchateau 2021-06-06 14:17:02 UTC
If you try (when GDM is running, but in different TTY):

DBus service must be enable.

> gdbus call --system --dest org.freedesktop.DBus \
>  --object-path /org/freedesktop/DBus --method org.freedesktop.DBus.ListNames

If you see 'org.freedesktop.Accounts', try this following command:

> gdbus call --system --dest org.freedesktop.Accounts \
>  --object-path /org/freedesktop/Accounts \
>  --method org.freedesktop.Accounts.ListCachedUsers

List of users (it's list of ObjectPath) will be displayed (UserName property contains login name).
Comment 9 Pavel Timofeev 2021-06-07 15:21:52 UTC
(In reply to Ruslan Makhmatkhanov from comment #7)
Thank you, Ruslan, for linking my post
Comment 10 bpurgar 2021-06-20 11:06:10 UTC
Same problem after updating to 0.6.55
No user icons in gdm login screen
Comment 11 bpurgar 2021-06-20 17:08:49 UTC
(In reply to Olivier Duchateau from comment #8)

gdbus call --system --dest org.freedesktop.DBus --object-path /org/freedesktop/DBus --method org.freedesktop.DBus.ListNames
(['org.freedesktop.DBus', ':1.92', ':1.7', ':1.93', ':1.94', ':1.83', 'org.freedesktop.ColorManager', ':1.50', ':1.84', ':1.51', ':1.52', ':1.86', ':1.53', 'org.freedesktop.PolicyKit1', ':1.87', ':1.54', ':1.88', ':1.55', 'org.freedesktop.ConsoleKit', ':1.89', ':1.56', 'org.freedesktop.UPower', 'org.freedesktop.UDisks2', ':1.2', ':1.49', ':1.17', ':1.4', ':1.90', 'org.freedesktop.Accounts', ':1.91'],)

gdbus call --system --dest org.freedesktop.Accounts  --object-path /org/freedesktop/Accounts --method org.freedesktop.Accounts.ListCachedUsers
([objectpath '/org/freedesktop/Accounts/User1001'],)



If i enable autologin in gdm's custom.conf i got this in logs :

gdm[25252]: accountsservice: ActUserManager: user (null) has no username (uid: -1)

maybe this helps ..