Attachment 58110 Details for Bug 47045 – Fix incorrect argument to is_excluded()

[patch] Fix incorrect argument to is_excluded()

Fix-incorrect-argument-to-isexcluded.patch (text/plain), 1.06 KB, created by Bastien Nocera on 2012-03-07 03:27:43 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Bastien Nocera

Created: 2012-03-07 03:27:43 UTC

Size: 1.06 KB

patch

obsolete

>From 64800affad9818b31fa18ccb7b6ba54d7cc6ecc8 Mon Sep 17 00:00:00 2001
>From: Bastien Nocera <hadess@hadess.net>
>Date: Wed, 7 Mar 2012 12:25:19 +0100
>Subject: [PATCH] Fix incorrect argument to is_excluded()
>
>We're checking whether a user in the wtmp database should
>be excluded, but we use the minimal UID instead of the entry's
>UID to check for addition.
>
>https://bugs.freedesktop.org/show_bug.cgi?id=47045
>---
> src/daemon.c |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
>diff --git a/src/daemon.c b/src/daemon.c
>index 68ed498..d452cf8 100644
>--- a/src/daemon.c
>+++ b/src/daemon.c
>@@ -198,7 +198,7 @@ reload_wtmp_history (Daemon *daemon)
> 
>                 if (daemon_local_user_is_excluded (daemon,
>                                                    wtmp_entry->ut_user,
>-                                                   daemon->priv->minimal_uid)) {
>+                                                   wtmp_entry->ut_pid)) {
>                         g_debug ("excluding user '%s'", wtmp_entry->ut_user);
>                         continue;
>                 }
>-- 
>1.7.7.6

Actions: View | Splinter Review

Attachments on bug 47045: 58110

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.