作者x4base (RU~)
看板PttBug
标题[讨论] svn版talk.c使用者名单好像有bug
时间Mon Mar 19 16:06:13 2012
我不知道有没有po错板
目前ptt trunk 里talk.c
draw_pickup中的以下程式码
列出使用者的地方
2034 vs_cols(ulist_coldef, cols, ULISTCOLS,
2035 num, pager,
2036 fcolor[state] ? xuid : uentp->userid,
2037 uentp->nickname,
2038 descript(show_mode, uentp,
2039 uentp->pager & !(friend & HRM), description,
sizeof(description)),
2040 #if defined(SHOWBOARD) && defined(DEBUG)
2041 show_board ? (uentp->brc_id == 0 ? "" :
2042 getbcache(uentp->brc_id)->brdname) :
2043 #endif
2044 modestring(uentp, 0),
2045 *mind ? xmind : "",
2046 #ifdef SHOW_IDLE_TIME
2047 idlestr
2048 #else
2049 ""
2050 #endif
2051 );
因为
1860 #define ULISTCOLS (9)
所以传给vs_cols的参数
从num开始算应该要有九个对吧？
相较之下 稍早出现的列出标题的片段：
1920 vs_cols(ulist_coldef, cols, ULISTCOLS,
1921 show_uid ? "UID" : "编号",
1922 "P", "代号", "昵称",
1923 MODE_STRING[show_mode],
1924 show_board ? "看板" : "动态",
1925 // because this field has ANSI...
1926 show_pid ? "PID" : "心情",
1927 #ifdef SHOW_IDLE_TIME
1928 "发呆",
1929 #else
1930 " "
,
1931 #endif
1932
"");
就有九个参数
因为"发呆"对应的是 idlestr
所以列出使用者的地方应该是少参数吧
我把
2046 #ifdef SHOW_IDLE_TIME
2047 idlestr
2048 #else
2049 ""
2050 #endif
2051 );
改成
2046 #ifdef SHOW_IDLE_TIME
2047 idlestr
,
2048 #else
2049 ""
,
2050 #endif
2051
"" );
就可以work了
要不然原本vs_cols里回圈读到i=8时
s = va_arg(ap, char*);
会让s读到不该读的记忆体
有时会有bad memory access
希望我没有想错～
谢谢
附上patch
##################################################
diff --git a/mbbsd/talk.c b/mbbsd/talk.c
index d8f619c..c0e62d5 100644
--- a/mbbsd/talk.c
+++ b/mbbsd/talk.c
@@ -2044,10 +2044,11 @@ draw_pickup(int drawall, pickup_t * pickup, int
pickup_way,
modestring(uentp, 0),
*mind ? xmind : "",
#ifdef SHOW_IDLE_TIME
- idlestr
+ idlestr,
#else
- ""
+ "",
#endif
+ ""
);
}
}
###################################################
--
※ 发信站: 批踢踢实业坊(ptt.cc)
◆ From: 140.110.203.61
※ 编辑: x4base 来自: 140.110.203.61 (03/19 16:12)