Attachment 13690 Details for Bug 14053 – implement /var/run/console/<username> tags

[patch] implement /var/run/console/<username> tags

pam_console-compat.patch (text/plain), 4.00 KB, created by Martin Pitt on 2008-01-13 04:50:12 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Pitt

Created: 2008-01-13 04:50:12 UTC

Size: 4.00 KB

patch

obsolete

>diff -Nur -x '*.orig' -x '*~' consolekit-0.2.3/src/ck-manager.c consolekit-0.2.3.new/src/ck-manager.c
>--- consolekit-0.2.3/src/ck-manager.c	2007-08-28 19:16:03.000000000 +0200
>+++ consolekit-0.2.3.new/src/ck-manager.c	2008-01-13 13:44:07.000000000 +0100
>@@ -28,6 +28,8 @@
> #include <signal.h>
> #include <sys/stat.h>
> #include <sys/types.h>
>+#include <pwd.h>
>+#include <errno.h>
> 
> #include <glib.h>
> #include <glib/gi18n.h>
>@@ -484,6 +486,79 @@
>         manager_update_system_idle_hint (manager);
> }
> 
>+#define PAM_CONSOLE_DIR "/var/run/console"
>+/*
>+  Create pam_console like tag file in /var/run/console
>+*/
>+static const char*
>+get_pam_console_tagfile(CkSession *session)
>+{
>+        guint uid;
>+        struct passwd *pwd;
>+        static char tagfile[256];
>+
>+        /* get user name */
>+        if (!ck_session_get_unix_user (session, &uid, NULL)) {
>+                g_warning ("create_pam_console_tag(): Could not get Unix user");
>+                return NULL;
>+        }
>+        if (!(pwd = getpwuid (uid))) {
>+                g_error ("Could not map unix user %i to user name", uid);
>+                return NULL;
>+        }
>+
>+        snprintf (tagfile, sizeof(tagfile), "%s/%s", PAM_CONSOLE_DIR, pwd->pw_name);
>+        return tagfile;
>+}
>+
>+static void
>+create_pam_console_tag(CkSession *session)
>+{
>+        const char* tagfile;
>+        struct stat st;
>+        FILE *f;
>+
>+        tagfile = get_pam_console_tagfile(session);
>+        if (!tagfile)
>+                return;
>+
>+        /* ensure that PAM_CONSOLE_DIR is a directory */
>+        mkdir (PAM_CONSOLE_DIR, 0755);
>+        if (stat (PAM_CONSOLE_DIR, &st) != 0 ||
>+            !S_ISDIR(st.st_mode)) {
>+                g_error (PAM_CONSOLE_DIR " is not a directory");
>+                return;
>+        }
>+
>+        g_debug ("Creating tag file %s", tagfile);
>+        if (!(f = fopen(tagfile, "w"))) {
>+                g_error ("Could not create tag file %s: %s", tagfile, strerror (errno));
>+                return;
>+        }
>+        fclose(f);
>+}
>+
>+static void
>+remove_pam_console_tag(CkSession *session)
>+{
>+        const char* tagfile;
>+        struct stat st;
>+
>+        tagfile = get_pam_console_tagfile(session);
>+        if (!tagfile)
>+                return;
>+
>+        if (stat (tagfile, &st) != 0 ||
>+            !S_ISREG(st.st_mode)) {
>+                g_error ("%s is not a regular file", tagfile);
>+                return;
>+        }
>+
>+        g_debug ("Removing tag file %s", tagfile);
>+        if (unlink(tagfile) != 0)
>+                g_error ("Could not remove tag file %s: %s", tagfile, strerror (errno));
>+}
>+
> /*
>   Example:
>   dbus-send --system --dest=org.freedesktop.ConsoleKit \
>@@ -587,6 +662,8 @@
> 
>         ck_seat_add_session (seat, session, NULL);
> 
>+        create_pam_console_tag(session);
>+
>         /* FIXME: connect to signals */
>         /* FIXME: add weak ref */
> 
>@@ -1222,6 +1299,32 @@
>                 }
>         }
> 
>+        /* remove pam_console tag file if this was the last session for the user */
>+        guint my_uid;
>+
>+        if (ck_session_get_unix_user (session, &my_uid, NULL)) {
>+            GHashTableIter iter;
>+            gpointer key, value;
>+            gboolean last_user_session = TRUE;
>+            guint uid;
>+
>+            g_hash_table_iter_init (&iter, manager->priv->sessions);
>+            while (g_hash_table_iter_next (&iter, &key, &value)) {
>+                    if (value == session)
>+                            continue;
>+                    if (!ck_session_get_unix_user ((CkSession*) value, &uid, NULL))
>+                            continue;
>+                    if (uid == my_uid) {
>+                            last_user_session = FALSE;
>+                            break;
>+                    }
>+            }
>+            if (last_user_session)
>+                    remove_pam_console_tag (session);
>+            else
>+                    g_debug ("Unix user %i has more active sessions, not removing pam_console tag file", my_uid);
>+        }
>+
>         g_hash_table_remove (manager->priv->sessions, ssid);
> 
>         g_free (sid);

Actions: View | Splinter Review

Attachments on bug 14053: 13690 | 14034 | 17717

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.