Re: [Mlterm-dev-ja] utmp support for linux glibc

[Araki K. <j00...@ip...>]

	荒木です:-)

        dev-ja ML にも CC してます。
        
	Subject: Re: [Mlterm-dev-ja] utmp support for linux glibc
	From: Araki Ken <j00...@ip...>
	Message-ID: <7zr8nhocgh.fsf@totally-fudged-out-message-id>
	Date: Tue, 19 Feb 2002 21:58:10 +0900

>> enable のとき root に suid はかなり危険な気がするので、デフォルトにするのは抵抗があります。
>> デフォルトは sgid にして、 動かない環境では修正して貰う方がよくないですか?
>         そうですね。では、添付のパッチのような感じにしてみます。

        と思ったんですが、/etc/group に、utmp があるかどうかを grep で調べて、
        あれば、root/utmp の 2755 、なければ、root/root の 4755 にするというの
        でどうでしょうか?

        後者を 2755 にするかどうかは悩むのですが、utmp グループがない環境では、
        おそらく、4755 でないと utmp は使えないでしょうし、それでもあえて
        --enable-utmp していると考えれば、4755 がデフォルトでもよいかな、と思
        いました。

        パッチを添付します。

	では
--
kiken
j00...@ip...

Index: autoconf/configure.in
===================================================================
RCS file: /home/ken/cvsroot/mlterm/autoconf/configure.in,v
retrieving revision 1.37
diff -u -r1.37 configure.in
--- autoconf/configure.in	2002/02/13 14:06:48	1.37
+++ autoconf/configure.in	2002/02/19 13:31:53
@@ -108,12 +108,17 @@
 	utmp=$enable_utmp)
 if test "$utmp" = "yes" ; then
 	UTMP_CFLAGS="-DUSE_UTMP"
-	PERMISSION="4755"
+	utmp=`grep utmp /etc/group 2>/dev/null`
+	if test "$utmp" ; then
+		INSTALL_OPT="-m 2755 -g utmp"
+	else
+		INSTALL_OPT="-m 4755 -o root"
+	fi
 else
-	PERMISSION="755"
+	INSTALL_OPT="-m 755"
 fi
 AC_SUBST(UTMP_CFLAGS)
-AC_SUBST(PERMISSION)
+AC_SUBST(INSTALL_OPT)
 
 AC_ARG_ENABLE(ucs4,
 	[  --enable-ucs4           ucs4[disabled]],
Index: src/Makefile.in
===================================================================
RCS file: /home/ken/cvsroot/mlterm/src/Makefile.in,v
retrieving revision 1.29
diff -u -r1.29 Makefile.in
--- src/Makefile.in	2002/02/19 08:31:55	1.29
+++ src/Makefile.in	2002/02/19 13:32:25
@@ -39,7 +39,7 @@
 	-L/usr/X11R6/lib -L/usr/local/lib -R/usr/X11R6/lib -R/usr/local/lib
 
 PROG = mlterm
-PERMISSION = @PERMISSION@
+INSTALL_OPT = @INSTALL_OPT@
 
 LIBTOOL_LINK = $(LIBTOOL) --mode=link $(CC)
 LIBTOOL_INSTALL = $(LIBTOOL) --mode=install $(INSTALL)
@@ -61,7 +61,7 @@
 	mkdir -p $(BINDIR)
 
 install : $(BINDIR)
-	$(LIBTOOL_INSTALL) -m $(PERMISSION) $(PROG) $(BINDIR)
+	$(LIBTOOL_INSTALL) $(INSTALL_OPT) $(PROG) $(BINDIR)
 
 wc :
 	find . -name "*.[ch]" -a \! -name "test_*" | xargs wc -l