Bug 625228
Summary: | gdm does not honor CheckDirOwner | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | Red Hat Enterprise Linux 5 | Reporter: | Jeff Bastian <jbastian> | ||||||
Component: | gdm | Assignee: | Ray Strode [halfline] <rstrode> | ||||||
Status: | CLOSED WONTFIX | QA Contact: | desktop-bugs <desktop-bugs> | ||||||
Severity: | medium | Docs Contact: | |||||||
Priority: | low | ||||||||
Version: | 5.5 | CC: | cww, tao | ||||||
Target Milestone: | rc | ||||||||
Target Release: | --- | ||||||||
Hardware: | All | ||||||||
OS: | Linux | ||||||||
Whiteboard: | |||||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||||
Doc Text: | Story Points: | --- | |||||||
Clone Of: | Environment: | ||||||||
Last Closed: | 2011-10-18 14:08:28 UTC | Type: | --- | ||||||
Regression: | --- | Mount Type: | --- | ||||||
Documentation: | --- | CRM: | |||||||
Verified Versions: | Category: | --- | |||||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||
Cloudforms Team: | --- | Target Upstream Version: | |||||||
Embargoed: | |||||||||
Attachments: |
|
Description
Jeff Bastian
2010-08-18 22:28:16 UTC
Created attachment 439527 [details]
patch for CheckDirOwner
This patch will check the CheckDirOwner status when looking at the owner of files in addition to the home directory. The warnings do not appear when using this patch.
Maybe a CheckFileOwner option should be added instead?
Comment on attachment 439527 [details] patch for CheckDirOwner >diff --git a/daemon/filecheck.c b/daemon/filecheck.c >index 5cf6006..8785f85 100644 >--- a/daemon/filecheck.c >+++ b/daemon/filecheck.c >@@ -110,8 +110,11 @@ gdm_file_check (const gchar *caller, uid_t user, const gchar *dir, > return FALSE; > } > >- /* ... owned by the user ... */ >- if G_UNLIKELY (statbuf.st_uid != user) { >+ /* ... owned by the user ... >+ If the directory is not owned by the user, it's likely the files are >+ not owned by the user either, so suppress the checks on file >+ ownership too if CheckDirEnabled is false. */ >+ if G_UNLIKELY (gdm_get_value_bool (GDM_KEY_CHECK_DIR_OWNER) && (statbuf.st_uid != user)) { > syslog (LOG_WARNING, _("%s: %s is not owned by uid %d."), caller, fullpath, user); > g_free (fullpath); > return FALSE; I modified the patch slightly in comment 3 by adding the "if CheckDirEnabled is false" statement to the comments. Created attachment 439529 [details]
patch for CheckDirOwner
Argh, that should be CheckDirOwner, not CheckDirEnabled.
Here's a whole new patch to avoid confusion.
This request was evaluated by Red Hat Product Management for inclusion in the current release of Red Hat Enterprise Linux. Because the affected component is not scheduled to be updated in the current release, Red Hat is unfortunately unable to address this request at this time. Red Hat invites you to ask your support representative to propose this request, if appropriate and relevant, in the next release of Red Hat Enterprise Linux. |