summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Search for libsystemd instead libsystemd-loginbaserock/0.6.40-8-gcad21b1+libsystemd_fixJavier Jardón2016-03-161-1/+1
|
* wtmp: fix wtmp file on solaris and netbsdAlan Coopersmith2016-01-272-13/+31
| | | | | | | | This commit moves detection of the location of the wtmp file to configure and fixes that detection to work on Solaris and NetBSD. https://bugs.freedesktop.org/show_bug.cgi?id=90882
* lib: fix misleading debug messageMichael Catanzaro2016-01-181-2/+4
| | | | | | | | This debug message unnecessarily implies that the accounts daemon is not running. There are other reasons why loading a user could fail; e.g. if a username simply does not exist. https://bugs.freedesktop.org/show_bug.cgi?id=93655
* lib: Use G_PARAM_STATIC_STRINGSMichael Catanzaro2016-01-112-26/+26
| | | | | | No need to dup and free a bunch of string literals. https://bugs.freedesktop.org/show_bug.cgi?id=93654
* accountsservice: Add SetPasswordHint functionJuan A. Suarez Romero2015-03-165-1/+125
| | | | | | | | | | | | | So far the only way of setting the password hint is by setting also the password. But there are several cases where setting the password isn't done through AccountsService, but still we want to set the password hint. For instance GDM setups the pasword for new used directly from PAM. Thus, let's add a new function to set the hint without changing the password. https://bugs.freedesktop.org/show_bug.cgi?id=89588
* lib: clean up debug messageRay Strode2015-03-121-2/+2
|
* lib: Copy "password-hint" instead of user's real nameJuan A. Suarez Romero2015-03-061-1/+1
| | | | https://bugs.freedesktop.org/show_bug.cgi?id=89472
* daemon: remove dead codeEric Koegel2015-02-251-4/+0
| | | | | | | | | In commit 2c51bd83b65076cf98bd1baf48d9add31435d118, DBusGProxy *bus_proxy was converted to GDBusProxy but the daemon hasn't needed or used the bus_proxy since the conversion to GDBUS. This patch simply removes it. https://bugs.freedesktop.org/show_bug.cgi?id=89318
* lib: add language to SetLanguage failureRay Strode2015-01-221-1/+1
| | | | | <PaulePanter> halfline: Could you please add the language to the error message in `src/libaccountsservice/act-user.c`?
* configure: bump to 0.6.400.6.40Ray Strode2015-01-221-1/+1
|
* NEWS: update for releaseRay Strode2015-01-221-0/+6
|
* lib: don't require X11 display to fetch userRay Strode2015-01-221-9/+9
| | | | | Not all X11 sessions know their display ahead of time, so don't require that it be known to proceed.
* daemon: fix object path derivation for large UIDsGiovanni Mascellani2014-10-311-2/+2
| | | | | | | | | | | | | | | | | | accountsservice exports an object on the system bus for every user it tracks. The path of the object is derived from the user's uid. The format string used for computing the object path supposes the UID is signed. UIDs are unsigned quantities, though, so very large UIDs get misrepresented as negative values in the resulting object path. The negative sign "-" is invalid in an object path. Under these circumstances, accountsservice will crash. This commit corrects the problem, by using a format string that is appropriate for unsigned values. https://bugs.freedesktop.org/show_bug.cgi?id=85688
* systemd: ensure that accounts service starts after NSS initializesStephen Gallagher2014-10-171-0/+6
| | | | | | | | | | | | | The various NSS calls don't give accurate results in some configurations until midway through boot up. This is because SSSD or winbind (or whatever) needs to initialize. In order to prevent accounts service from using NSS prematurely, we need to add an ordering constraint between the nss-user-lookup.target and accountsservice. This commit accomplishes this by adding the appropriate Wants= and After= directives to the accountsservice systemd unit file.
* configure: bump to 0.6.390.6.39Ray Strode2014-10-161-1/+1
|
* NEWS: update for releaseRay Strode2014-10-161-0/+7
|
* daemon: close up ListCachedUsers race at start upRay Strode2014-10-161-2/+1
| | | | | | | | if accountsservice is activated by the ListCachedUsers call, then the call may get processed before the list of users is loaded. This commit closes the race by synchronously loading the list of users at start up.
* lib: Add asynchronous api for user uncachingOndrej Holy2014-09-302-0/+92
| | | | | | | It is needed for deleting enterprise accounts in gnome-control-center. See https://bugzilla.gnome.org/show_bug.cgi?id=727871 for details. https://bugs.freedesktop.org/show_bug.cgi?id=84091
* lib: make sure replace key when replacing value in object_path hashRay Strode2014-09-191-3/+3
| | | | | | | | If add_user gets called on user object that's already being tracked by object path, we can end up replacing and freeing the user object but leaving the old object path memory in the hash table. This commit makes sure we replace the key when replacing the object.
* autogen.sh: Fix gtkdocize call for srcdir != builddirStef Walter2014-09-181-1/+1
| | | | | | | When srcdir != builddir gtkdocize needs to be run in the right directory, or it will complain about missing configure.ac https://bugs.freedesktop.org/show_bug.cgi?id=84047
* Ignore some more built filesStef Walter2014-09-181-0/+2
| | | | https://bugs.freedesktop.org/show_bug.cgi?id=84047
* Release version 0.6.380.6.38Stef Walter2014-09-182-1/+7
|
* lib: add more debug spewRay Strode2014-09-171-0/+2
| | | | Another dose of g_debug to help find a crasher
* lib: add more debug spewRay Strode2014-09-161-0/+3
| | | | To help find a crasher
* lib: fix some annotations to quiet compile outputRay Strode2014-09-081-0/+8
|
* lib: filter out duplicate "new-user" messages from accountsserviceRay Strode2014-09-081-0/+27
| | | | | | AccountsService occassionally sends out a new-user message twice in a row for a particular user. There's probably a bug there, but for now be more vigilant about filtering out the spurious message.
* lib: take out extraneous "user" word in debug spewRay Strode2014-09-081-2/+1
|
* libaccountsservice: sprinkle some extra debug spew aroundRay Strode2014-07-211-1/+15
| | | | | I'm trying to get a grip on a crasher, so this commit introduces a few more debug messages.
* lib: rename get_all_call to get_all_cancellableRay Strode2014-07-211-9/+9
| | | | It's a little strange to call a cancellable object a call.
* lib: clear the cancellable unconditionally in on_get_all_finishedRay Strode2014-07-211-2/+2
| | | | | | This means we don't have to rely on the cancellable getting cleaned up from the caller when failing (which might not happen depending on the reason it finishes prematurely).
* lib: call g_clear_object in update_info instead of g_object_unrefRay Strode2014-07-211-1/+1
| | | | | This will let us clear the cancellable in the finished handler unconditionally, removing some subtleness in the code.
* lib: user g_clear_object instead of g_object_unrefRay Strode2014-07-211-2/+1
| | | | Just a small clean up.
* lib: log when a user changes from system account to normal and vice versaRay Strode2014-07-211-0/+4
| | | | This just adds some debug spew that might help with debugging an issue.
* data: Fix desktop-centric polkit policyStef Walter2014-05-071-6/+6
| | | | | | | | Change the polkit policy so accountsservice allows use of the DBus API by admin users that are not logged in via a monitor+keyboard. This includes users logged in via ssh or Cockpit. https://bugs.freedesktop.org/show_bug.cgi?id=78279
* configure: bump to 0.6.370.6.37Ray Strode2014-03-201-1/+1
|
* NEWS: update for releaseRay Strode2014-03-201-0/+4
|
* wtmp-helper: rename stub generatorRyan Lortie2014-03-201-2/+2
| | | | | | | | | | After transporting the generator to a separate file and creating a stub, I renamed it. Unfortunately, I forgot to rename the stub. This breaks non-utmpx systems. Fix that. https://bugs.freedesktop.org/show_bug.cgi?id=76410
* configure: bump to 0.6.360.6.36Ray Strode2014-03-191-1/+1
|
* NEWS: update for releaseRay Strode2014-03-191-0/+347
|
* wtmp: unbreak the build on FreeBSDRyan Lortie2014-03-191-0/+2
| | | | | | | We hardcode the value of the file that we need to watch for wtmp changes on FreeBSD since it is not exposed in any header file. https://bugs.freedesktop.org/show_bug.cgi?id=75938
* wtmp: stop using private _PATH_WTMPXRyan Lortie2014-03-191-7/+11
| | | | | | | | | | | | | | Instead of using this private macro, use WTMPX_FILENAME. Also, deal explicitly with the implications of this define being missing: throw an #error. This means that some systems that are currently broken (FreeBSD for example) will continue to be broken, but will throw an #error about it. Fixes coming in the following commits. https://bugs.freedesktop.org/show_bug.cgi?id=75938
* wtmp: split out the platform-specific bitsRyan Lortie2014-03-191-6/+21
| | | | | | | The logic for how to begin the iteration of the wtmp file is platform-specific. Split out. https://bugs.freedesktop.org/show_bug.cgi?id=75938
* daemon: split wtmp code into separate fileRyan Lortie2014-03-194-180/+258
| | | | | | | | | Split the code for opening and iterating the wtmp entries into a separate 'wtmp-helper' file. For now, this is a direct transplant of the existing code to a new file with some minimal renaming and a function to get the name of the file to monitor for changes (which is still setup from daemon.c).
* daemon: emulate fgetpwent() if we don't have itRyan Lortie2014-03-144-0/+118
| | | | | | | | | | | We use fgetpwent directly on /etc/passwd in order to ensure we only get a list of local users (and not ones from the network directory service). Unfortunately, this function is not commonly found on non-GNU systems. Provide our own implementation of fgetpwent if the operating system does not provide it. https://bugs.freedesktop.org/show_bug.cgi?id=41747
* user: fix memory leak in save_extra_data functionRay Strode2014-01-291-0/+1
| | | | | | | | | | | The save_extra_data function serializes a key file assocated with the user to disk. It fails to free the in memory buffer, however. This commit fixes that. see downstream bug https://bugzilla.redhat.com/show_bug.cgi?id=1003033
* daemon: call userdel consistentlyMatthias Clasen2014-01-101-3/+4
| | | | | | | | | | | | | | | commit c5905497733bebf9936a7b028a11ca87caf2d71f changed one of the two calls to userdel in the code to use -f. This change was made to ensure users in the logind "closing" state can be deleted (userdel runs pgrep to make sure a user isn't logged in unless -f is passed). Unfortunately, that commit missed the other userdel invocation, and so now there is inconsistent behavior. This commit changes makes sure both invocations pass -f. https://bugs.freedesktop.org/show_bug.cgi?id=65752
* Change up user classification logic againRay Strode2013-11-192-105/+32
| | | | | | | | | | | | relying on login.defs is fragile, and the user heuristics are fragile. This commit requires an explicit uid minimum get configured, and heuristics now only get applied to the specific problematic range they were added to address. https://bugs.freedesktop.org/show_bug.cgi?id=71801
* Revert "daemon: rip out extension interface"Ray Strode2013-11-075-1/+289
| | | | | | This reverts commit f86c93014e698d81d43fe1ebaf805fa794e5a984. This was meant to be a downstream patch.
* Avoid deleting the root userMatthias Clasen2013-11-041-3/+3
| | | | | | | | | | | | | The check we have in place against deleting the root user can be tricked by exploiting the fact that we are checking a gint64, and then later cast it to a uid_t. This can be seen with the following test, which will delete your root account: qdbus --system org.freedesktop.Accounts /org/freedesktop/Accounts \ org.freedesktop.Accounts.DeleteUser -9223372036854775808 true Found with the dfuzzer tool, https://github.com/matusmarhefka/dfuzzer
* daemon: rip out extension interfaceRay Strode2013-10-225-289/+1
| | | | It requires newer glib than we're shipping