[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#249784: kdm / selinux patch



On Sunday 28 November 2004 14:33, Luke Kenneth Casson Leighton wrote:
> yes the patch works.

+        if (setexeccon(scontext)) {
+             freecon(scontext);
+             LogError("Failed to set exec security context %s for %s.", 
scontext, name);
+        }
+        freecon(scontext);

That looks like a double-deletion to me. Not great to have in code that runs 
as root.
-- 
Thanks,
Richard



Reply to: