All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] 98syslog: try to dynamically check for rsyslog moddir
@ 2010-09-08 14:52 Andrey Borzenkov
  2010-09-08 16:48 ` Tomasz Paweł Gajc
  0 siblings, 1 reply; 6+ messages in thread
From: Andrey Borzenkov @ 2010-09-08 14:52 UTC (permalink / raw)
  To: initramfs-u79uwXL29TY76Z2rM5mHXA
  Cc: rsyslog-4gKAAF5ltrKNkp5+ER1W/AC/G2K4zDHf,
	tpg-4qZELD6Fgxhg9hUCZPvPmw

Thomas pointed out that default module path may not be correct
(Mandriva installs modules under architecture dependent path).
Unfortunately rsyslog does not seem to offer any way to query for
default configuration values. Here is rather hack - try to load
non-existing module and fetch path from error message.

Example of error message from version 4.6.4:

rsyslogd: could not load module '/lib64/rsyslog/_No_sUcH_MOduLe.so', dlopen: /lib64/rsyslog/_No_sUcH_MOduLe.so: cannot open shared object file: No such file or directory

Signed-off-by: Andrey Borzenkov <arvidjaar-JGs/UdohzUI@public.gmane.org>

---

 modules.d/98syslog/install |   17 ++++++++++++++++-
 1 files changed, 16 insertions(+), 1 deletions(-)

diff --git a/modules.d/98syslog/install b/modules.d/98syslog/install
index 1fd1b6f..e3d02bf 100755
--- a/modules.d/98syslog/install
+++ b/modules.d/98syslog/install
@@ -1,6 +1,21 @@
 #!/bin/sh
 if type -P rsyslogd >/dev/null; then
-	installs="rsyslogd /lib/rsyslog/lmnet.so /lib/rsyslog/imklog.so /lib/rsyslog/imuxsock.so"
+	# module path can be changed during build and currently rsyslog
+	# does not provide any means to query for it. Use hack - try to
+	# load non-existent module and look at error message
+	R_CONF=/tmp/dracut.rsyslog.conf.$$
+	R_ERR=/tmp/dracut.rsyslog.err.$$
+	R_MODDIR=
+	echo '$ModLoad _No_sUcH_MOduLe' > $R_CONF
+	if [ -s $R_CONF ]; then
+		rsyslogd -N 1 -f $R_CONF 2> $R_ERR > /dev/null
+		R_MODDIR=$(grep ' could not load module .*_No_sUcH_MOduLe' $R_ERR)
+		R_MODDIR=${R_MODDIR#*could not load module \'}
+		R_MODDIR=${R_MODDIR%\', *}
+		R_MODDIR=${R_MODDIR%/*}
+	fi
+	[ -n "$R_MODDIR" ] || R_MODDIR=/lib/rsyslog
+	installs="rsyslogd $R_MODDIR/lmnet.so $R_MODDIR/imklog.so $R_MODDIR/imuxsock.so"
 elif type -P syslogd >/dev/null; then
     installs="syslogd"
 elif type -P syslog-ng >/dev/null; then

^ permalink raw reply related	[flat|nested] 6+ messages in thread
* Re: Re: [PATCH] 98syslog: try to dynamically check for rsyslog moddir
@ 2010-09-10 14:33 Andrey Borzenkov
       [not found] ` <E1Ou4kw-0005pA-00.arvidjaar-mail-ru-eJLJSXF0gEtsdVUOrk1QfQ@public.gmane.org>
  0 siblings, 1 reply; 6+ messages in thread
From: Andrey Borzenkov @ 2010-09-10 14:33 UTC (permalink / raw)
  To: Harald Hoyer
  Cc: Tomasz Paweł Gajc, initramfs-u79uwXL29TY76Z2rM5mHXA,
	rsyslog-4gKAAF5ltrKNkp5+ER1W/AC/G2K4zDHf,
	tpg-4qZELD6Fgxhg9hUCZPvPmw

[-- Attachment #1: Type: text/plain, Size: 2589 bytes --]

On Friday 10 of September 2010 15:48:22 Harald Hoyer wrote:
> On 09/08/2010 06:48 PM, Tomasz Paweł Gajc wrote:
> > Dnia 2010-09-08, o godz. 18:52:13
> > 
> > Andrey Borzenkov<arvidjaar-JGs/UdohzUI@public.gmane.org>  napisał(a):
> >> Thomas pointed out that default module path may not be correct
> >> (Mandriva installs modules under architecture dependent path).
> >> Unfortunately rsyslog does not seem to offer any way to query
> >> for
> >> default configuration values. Here is rather hack - try to load
> >> non-existing module and fetch path from error message.
> >> 
> >> Example of error message from version 4.6.4:
> >> 
> >> rsyslogd: could not load module
> >> '/lib64/rsyslog/_No_sUcH_MOduLe.so', dlopen:
> >> /lib64/rsyslog/_No_sUcH_MOduLe.so: cannot open shared object
> >> file: No such file or directory
> >> 
> >> Signed-off-by: Andrey Borzenkov<arvidjaar-JGs/UdohzUI@public.gmane.org>
> >> 
> >> ---
> >> 
> >>   modules.d/98syslog/install |   17 ++++++++++++++++-
> >>   1 files changed, 16 insertions(+), 1 deletions(-)
> >> 
> >> diff --git a/modules.d/98syslog/install
> >> b/modules.d/98syslog/install index 1fd1b6f..e3d02bf 100755
> >> --- a/modules.d/98syslog/install
> >> +++ b/modules.d/98syslog/install
> >> @@ -1,6 +1,21 @@
> >> 
> >>   #!/bin/sh
> >>   if type -P rsyslogd>/dev/null; then
> >> 
> >> -
> >> installs="rsyslogd /lib/rsyslog/lmnet.so /lib/rsyslog/imklog.so
> >> /lib/rsyslog/imuxsock.so" +	# module path can be changed during
> >> build and currently rsyslog
> >> +	# does not provide any means to query for it. Use hack - try
> >> to
> >> +	# load non-existent module and look at error message
> >> +	R_CONF=/tmp/dracut.rsyslog.conf.$$
> >> +	R_ERR=/tmp/dracut.rsyslog.err.$$
> >> +	R_MODDIR=
> >> +	echo '$ModLoad _No_sUcH_MOduLe'>  $R_CONF
> >> +	if [ -s $R_CONF ]; then
> >> +		rsyslogd -N 1 -f $R_CONF 2>  $R_ERR>  /dev/null
> >> +		R_MODDIR=$(grep ' could not load
> >> module .*_No_sUcH_MOduLe' $R_ERR)
> >> +		R_MODDIR=${R_MODDIR#*could not load module \'}
> >> +		R_MODDIR=${R_MODDIR%\', *}
> >> +		R_MODDIR=${R_MODDIR%/*}
> >> +	fi
> >> +	[ -n "$R_MODDIR" ] || R_MODDIR=/lib/rsyslog
> >> +	installs="rsyslogd $R_MODDIR/lmnet.so $R_MODDIR/imklog.so
> >> $R_MODDIR/imuxsock.so" elif type -P syslogd>/dev/null; then
> >> 
> >>       installs="syslogd"
> >>   
> >>   elif type -P syslog-ng>/dev/null; then
> > 
> > Indeed this patch is more precise than my simplified one ;)
> 
> ugly hack :-/ plus you want mktemp to prevent attacks..

Just to make sure - it was NAK? :)

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2010-09-10 18:25 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-09-08 14:52 [PATCH] 98syslog: try to dynamically check for rsyslog moddir Andrey Borzenkov
2010-09-08 16:48 ` Tomasz Paweł Gajc
     [not found]   ` <20100908184826.7968b1e9-5tc4TXWwyLM@public.gmane.org>
2010-09-10 13:48     ` Harald Hoyer
  -- strict thread matches above, loose matches on Subject: below --
2010-09-10 14:33 Andrey Borzenkov
     [not found] ` <E1Ou4kw-0005pA-00.arvidjaar-mail-ru-eJLJSXF0gEtsdVUOrk1QfQ@public.gmane.org>
2010-09-10 14:39   ` Harald Hoyer
2010-09-10 15:15   ` Peter Jones
     [not found]     ` <4C8A4B91.20905-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2010-09-10 18:25       ` Tomasz Paweł Gajc

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.