#42 Path separator specified wrongly in openwem.conf comments

[Manju]

For many multi-valued path options in openwbem.conf (e.g, cppprovifc.prov_location), the separator is specified as ":" for Windows and ";" for POSIX systems.

But giving two paths separated by ";" (path1;path2) for cppprovifc.prov_location in a Linux machine and running owcimomd resulted in the error "C++ provider ifc failed getting contents of directory:path1;path2", and it failed to load any C++ provider.

The separator should actually be ":" for Linux and ";" for Windows:
In src/common/OW_Types.hpp, the following are defined:
#ifdef OW_WIN32
#define OW_PATHNAME_SEPARATOR ";"
#else
#define OW_PATHNAME_SEPARATOR ":"
#endif

The following patch changes the comments in etc/openwbem.conf.sh to fix this problem in openwbem 3.2.2. (I haven't checked for this bug in any other version.) Apply it from the directory openwbem-3.2.2/etc/ and use the option -p1 for patch.

%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%

--- etc/openwbem.conf.sh 2008-01-11 16:51:15.000000000 +0530
+++ etc/openwbem.conf.sh.new 2008-01-11 14:25:31.000000000 +0530
@@ -20,7 +20,7 @@
################################################################################
# For each directory specified, all the files contained in the directory will
# be loaded and processed as additional config files.
-# This is a multi-valued option. ':' (windows) or ';' (POSIX) is the separator.
+# This is a multi-valued option. ';' (windows) or ':' (POSIX) is the separator.
# This option will be evaluated after the main config file is parsed, and so
# additional directories specified in additional config files will not be
# examined.
@@ -526,7 +526,7 @@
################################################################################
# owcimomd.services_path Specifies the directory containing the services
# shared libraries to be loaded by the CIMOM.
-# This is a multi-valued option. ':' (windows) or ';' (POSIX) is the separator.
+# This is a multi-valued option. ';' (windows) or ':' (POSIX) is the separator.
# You probably don't need to modify this option.
# The default is "@libdir@/openwbem/services"
owcimomd.services_path = @libdir@/openwbem/services
@@ -534,7 +534,7 @@
################################################################################
# owcimomd.request_handler_path Specifies the directory containing the
# request handler shared libraries to be loaded by the CIMOM.
-# This is a multi-valued option. ':' (windows) or ';' (POSIX) is the separator.
+# This is a multi-valued option. ';' (windows) or ':' (POSIX) is the separator.
# You probably don't need to modify this option.
# The default is "@libdir@/openwbem/requesthandlers"
owcimomd.request_handler_path = @libdir@/openwbem/requesthandlers
@@ -566,7 +566,7 @@
# interfaces will be loaded from. owcimomd assumes all shared libraries in
# these directories are provider interfaces. If a shared library in this directory
# does not support the provider interface api, it will be rejected.
-# This is a multi-valued option. ':' (windows) or ';' (POSIX) is the separator.
+# This is a multi-valued option. ';' (windows) or ':' (POSIX) is the separator.
# You probably don't need to modify this option.
# The default is "@libdir@/openwbem/provifcs"
owcimomd.provider_ifc_libs = @libdir@/openwbem/provifcs
@@ -575,7 +575,7 @@
# One of the provider interfaces provided with owcimomd is the C++ provider
# interface. The cppprovifc.prov_location option specifies where the C++
# provider interface will load it's providers from.
-# This is a multi-valued option. ':' (windows) or ';' (POSIX) is the separator.
+# This is a multi-valued option. ';' (windows) or ':' (POSIX) is the separator.
# You probably don't need to modify this option.
# The default is "@libdir@/openwbem/c++providers"
cppprovifc.prov_location = @libdir@/openwbem/c++providers
@@ -584,7 +584,7 @@
# One of the provider interfaces provided with owcimomd is the OWBI1 provider
# interface. The owbi1provifc.prov_location option specifies where the OWBI1
# provider interface will load it's providers from.
-# This is a multi-valued option. ':' (windows) or ';' (POSIX) is the separator.
+# This is a multi-valued option. ';' (windows) or ':' (POSIX) is the separator.
# You probably don't need to modify this option.
# The default is "@libdir@/openwbem/owbi1providers"
owbi1provifc.prov_location = @libdir@/openwbem/owbi1providers
@@ -609,7 +609,7 @@
# One of the provider interfaces provided with owcimomd is the NPI provider
# interface. The npiprovifc.prov_location option specifies where the NPI
# provider interface will load it's providers from.
-# This is a multi-valued option. ':' (windows) or ';' (POSIX) is the separator.
+# This is a multi-valued option. ';' (windows) or ':' (POSIX) is the separator.
# The default is "@libdir@/openwbem/npiproviders"
npiprovifc.prov_location = @libdir@/openwbem/npiproviders

@@ -617,7 +617,7 @@
# One of the provider interfaces provided with owcimomd is the CMPI provider
# interface. The cmpiprovifc.prov_location option specifies where the CMPI
# provider interface will load it's providers from.
-# This is a multi-valued option. ':' (windows) or ';' (POSIX) is the separator.
+# This is a multi-valued option. ';' (windows) or ':' (POSIX) is the separator.
# The default is "@libdir@/openwbem/cmpiproviders"
cmpiprovifc.prov_location = @libdir@/openwbem/cmpiproviders

@@ -634,7 +634,7 @@
# One of the provider interfaces provided with owcimomd is the perl provider
# interface. The perlprovifc.prov_location option specifies where the perl
# provider interface will load it's providers from.
-# This is a multi-valued option. ':' (windows) or ';' (POSIX) is the separator.
+# This is a multi-valued option. ';' (windows) or ':' (POSIX) is the separator.
# The default is "@libdir@/openwbem/perlproviders"
perlprovifc.prov_location = @libdir@/openwbem/perlproviders

%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%

--Manju

[Kevin Harris]

Will be fixed shortly with a merge of some local changes to OW trunk. I'll close it when we actually merge.