SQL inject. Error при "<?=''ss''?>" в назван

Проблемы с установкой или работой phpBB 2.0.x? Ищите ответы здесь!
Lektor
phpBB 2.0.7
Сообщения: 574
Стаж: 19 лет

SQL inject. Error при "<?=''ss''?>" в назван

Сообщение Lektor »

Вот при создании новой темы если название темы ввести например: <?=''ss''?>

Получаю вот что:

Код: Выделить всё

Error in posting

DEBUG MODE

SQL Error : 1136 Column count doesn't match value count at row 1

INSERT INTO phpbb_topics (topic_title, topic_poster, topic_time, forum_id, topic_status, topic_type, topic_dl_type, topic_vote, topic_show_first_post) VALUES ('<?=''ss''?>', 2, 1171241170, 2, 0, 0, 0, 0, 0, 0)

Line : 241
File : functions_post.php
Форум конечно замоден (модов где-то 25)

Вот около 241 строка:

Код: Выделить всё

	if ($mode == 'newtopic' || ($mode == 'editpost' && $post_data['first_post']))
	{
		$topic_vote = (!empty($poll_title) && count($poll_options) >= 2) ? 1 : 0;

		//bt
		global $post_info, $is_auth, $HTTP_POST_VARS;
		$topic_dl_type = (isset($HTTP_POST_VARS['topic_dl_type']) && ($post_info['topic_dl_type'] || $post_info['allow_dl_topic'] || $is_auth['auth_mod'])) ? TOPIC_DL_TYPE_DL : TOPIC_DL_TYPE_NORMAL;

  	$sql  = ($mode != "editpost") ? "INSERT INTO " . TOPICS_TABLE . " (topic_title, topic_poster, topic_time, forum_id, topic_status, topic_type, topic_dl_type, topic_vote, topic_show_first_post) VALUES ('$post_subject', " . $userdata['user_id'] . ", $current_time, $forum_id, " . TOPIC_UNLOCKED . ", $topic_type, $topic_dl_type, $topic_vote, $topic_show_first_post, $topic_show_first_post)" : "UPDATE " . TOPICS_TABLE . " SET topic_title = '$post_subject', topic_type = $topic_type, topic_show_first_post = '$topic_show_first_post', topic_dl_type = $topic_dl_type " . ((@$post_data['edit_vote'] || !empty($poll_title)) ? ", topic_vote = " . $topic_vote : "") . " WHERE topic_id = $topic_id";
		//bt end

		if (!$db->sql_query($sql))
		{
			message_die(GENERAL_ERROR, 'Error in posting', '', __LINE__, __FILE__, $sql);
		}

		if ($mode == 'newtopic')
		{
			$topic_id = $db->sql_nextid();
		}
	}
Что не так? Или надо приводить весь список модов?
Последний раз редактировалось Lektor 12.02.2007 18:28, всего редактировалось 1 раз.
Аватара пользователя
Siava
Поддержка
Поддержка
Сообщения: 5426
Стаж: 20 лет 3 месяца
Откуда: Питер
Благодарил (а): 177 раз
Поблагодарили: 749 раз

Сообщение Siava »

Создал тему с таким названием, описанием, содержимым без проблем и ошибок. В чём проблемы-то? Модов раз в 10 больше :)
Искать надо в functions_post.php
Еще одно нарушение правил и будете забанены. © Mr. Anderson
Ты очистил кеш? © Sheer
https://siava.ru (phpbb 2.0.x 3.5.x)
Аватара пользователя
/DiOs
Former team member
Сообщения: 1139
Стаж: 19 лет 2 месяца
Откуда: Ленинград

Сообщение /DiOs »

Lektor
Название темы крайне информативное :evil: Либо за сутки приводите в порядок, либо прямиком в мусор.
И если я неправ, пусть старшие товарищи поправят меня
Lektor
phpBB 2.0.7
Сообщения: 574
Стаж: 19 лет

Сообщение Lektor »

Блин .. думаю я рано создал эту ему.. не до конца разобрался ..
2/DiOs можно тема повисит до вечера .. Я переоформлю т.к. щас даже не готов риквест сфорс\мировать .. щас разберусь .. (скорее всего сам разберусь ..)
Аватара пользователя
vooon
Former team member
Сообщения: 55
Стаж: 18 лет 2 месяца

Сообщение vooon »

Ошибка возникает вот из-за этого:
VALUES ('<?=''ss''?>', 2, 1171241170, 2, 0, 0, 0, 0, 0, 0)

должно быть
'<?=\'\'ss\'\'?>'

хотя непонятно почему это не сделал форум.
возможно поможет это $post_subject (из второго листинга)
заменить на htmlspecialchars( $post_subject )
Аватара пользователя
Vladson
Former team member
Сообщения: 816
Стаж: 21 год
Откуда: Estonia, Tallinn

Сообщение Vladson »

vooon писал(а):htmlspecialchars
тут совершенно не нужен, он делает совершенно другие вещи, тут надо разбираться с тем какие моды испортили всю малину
Серый цвет - светлый (светлее чёрного), но он и тёмный (темнее белого), он же промежуточный (между чёрным и белым). Теорию относительности никто не отменял. Истина в целом - понятие виртуально-ситуативное.
Xpert
phpBB Guru
phpBB Guru
Сообщения: 5484
Стаж: 21 год 1 месяц
Поблагодарили: 2 раза

Сообщение Xpert »

Надо смотреть обработку параметра ф-ей str_replace(вроде есть таковая применительно к этому параметру).
Эксперт - это человек, который избегает мелких ошибок на пути к грандиозному провалу.
Любая более-менее сложная задача имеет несколько простых, изящных, лёгких для понимания неправильных решений
Аватара пользователя
crash
Former team member
Сообщения: 6517
Стаж: 20 лет 9 месяцев
Откуда: Бердск

Сообщение crash »

закрою спор ))))
Lektor писал(а):(topic_title, topic_poster, topic_time, forum_id, topic_status, topic_type, topic_dl_type, topic_vote, topic_show_first_post)
если я умею считать то тут 9 значений..
Lektor писал(а):('<?=''ss''?>', 2, 1171241170, 2, 0, 0, 0, 0, 0, 0)
а тут если не ошибаюсь 10.
Делайте выводы)))
Как правильно задавать вопросы
Для особо одаренных: поиск - это есть круто.
FAQ v.2 | FAQ v.3 | Шаблон запроса

Вернуться в «Поддержка phpBB 2.0.x»