[Moosefs-users] Bug in 1.6.24 Debian init scripts

[Steve W. <st...@pu...>]

Hi,

The initialization of the DATADIR, MFSUSER, and MFSGROUP variables in 
the Debian init scripts is incorrect in the latest release of MooseFS 
(1.6.24).  Here are the original lines:
>     DATADIR=$(sed -e 's/^DATA_PATH[     ]*=[    ]*\([^  ]*\)[   
> ]*$/\1/' "$CFGFILE")
>     MFSUSER=$(sed -e 's/^WORKING_USER[  ]*=[    ]*\([^  ]*\)[   
> ]*$/\1/' "$CFGFILE")
>     MFSGROUP=$(sed -e 's/^WORKING_GROUP[    ]*=[    ]*\([^  ]*\)[   
> ]*$/\1/' "$CFGFILE")
This places the contents of the whole config file into each variable 
with one or more lines modified by sed.

These lines should be changed to something like the following:
>     DATADIR=$(grep "^DATA_PATH" "$CFGFILE" | tail -1 | sed -e 
> 's/^DATA_PATH[ \t]*=[ \t]*\([^ \t]*\)[ \t]*$/\1/')
>     MFSUSER=$(grep "^WORKING_USER" "$CFGFILE" | tail -1 | sed -e 
> 's/^WORKING_USER[ \t]*=[ \t]*\([^ \t]*\)[ \t]*$/\1/')
>     MFSGROUP=$(grep "^WORKING_GROUP" "$CFGFILE" | tail -1 | sed -e 
> 's/^WORKING_GROUP[ \t]*=[ \t]*\([^ \t]*\)[ \t]*$/\1/')

Thanks,
Steve