[Mlterm-dev-ja] x_get_all_config_font_names() でメモリ破壊

[NODA, K. <no...@gm...>]

mlterm開発者の皆様、こんにちは
高品質なフリーソフトウェアをどうもありがとうございます。

私の環境ではvalgrindで警告が出ていたので以下のパッチを作ってみました。
ご検討ください。

# HG changeset patch
# User NODA, Kai <no...@gm...>
# Date 1315499802 -32400
# Node ID 6bdd98e43f886a5808bdb0a4b5d5ce7158710f65
# Parent  96d3c62a0c2cf24939fbb619a4f2d4400e62595c
Fixed a memory corruption.

In x_get_all_config_font_names(), when size == 1 and d_size == 0 and
array[0]->key == DEFAULT_FONT, neither strcpy() nor sprintf() are done, thus
p is left at the first character of the blank buffer font_name_list.
In that case it's wrong to do simply *(p - 1) = '\0'.

diff -r 96d3c62a0c2c -r 6bdd98e43f88 xwindow/x_font_config.c
--- a/xwindow/x_font_config.c   Sun Sep 04 08:39:06 2011 +0900
+++ b/xwindow/x_font_config.c   Fri Sep 09 01:36:42 2011 +0900
@@ -1983,7 +1983,8 @@
                }
        }
 
-       *(p - 1) = '\0' ;
+       if (p > font_name_list) --p;
+       *p = '\0' ;
 
 #ifdef  DEBUG
        kik_debug_printf( KIK_DEBUG_TAG " Font list is %s\n" , font_name_list) ;

以上です。

-- 
NODA, Kai <no...@gm...>