- 帖子
- 105
- 主题
- 98
- 精华
- 0
- 积分
- 263
- 贡献
- 105
- 激情
- 469
- 阅读权限
- 100
- 最后登录
- 2019-8-18
|
[Bug&安全&补丁] 解决问题sprintf() [function.sprintf]: Too few arguments
本帖最后由 逸乐吱声 于 2013-2-18 17:22 编辑
Q+ r( O' q& H7 ?$ j F$ E! L* E8 Y1 |3 c- c8 r" A& I- w8 t K* D
Discuz [function.sprintf]: Too few arguments 错误
: ~$ u* W1 r& Q# I/ G" [9 i9 S3 @' |6 x8 P
# d! {/ _. ^( f
5 p2 G! k* ^1 i4 G0 D3 s* O今天一个客户反映论坛中一篇帖子打不开,其它帖子正常。4 c' V4 O0 D9 d
我打开看了下,详细错误信息是:6 O+ U. J3 Z5 M
; A/ K* c$ w# n! [9 M. ?7 q- PHP Warning: sprintf() [<a href='function.sprintf'>function.sprintf</a>]: Too few arguments in E:\wwwroot\bbs\include\discuzcode.func.php on line 369
复制代码 % Q5 \2 o! E) `* U9 {7 v% B
( N% [/ Z9 q8 {求助了下百度,谷哥,很多都是英文帖子,也看懂了一点,但还是没有头绪。( U* _7 R# ]9 m3 L8 Y
8 `. g' K' R1 B! G& x6 X5 ^ t5 ]于是找到文件中错误代码的位置:0 [4 n" V* K- O1 J$ H1 L; K d" r. Q+ h+ V" r
5 [: X* Z- A7 u0 T7 T- function bbcodeurl($url, $tags) {
- if(!preg_match("/<.+?>/s", $url)) {
- if(!in_array(strtolower(substr($url, 0, 6)), array('http:/', 'https:', 'ftp://', 'rtsp:/', 'mms://'))) {
- $url = 'http://'.$url;
- }
- //这一行错误
- return str_replace(array('submit', 'logging.php'), array('', ''), sprintf($tags, $url, addslashes($url)));
- } else {
- return ' '.$url;
- }
- }
复制代码
$ L4 d" X Y6 ?+ y+ `. w4 c# f+ O- c, q( G9 R
查了下php手册,sprintf并没有很特殊的要求,只有字符串中的%要与参数一一对应。另外又想,既然其它帖子没0 l3 _4 W# Y6 m U M% ~9 o
问题,只有这一篇,肯定跟帖子内容也有关系的。于是把相关的帖子导出来,并不多,看了下,果然帖子中有如
& ~; h: e. c! \, S5 l& u3 E下一个图片地址:
# T6 \) {8 }* c) O8 b
- R( Z4 e# W; ^. W- http://2012.zhenai.com/userfiles/images/%E5%B7%85%E5%B3%B0.jpg
复制代码
; c d9 I T* f$ n+ _! G& l
4 V( j# J! D* \0 R# G* `看来问题在这里了。肯定是这个网址中的%没有编码就被传入到sprintf中了。调用到bbcodeurl函数的有几个地方4 S5 c# J" T9 Z$ v( j- H
,在 discuzcode 这个主函数里有几处
4 ~: u' L; @9 {" D* _. z9 u/ f8 F( Y! P: n( O Y( r ^
- if(!$bbcodeoff) {
- if($parsetype != 1 && strpos($msglower, '[swf]') !== FALSE) {
- $message = preg_replace("/\[swf\]\s*([^\[\<\r\n]+?)\s*\[\/swf\]/ies",
- "bbcodeurl('\\1', ' <img src=\"images/attachicons/flash.gif\" align=\"absmiddle\" alt=\"\" /> <a
- href=\"%s\" target=\"_blank\">Flash: %s</a> ')", $message);
- }
- if(strpos($msglower, '[/img]') !== FALSE) {
- $message = preg_replace(array(
- "/\[img\]\s*([^\[\<\r\n]+?)\s*\[\/img\]/ies",
- "/\[img=(\d{1,4})[x|\,](\d{1,4})\]\s*([^\[\<\r\n]+?)\s*\[\/img\]/ies"
- ), $allowimgcode ? array(
- "bbcodeurl('\\1', '<img src=\"%s\" onload=\"thumbImg(this)\" alt=\"\" />')",
- "parseimg('\\1', '\\2', '\\3')"
- ) : array(
- "bbcodeurl('\\1', '<a href=\"%s\" target=\"_blank\">%s</a>')",
- "bbcodeurl('\\3', '<a href=\"%s\" target=\"_blank\">%s</a>')"
- ), $message);
- }
- }
复制代码 1 ?. [0 D6 a% ?4 N
6 f3 p8 P6 L* Y: k# _
另外,这里也调用到了 parseimg函数, parseimg又调用了bbcodeurl, a: }8 `: Q% F E" G: S* h2 E. e' B3 k# b
5 @: B0 i3 _" u6 \0 d W- function parseimg($width, $height, $src) {
- return bbcodeurl($src, '<img'.($width > 0 ? " width=\"$width\"" : '').($height > 0 ? "
- height=\"$height\"" : '')." src=\"$src\" border=\"0\" alt=\"\" />");
- }
复制代码
2 K; ?4 C1 |# N7 ?, d( U h S0 f1 Q7 L, B
看来问题就在这里了,这里直接把url替换进img标签中,又把整个标签传给 bbcodeurl,url中的%没有被编码。于是加了个replace
1 L: ~( z# n' c$ ^; ?: {1 Wview sourceprint?5 I$ ?$ y9 W, B7 f/ }$ Y. _$ _2 ], _6 {8 R% ~! _
- function parseimg($width, $height, $src) { return bbcodeurl($src, str_replace('%','%%','<img'.($width > 0 ? " width=\"$width\"" : '').($height > 0 ? " height=\"$height\"" : '')." src=\"$src\" border=\"0\" alt=\"\" />"));}
复制代码
2 n) D$ R: ~ Z
: s. f# f. L. B! H" f6 {上传覆盖旧文件,问题成功解决。6 k/ _9 g" M' |+ Y3 K% P' i4 [8 u6 C- z. s8 o Q) M% s8 n; W
q/ x6 \/ R, d6 S参考:http://www.shirne.com/?cid=17&id=157% m J) h& {* @* V5 P& O2 d
|
|