[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    Re: kdm / selinux patch
From:       Richard Smith <kde () metafoo ! co ! uk>
Date:       2004-11-29 12:37:04
Message-ID: 200411291237.04954.kde () metafoo ! co ! uk
[Download RAW message or body]

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
 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
[prev in list] [next in list] [prev in thread] [next in thread]