[freewnn:00782] Re: [RFC] error.c rewrite

[aono@XXXX (Tomoki AONO)]

青野です。こんばんは。
＃多忙(?)のため、反応が悪くすみません。

<86g056xtbk.wl@chrysanthe.oikumene.gcd.org>の記事において
hiroo@oikumene.gcd.orgさんは書きました。

>> αバージョンのリリースを優先すべきでしょうね。

そう思います。
＃意思決定の話に戻ると厄介ですが…。青野の私見は
＃[freewnn:00779]をご参照下さい。

>> > ついでに[freewnn:00774]のsignal_handlerについてもいくつか
>> 
>> このままだとほかにも、fork() する場合は _exit(2) とか、もっとちゃんと
>> 考えないといけませんね…。

え、そうなんですか…。もっと他のソースを見て勉強しなければ…。

[freewnn:00772]のコードにミスがありましたのでパッチをつけ
ます。ANSI_PROTOTYPES未定義時のチェックのし忘れでした。

--- Wnn/jserver/error.c.orig	Tue Dec 25 18:28:45 2001
+++ Wnn/jserver/error.c	Wed Jan 23 16:36:30 2002
@@ -155,8 +155,8 @@
 #ifdef ANSI_PROTOTYPES
 srv_debug(const char *fmt, ...)
 #else
-srv_debug(format, va_alist)
-  const char *format;
+srv_debug(fmt, va_alist)
+  const char *fmt;
   va_dcl
 #endif
 {
@@ -173,8 +173,8 @@
 #ifdef ANSI_PROTOTYPES
 srv_err(const char *fmt, ...)
 #else
-srv_err(format, va_alist)
-  const char *format;
+srv_err(fmt, va_alist)
+  const char *fmt;
   va_dcl
 #endif
 {

P.S.

いささか旧聞に属しますが、以前のFreeWnn/Wnn4.2のmkdir()代
替関数の実装(system()使用)で「;」などのシェル特殊文字のチェッ
クがなかったことをUNYUN氏が氏のサイトで指摘しています。
(cf. http://www.shadowpenguin.org/sc_advisories/advisory044.html)
fork() & exec()を使用するようになった現在(1.1+パッチも含む)
はこの問題点がないそうですが、ご参考までに。
＃以上は 小島氏のセキュリティホールmemoで知りました(^^;。
----
青野智樹	(aono@cc.osaka-kyoiku.ac.jp)