修正phpBB 2.0.x两个安全漏洞

王朝php·作者佚名 2006-01-08

宽屏版字体: 小|中|大|超大

phpBB开发组最近公布了phpBB 2.0.x两个安全漏洞的修正办法：

1. SQL注射漏洞：

修改viewtopic.php。在

代码:

if ( isset($HTTP_GET_VARS[POST_TOPIC_URL]) )

{

$topic_id = intval($HTTP_GET_VARS[POST_TOPIC_URL]);

}

else if ( isset($HTTP_GET_VARS['topic']) )

{

$topic_id = intval($HTTP_GET_VARS['topic']);

}

之前加入：

代码:

$topic_id = $post_id = false;

将以下代码：

代码:

$join_sql_table = ( !isset($post_id) ) ? '' : ", " . POSTS_TABLE . " p, " . POSTS_TABLE . " p2 ";

$join_sql = ( !isset($post_id) ) ? "t.topic_id = $topic_id" : "p.post_id = $post_id AND t.topic_id = p.topic_id AND p2.topic_id = p.topic_id AND p2.post_id <= $post_id";

$count_sql = ( !isset($post_id) ) ? '' : ", COUNT(p2.post_id) AS prev_posts";

$order_sql = ( !isset($post_id) ) ? '' : "GROUP BY p.post_id, t.topic_id, t.topic_title, t.topic_status, t.topic_replies, t.topic_time, t.topic_type, t.topic_vote, t.topic_last_post_id, f.forum_name, f.forum_status, f.forum_id, f.auth_view, f.auth_read, f.auth_post, f.auth_reply, f.auth_edit, f.auth_delete, f.auth_sticky, f.auth_announce, f.auth_pollcreate, f.auth_vote, f.auth_attachments ORDER BY p.post_id ASC";

替换为：

代码:

$join_sql_table = ( empty($post_id) ) ? '' : ", " . POSTS_TABLE . " p, " . POSTS_TABLE . " p2 ";

$join_sql = ( empty($post_id) ) ? "t.topic_id = $topic_id" : "p.post_id = $post_id AND t.topic_id = p.topic_id AND p2.topic_id = p.topic_id AND p2.post_id <= $post_id";

$count_sql = ( empty($post_id) ) ? '' : ", COUNT(p2.post_id) AS prev_posts";

$order_sql = ( empty($post_id) ) ? '' : "GROUP BY p.post_id, t.topic_id, t.topic_title, t.topic_status, t.topic_replies, t.topic_time, t.topic_type, t.topic_vote, t.topic_last_post_id, f.forum_name, f.forum_status, f.forum_id, f.auth_view, f.auth_read, f.auth_post, f.auth_reply, f.auth_edit, f.auth_delete, f.auth_sticky, f.auth_announce, f.auth_pollcreate, f.auth_vote, f.auth_attachments ORDER BY p.post_id ASC";

2. styles_admin 越界访问漏洞：

修改admin/admin_styles.php。

将以下代码：

代码:

// Load default header

// Check if the user has cancled a confirmation message.

$phpbb_root_path = "./../";

$confirm = ( isset($HTTP_POST_VARS['confirm']) ) ? TRUE : FALSE;

$cancel = ( isset($HTTP_POST_VARS['cancel']) ) ? TRUE : FALSE;

if (empty($HTTP_POST_VARS['send_file']))

{

$no_page_header = ( $cancel ) ? TRUE : FALSE;

require($phpbb_root_path . 'extension.inc');

require('./pagestart.' . $phpEx);

}

if ($cancel)

{

redirect('admin/' . append_sid("admin_styles.$phpEx", true));

}