convert manual page sources to Docbook, add polkit-config-file-validate tool

Also drop the polkit-check-caller and polkit-check-session tools;
they're not really useful in their current incarnation.

delay loading the configuration until it's needed

This is especially good for saving CPU cycles as we may get a number
events from inotify and previously we kept reloading/parsing the
configuration file on every event.

Merge branch 'master' of ssh://david@git.freedesktop.org/git/PolicyKit

give a little love to polkit-list-actions(1) and polkit-grant(1)

In addition polkit-grant(1) gained a few new features

 --list          : for listing all grants
 --delete <user> : for deleting all grants given to an user

fixed compiler warning about uid_t handling

This fixes the same problem with uid_t as we had with HAL some time ago
on 64bit architectures in PolicyKit. This time I removed the useless check:

   uid == ((unsigned long) -1)

because this is always false on 64bit (comparison is always false due to
limited range of data type) and because the DBusError from the
dbus_bus_get_unix_user() call is set if the function returns DBUS_UID_UNSET
so we need only to check if the error is set.

fixed code documentation issues

Fixed code documentation issues:
 * s/<programlisting>/@code/
 * removed @void: from polkit_grant_new()

add support for annotations

work when SELinux is disabled

I've compiled with selinux and have it turned off. The attached patch
stops the warnings for me.

Richard.

gracefully handle bad config/policy files, drop polkit-reload-config, syslog

 - don't abort/malfunction if the /etc/PolicyKit/PolicyKit.conf
   configuration file is malformed; simply just continue as normal
   but return 'no' to every question asked. Also use syslog(3) to
   report this to the system log

 - if a .policy file is malformed, simply skip it and still include
   other well-formed .policy files. Use syslog(3) to report if indeed
   a .policy file is malformed.

 - drop /var/lib/PolicyKit/reload and rely on inotify to detect changes to
   - /etc/PolicyKit/PolicyKit.conf
   - Policy files in /usr/share/PolicyKit/policy
   - privileges in /var/lib/PolicyKit and /var/run/PolicyKit

As a result, changes made to /etc/PolicyKit/PolicyKit.conf (typically
an admin edits this file) and .policy files (typically these can
change on package upgrades) in /usr/share/PolicyKit/policy are
instantly picked up.

avoid the now defunct group concept in polkit-list-actions

minor doc cleanups

rip out group concept from .policy files and rename <policy> element to <action>

make libpolkit-grant less noisy

make polkit-grant(1) work with <define_admin_user /> feature

handle the case where we don't load descriptions

properly support i18n'ed messages from .policy files

Ugh, I'm not sure if there's a smarter way of dealing with xml:lang
when using expat (google searches for this suggests no) but the way I
fixed this is surely a bitch.

export PolKitConfig and provide a <define_admin_auth/> config file directive

Also change the libpolkit-grant API a bit to work with these changes.

when granting privileges, touch the /var/lib/PolicyKit/reload file

This means that all libpolkit-using processes should recieve a
config_changed() callback whenever privileges are granted.

Remember to update your RPM spec files etc. such that group polkituser
(or equiv.) is permitted to write to the reload file.

make the /var/lib/PolicyKit/reload writable for group polkituser

use correct type for conversation_done() function

fix proposed by Danny Kukawka <danny.kukawka@web.de>

post-release version number bump

provide a high-level interaction diagram of how this works

use waitpid() to avoid Zombie processes

put back isatty() checks when calling user is not uid 0

remove the isatty() call so it's easier to audit the helper

The isatty() check is just to catch users poking around; it provides little or no real security. With this change, you can do stuff like

$ /usr/libexec/polkit-grant-helper-pam
davidz
PAM_PROMPT_ECHO_OFF Password:
<enter real password here>
SUCCESS

$ /usr/libexec/polkit-grant-helper-pam
davidz
PAM_PROMPT_ECHO_OFF Password:
not_my_password
polkit-grant-helper-pam: pam_authenticated failed: Authentication failure
FAILURE

which is useful for auditing.

move PAM stack usage to separate helper

So it turns out that I hadn't been using shadow passwords on my other
development box (don't ask) and that's why auth as root worked fine
when just running as an unprivileged user. However, to auth as another
user (such as root), the process embedding pam needs to run as
root. Therefore, split out the actual authentication bits into a small
and easy to audit helper, polkit-grant-helper-pam.

The auth now goes like this:

 polkit-gnome <-links with-> libpolkit-grant
                                   ^
                                   |
                                spawns
                                   |
                                   V
                     /usr/libexec/polkit-grant-helper
                                   ^
                                   |
                                spawns
                                   |
                                   V
                   /usr/libexec/polkit-grant-helper-pam

where

 polkit-grant-helper
    is setgid polkit; it links with libdbus and libpolkit.

 polkit-grant-helper-pam
    is setuid root; it links only with libpam

added missing files to git repo

Added missing files from PolicyKit 0.4 release package to the
git repo: data/PolicyKit.conf.in, data/config.dtd

actually reload config and policy files when they change

update NEWS

for PolKitContext, mention that a mechanism need to provide .policy files

fix up proper naming of some methods on the PolKitPolicyFileEntry class

fix docs

require that policy files also provide a <message> element

Declaring an action now requires two textual elements (that both are
subject to translation):

 description: This is intended to be used in policy editors, for
              example "Mount internal volumes".
 message:     This is to be used in auth dialogs, for example "System
              Policy prevents mounting this internal volume".

This is actually needed for security reasons. The idea is that the
desktop environment can provide infrastructure that Callers
(e.g. applications) can use to ask the user to authenticate to gain a
privilege. One such example is PolicyKit-gnome; it's a D-Bus session
based service that applications can use to ask the user to
auth.

Before this change the caller provided the markup, e.g. gnome-mount
would do

 action = "hal-storage-mount-fixed";
 markup = _("System policy prevents mounting internal drives");
 result = org.gnome.PolicyKit.ShowDialog (action, markup);

and the problem here is that any application in the session can spoof
the dialog by providing false information and getting to use to click
through on that.

With this change, where the org.gnome.PolicyKit auth service reads the
message from a system-controlled file, this can't happen. What the
user sees really reflects the action he's asking to consider allowing
to happen.

Especially with things like XACE (previously known as SEX) this is
important as we can make the process providing the D-Bus service
org.gnome.PolicyKit run in a dedicated security context, audit it to
make sure it's secure. Then have the window manager paint trust window
decorations or other things to make the user feel fuzzy, warm and
safe.

Btw, with this change the PolicyKit-gnome API will be simplified to

 action = "hal-storage-mount-fixed";
 result = org.gnome.PolicyKit.ShowDialog (action);

which is just about as simple as it can get.

Credit goes to Ryan Lortie <desrt@desrt.ca> for pointing this out
on #gnome-hackers earlier this morning.

fix typo

change default username

- change user from 'polkit' to 'polkituser'
- create directories in /var from polkit instead of polkit-grant

fix up some of the docs

remove RESULT_NOT_AUTHORIZED_TO_KNOW and s/RESULT_UNKNOWN_ACTION/RESULT_UNKNOWN/

switch from VIA_ROOT to VIA_ADMIN

In the future PolicyKit will be able to be built such that VIA_ADMIN
either means 1) ask for root password; or 2) ask for a user in the
'wheel' group to authenticate. Right now it's only the former.

update TODO

document the /etc/PolicyKit/PolicyKit.conf file with a manual page

Also add a man page for polkit-list-actions(1).

add support for an /etc/PolicyKit/PolicyKit.conf config file

With this, system administrators can override policy. Partial support,
more to come (including manual pages and documentation) later.

get proper pid and SELinux context

replace configuration reload mechanism

Instead of asking the user of libpolkit to provide a huge file
monitoring abstraction we simply ask for a simple interface for
watching file descriptors and use inotify (on Linux) to watch a file,
/var/lib/PolicyKit/reload. We provide a new tool,
polkit-reload-config, that simply touches this file.

remove the notion of modules

post-release version bump

update NEWS

add some more TODO items

update HACKING and TODO

update TODO

add the spec

fix small typo so _keep_always granting works again

make .policy files live in /usr/share, not /etc

build fixes

move all grant writing/checking into a separate private library

remove the distinction for local vs. remote users

Again, we punt this to the applications/mechanisms - they know better.

rip out the notion of Resources

It makes things a _lot more_ complicated having to deal with resources
and there's a much nicer way to deal with it: Punt it to the apps:

It's much more natural for the application to have a notion about
about what resources are "trusted" (and e.g. requires lesser
privileges) and what resources aren't.

Consider dial-up networking; here the privileged application that
performs the dial-up operation consults a list (maintained by the
system administrator) of allowed numbers to dial. If the unprivileged
networking UI applet that requests a number to be dialed is on the
list it uses the PolicyKit action 'nm-dialup-trusted-location', if it
isn't then it uses the PolicyKit action 'nm-dialup-untrusted-location'.

switch to XML for policy definition files and introduce descriptions

Descriptions will be subject to i18n/l10n efforts at some point.

Also add a new tool polkit-list-actions.

also remember to do s/libpolkit/polkit/ on the pkg-config files

rename libpolkit to polkit

It's more consistent and, uhm, easier to type.

add some validation to public facing functions

remove misguided action parameters

This feature was introduced with

 commit 02a4c5101ca4751963f76a0e016d3308389dc2a5
 http://gitweb.freedesktop.org/?p=PolicyKit.git;a=commit;h=02a4c5101ca4751963f76a0e016d3308389dc2a5

It makes things a lot harder for privilege granting if a feature like
action parameters are present. Thinking about it, they're not really
necessary; the parameters should just be encoded in the resource name;
e.g. with the example given in the commit

> This is useful for letting mechanisms convey information which may be
> useful in making a decision whether an action is OK. For example,
> NetworkManager could use this to provide the phone-number parameter
> with a hypothetical "nm-dialup" action. Then a site or vendor can
> provide insert
>
>  mandatory polkit-run-program.so \
>     program="/usr/lib/check-dialup-number.sh" privilege="nm-dialup"
>
> into /etc/PolicyKit/PolicyKit.conf and have said program check
>
>  $POLKIT_ACTION_PARAM_PHONE_NUMBER
>
> in that program.

is broken; the right thing here is for a hypothetical NetworkManager
to pass the dial up connection details as the resource

 resource.type = "NetworkManager"
 resource.id = "/org/freedesktop/NM/DialUpConnection/number=555-HOT-CHICKS"

in a well-defined format etc. etc.

add the PAM configuration file

add pkg-config files for the two new libraries

add the correct version files for docs

add grant functionality + lots of other changes

 - Split libpolkit into three libraries
   - libpolkit : to be used only by mechanisms and modules
   - libpolkit-dbus : utility library for libpolkit to get caller, session
                      etc. info from the bus and ConsoleKit
   - libpolkit-grant : client side library for obtaining privileges; uses
                       a setgid helper internally

 - grant functionality
   - a helper library, libpolkit-grant, to gain privileges
     - includes a setgid $POLKIT_GROUP helper to write granted privileges
   - a PK module, to read and interpret granted privileges
   - a cmdline app, polkit-grant, using said library

 - Other changes
   - so it turns out that sizeof(bool) != sizeof(gboolean), sizeof(dbus_bool_t)
     This blows so define our own polkit_bool_t type
   - add some validation routines

The grant functionality, especially the setgid helper needs thorough
security review before we can release it.

remove all usage of glib from the header files

This paves the way for getting rid of a glib dependency; when and if
that happens is to be determined; right now it just doesn't make a lot
of sense to reimplement GKeyFile, GSList, GHashTable, g_spawn_sync and
other useful routines. But it might make sense if we want to get the
message bus daemon to link with libpolkit so you e.g. can say

 <policy polkit="acme-frobnicate">
   <allow send_interface="com.acme.Frobnicator"
 </policy>

to allow a caller on the system message bus to access that interface
if, and only if, he can do the "acme-frobnicate" action according to
PolicyKit.

associate parameters (key/value pairs) with the Action class

This is useful for letting mechanisms convey information which may be
useful in making a decision whether an action is OK. For example,
NetworkManager could use this to provide the phone-number parameter
with a hypothetical "nm-dialup" action. Then a site or vendor can
provide insert

 mandatory polkit-run-program.so program="/usr/lib/check-dialup-number.sh" privilege="nm-dialup"

into /etc/PolicyKit/PolicyKit.conf and have said program check

 $POLKIT_ACTION_PARAM_PHONE_NUMBER

in that program.

remember to set session's ConsoleKit object path

policy files now have the .policy extension, not .priv

no need to reference polkit-module-default twice in a man page

minor fixes to the PolicyKit manual page

mass renaming of classes

 Privilege -> Action
 PrivilegeFile -> PolicyFile
 PrivilegeFileEntry -> PolicyFileEntry
 PrivilegeCache -> PolicyCache

Hopefully it makes a bit more sense now.

fix up Makefile.am

generate and dist ChangeLog

add built-in options and a new module pam-polkit-run-program.so

stop disting ChangeLog and start disting modules/PolicyKit.conf

Am happy to report that 'make distcheck' now works!

add module loading to PolicyKit

This paves the way for writing

 1. A module that tracks temporary (look in /var/run) and permanent (look
    in /var/lib) privilege grants
 2. A D-Bus service to authenticate a client to obtain to a privilege
    grant and then writing the grant in temporary or permanent storage

Also, this feature lets people very easily lock down the system; just
edit /etc/PolicyKit/PolicyKit.conf; add pam-module-deny-all / -allow-all
stanzas with various privilege=<regexp> and user=<username> options.

watch the correct directory

really ignore privilege files starting with "." this time

Also avoid dumping all privilege file entries to debug whenever we
populate our privilege cache.

refine file monitoring interface and implement policy reload

make polkit-privilege-file-validate accept multiple files

move to using _pk_debug and respect $POLKIT_DEBUG

read privilege files and actually use the policy described in those

provide an example privilege file

include PolKitPrivilegeFile in API docs

introduce a PolKitResult enumeration and make privilege files use that

add support for privilege files and provide a validation tool

get rid of AS_AC_EXPAND and use autoconf docdir

Inspired by recent patches to both HAL and ConsoleKit from Michael
Biebl <mbiebl@gmail.com>.

no need to mention $(top_srcdir) twice

build fix so 'make distcheck' works

add two tools polkit-check-caller and polkit-check-session

implement the classes carrying data

document the API

Now I only need to implement it...

split libpolkit into more source/header files

proposed API

update to the gtk-doc.make from the HAL project

reset project and remove all existing code

Some of the code, e.g. the daemon, will be brought back in other forms.

fix make distcheck

A quick (and dirty) fix to get 'make distcheck' complete running and
to fix polkitd-test

stop configure if pam/pam-devel is missing

This little patch fixes configure to stop and print a error message if
pam/pam-devel is missing while run configure for PolicyKit

Patch from Danny Kukawka <danny.kukawka@web.de> with some cleanup.s

* pam-polkit-console/pam-polkit-console.c: (pam_sm_open_session),
(pam_sm_close_session): fixed usage of pam_get_item() to respect
strict-aliasing rules

        * polkitd/polkit-manager.c: (polkit_manager_get_caller_info):
added ifdef around currently unused calling_selinux_context

        * tools/polkit-grant-privilege.c: (grant_complete_cb): replaced
libpolkit_free_context() with correct libpolkit_grant_free_context()