Bug report: изменение 255 на * в IP забаненых пользователей

[serge35]

1) Версия PHPBB 2.0.19
2) Суть проблемы: если Вы в администраторской панели пытаетесь забанить пользователя по IP и в одном из октетов IP адреса или диапазона IP-адресов встречается 255, то после нажатия кнопки "отправить" и возвращения на эту страницу, в списке ip-адресов забаненых пользователей вместо 255 появляется символ "*"

[crash]

serge35
пример айпи

Добавлено спустя 3 минуты 50 секунд:

хотя по идее оно правильно чситает наверное)

[serge35]

serge35
пример айпи

Добавлено спустя 3 минуты 50 секунд:

хотя по идее оно правильно чситает наверное)

82.255.113.154 превращается в 82.*.113.154

[crash]

видимо phpBB решает что если указано 255, то использоват всю сеть. надо смотреть исходники

[Balamut]

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

				$ip_list[] = encode_ip(str_replace('*', '255', trim($ip_list_temp[$i])));

Добавлено спустя 47 секунд:

В admin_user_ban.php

Добавлено спустя 8 минут 51 секунду:

Ага, поглядел, ещё и

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

			$ban_ip = str_replace('257', '*', decode_ip($banlist[$i]['ban_ip']));

[serge35]

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

				$ip_list[] = encode_ip(str_replace('*', '255', trim($ip_list_temp[$i])));

Добавлено спустя 47 секунд:

В admin_user_ban.php

Добавлено спустя 8 минут 51 секунду:

Ага, поглядел, ещё и

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

			$ban_ip = str_replace('257', '*', decode_ip($banlist[$i]['ban_ip']));

А в чём логика, если "на пальцах" для идиотов объяснять?
Если я баню пользователя с ip 82.255.113.154 и этот ip преобразуется в 82.*.113.154,
то реально банятся все ip из диапазона 82.[0-255].113.154?

[Siava]

Вы когда-нибудь видели IP адрес с 255?

[serge35]

Вы когда-нибудь видели IP адрес с 255?

Видели (см выше)
правда, я в первом октете ошибся не 82.255.113.154, а 85.255.113.154
Сети такие бывают и это факт, с которым нам придётся жить дальше :wink: .
В подтверждение своих слов привожу выдержку из whois для этой сети:

inetnum: 85.255.112.0 - 85.255.127.255
netname: inhoster
descr: Inhoster hosting company
descr: OOO Inhoster, Poltavskij Shliax 24, Kharkiv, 61000, Ukraine
country: UA
org: ORG-EST1-RIPE
admin-c: AK4026-RIPE
tech-c: AK4026-RIPE
tech-c: FWHS1-RIPE
status: ASSIGNED PI
mnt-by: RIPE-NCC-HM-PI-MNT
mnt-lower: RIPE-NCC-HM-PI-MNT
mnt-by: RECIT-MNT
mnt-routes: RECIT-MNT
mnt-domains: RECIT-MNT
mnt-by: DAV-MNT
mnt-routes: DAV-MNT
mnt-domains: DAV-MNT
source: RIPE # Filtered

Вот такие пироги с котятами. Их едят, а они - глядят

Что с багой делать будем, господа?

[Balamut]

Замените в указанных строчках 255 на 256, например.
Тогда замены не будет.

Добавлено спустя 1 минуту 37 секунд:

Ага, поглядел, ещё и

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

			$ban_ip = str_replace('257', '*', decode_ip($banlist[$i]['ban_ip']));

Естественно, там было 255.
Это я менял и смотрел, будет ли заменяться 255.

[serge35]

Замените в указанных строчках 255 на 256, например.
Тогда замены не будет.

Добавлено спустя 1 минуту 37 секунд:

Ага, поглядел, ещё и

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

			$ban_ip = str_replace('257', '*', decode_ip($banlist[$i]['ban_ip']));

Естественно, там было 255.
Это я менял и смотрел, будет ли заменяться 255.

Понял. Большое спасибо. Буду исправлять. Может имеет смысл эту информацию до разработчиков довести, чтобы они в новых версиях пофиксили.

[Balamut]

Ввобще-то я так и не понял разработчиков phpBB, для каких целей они это сделали...

Может кто-нибудь прояснит ситуацию?

[crash]

Smayliks
походу они рещили, что 255 будут указывать в адресе хостов, а не сети. а в хосте 255 быть не магет, по крайней мере сети класса С, вот и сделали.
а мож просто не подумали

[Balamut]

crash, да но форум может быть и в локальной сети - там вообще могут быть какие угодно ip'шники.
Видимо правда не подумали.

[crash]

Smayliks
могут, но в основном сети локальные все равно бьются (ну если они большие) на подсети и в основном класса С, а в классе С смотреть выше)))

[Balamut]

но в основном

ключевая фраза. =)

Точно сказать не могу (не помню), но кажется у меня на работе на серваке я ставил IP'шник 10.10.10.255