Nodeny не проверяет уникальность ip-адресов!

[elite]

Попробуйте создать клиента с ип 10.10.10.5, а потом с ип 10.10.10.005

[Efendy]

Попробуйте создать клиента с ип 10.10.10.5, а потом с ип 10.10.10.005

в данном случае баг не с тем, что это одинаковые ip, а с тем, что 2й ip по сути некорректен (не вообще, а в пределах NoDeny ибо ip воспринимаются/сравниваются как строки). Щас посмотрю что нужно подправить

[Efendy]

быстрее всего в голову пришло такое:

в user.pl в

Код:

sub check_ip
{
    $F{ip} =~ s/\.0+([^0\.]+)/\.$1/g;

щас тока протещу, а то до этого ахинею написал)

[elite]

так, а при создании нового пользователя, проверка будет работать или надо еще где-то ее вставить?

[Efendy]

Придумал аццкий регексп:

Код:

s/(^|\.)0+([^0\.]+|\d$)/$1$2/g;

Проверка:

Код:

foreach my $ip qw(
        0.0.0.0
        10.0.0.1
        0010.1.2.30
        020.000030.04.05
        1.2.3.4
        123.0.234.000
){
        $_ = $ip;
        s/(^|\.)0+([^0\.]+|\d$)/$1$2/g;
        print "$ip -> $_\n";
}

выдает:

0.0.0.0 -> 0.0.0.0
10.0.0.1 -> 10.0.0.1
0010.1.2.30 -> 10.1.2.30
020.000030.04.05 -> 20.30.4.5
1.2.3.4 -> 1.2.3.4
123.0.234.000 -> 123.0.234.0

Насчет создания клиента - сейчас гляну

[Efendy]

Итого в user.pl после

Код:

sub check_ip
{

ставим:

Код:

  $F{ip} =~ s/(^|\.)0+([^0\.]+|\d$)/$1$2/g;

В adduser.pl после

Код:

$Fip=&trim($F{ip});

ставим:

Код:

  $Fip =~ s/(^|\.)0+([^0\.]+|\d$)/$1$2/g;

[Efendy]

На самом деле покрываются не все возможные варианты извращений. Более правильный вариант:

Код:

$F{ip} = join '.', map{ s/^0+(\d+)$/$1/; $_ } split /\./,$F{ip};

и

Код:

$Fip = join '.', map{ s/^0+(\d+)$/$1/; $_ } split /\./,$Fip;

[0xbad0c0d3]

А найух регексп!?

Код:

$F{ip} = join '.', map{ int } split /\./,$F{ip};

[Efendy]

А найух регексп!?

Код:

$F{ip} = join '.', map{ int } split /\./,$F{ip};

предполагал, что число с лидирующим нулем будет воспринято как восьмеричное:

Код:

print int(010);

выдаст 8.
Но оказывается, что в случае со строками, perl поступает иначе. Так что твой вариант лучше

Тогда проще так:

Код:

s/(\d+)/int $1/ge;