found the original message. not sure if post ever made it to lists.
bug highlighted with ^^^^
please acknowledge receipt of message, confirming awareness of bug in
patch.
thanks.
l.
On Fri, Aug 27, 2004 at 09:32:02AM -0400, Daniel J Walsh wrote:
Further cleanup and using all static inlines versus defines. Renamed
a
couple of functions to make them clearer.
Dan
--- /dev/null 2004-06-21 15:29:38.000000000 -0400
+++ udev-030/selinux.h 2004-08-27 09:26:40.160862612 -0400
+static inline void selinux_setfscreatecon(char *file, unsigned int
mode) {
+ int retval = 0;
+ security_context_t scontext=NULL;
+
+ if (is_selinux_running()) {
+ if (matchpathcon(file, S_IFDIR, &scontext) < 0) {
^^^^^^^
this should be matchpatchon(file, mode, &scontext)
+ dbg("matchpathcon(%s) failed\n", file);
+ } else {
+ retval=setfscreatecon(scontext);
+ if (retval < 0)
+ dbg("setfiles %s failed with error '%s'",
+ file, strerror(errno));
+ freecon(scontext);
+ }