[Erledigt] Ip Sperre funktioniert bei einer 2 Ip nicht mehr

Erdbeer · 09.12.2011, 11:57

Ich habe folgendes Problem, meine Ip Sperre funktioniert nicht mehr, wenn ich mehr als 1 eine Ip in die ipbans.data eintrage warum?

index.php

PHP-Code:

  // IP Validieren

function isBanned( $ipAdressString, $banlistFile = 'doc/ipbans.data', $entrySep = "\r\n" ) {

   $inboundIPArray = explode('.', trim($ipAdressString));

 
   if ( is_readable($banlistFile) ) {

      // load file

      $banlist = file_get_contents($banlistFile);

 
      // create array of contents

      $banlistArray = explode($entrySep, $banlist);

 
      // filter IPs, exclude corrupt definitions from banlist

      foreach ( $banlistArray as $key => &$entry ) {

         $entry = explode('.', trim($entry));

         if ( count($entry) != 4 ) unset($banlistArray[$key]);

      }

 
      // process current ip

      foreach ( $banlistArray as $item ) {

         $current = array();

         foreach ( $item as $boundPos => $ipToken ) {

            if ( (int)$ipToken == (int)$inboundIPArray[$boundPos] || $ipToken == '*' )

               $current[$boundPos] = 1;

            else $current[$boundPos] = 0;

         }

         if ( array_sum($current) == 4 ) return true;

      }

   }

   return false;

}

 
if ( isBanned( $currentAddress ) ) echo 'Deine IP-Adresse wurde auf unserem System gesperrt!';

else 'Willkommen :)';

ipbans.data

PHP-Code:

  178.8.44.153

178.8.*.*

91.9.65.205

91.9.*.*

lstegelitz · 09.12.2011, 14:21

Zitat:

PHP-Code:

  // IP Validieren
function isBanned( $ipAdressString, $banlistFile = 'doc/ipbans.data', $entrySep = "\r\n" ) {
   $inboundIPArray = explode('.', trim($ipAdressString));

   if ( is_readable($banlistFile) ) {
      // load file
      $banlist = file_get_contents($banlistFile);

      // create array of contents
      $banlistArray = explode($entrySep, $banlist);

Ins Blaue geraten: Deine Bannlisten-Datei enthält nicht \r\n als Zeilentrenner (sondern nur \n).

PHP-Code:

        $banlist = file_get_contents($banlistFile);

      // create array of contents
      $banlistArray = explode($entrySep, $banlist);

kann man zusammenfassen:

PHP-Code:

  $banlistArray = file($banlistFile);

Wolla · 09.12.2011, 20:56

file() nimmt die Zeilenschaltungen mit in den Array. Man sollte das mit dem zweiten Parameter unterdrücken:

$banlistArray = file($banlistFile,FILE_IGNORE_NEW_LINES);

ChrisvA · 09.12.2011, 20:59

Was vielleicht auch mal interessant wäre, ist die Ausgabe von print_r($banlistArray);
Dann sieht man ja wo es hakt.

Erdbeer · 14.12.2011, 20:54

\r habe ich herausgenommen scheint nun zufunktionieren vielen Dank.

Arne Drews · 14.12.2011, 22:13

Du kommst mit einer foreach aus, wenn Du's anders angehst:

PHP-Code:

  function isBanned( $ipAdressString, $banlistFile = 'doc/ipbans.data' ) {

   $inboundIPArray = explode( '.', trim($ipAdressString) );

 
   if( is_readable($banlistFile) ) {

      // get valid ip addresses from file

      $banlistArray = preg_grep( '/^((\d{1,3}|\*)\.){3}(\d{1,3}|\*)$/', file($banlistFile, FILE_IGNORE_NEW_LINES) );

 
      // process current ip

      foreach( $banlistArray as $item ) {

        // check for masked octets

        $ipOctets = explode( '.', $item );

        $ipMasked = (int)array_search( '*', $ipOctets );

 
        // slicing unmasked octets

        $ipOctets = array_slice( $ipOctets, 0, count($ipOctets) - $ipMasked );

        $ipCurrent = array_slice( $inboundIPArray, 0, count($ipOctets) );

 
        // filter

        if( 0 == count(array_diff($ipCurrent, $ipOctets)) ) return true;

      }

   }

   return false;

}

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
[Erledigt] Werte von Datei A an Funktion von Datei B funktioniert nicht	Susi25	PHP Einsteiger	8	20.08.2011 15:47
.htaccess funktioniert nicht im Server?	Aceh	PHP Einsteiger	2	21.05.2011 15:42
jQuery Autocomplete AutoFill funktioniert nicht	Atikos	JavaScript, Ajax und mehr	15	12.04.2011 16:16
unlink() funktioniert nicht	Chacky	PHP Einsteiger	6	17.02.2011 16:22
[Erledigt] str_replace funktioniert nur lokal	Cogax	PHP Tipps 2010	19	27.10.2010 21:28
IP Sperre bei Star Rating System	UNIKAT	JavaScript, Ajax und mehr	6	24.10.2010 16:34
Code funktioniert plötzlich nicht mehr nach Serverumzug	Hawk	PHP Tipps 2010	4	18.08.2010 08:12
Verschlüsselung funktioniert nicht richtig	netmario	PHP Tipps 2010	2	19.05.2010 22:57
[Erledigt] Voting funktion mit user sperre NICHT ip sperre	Naver	PHP Tipps 2010	15	15.05.2010 21:07
mod_rewrite mit slash/ohne slash funktioniert nur teilweise	taurus	Server, Hosting und Workstations	4	13.09.2009 12:57
[Erledigt] IP sperre funktioniert nicht richtig	w35T3n	PHP Tipps 2009	9	02.08.2009 10:42
[ERLEDIGT]PHP while-Schleife für Ausgabe funktioniert plötzlich nicht	fuckinghot19	PHP Tipps 2009	6	05.06.2009 22:28
[Erledigt] Umlaute ersetzen funktioniert nicht richtig	BlackSpirit	PHP Tipps 2009	19	20.04.2009 12:04
send button funktioniert nicht, finde fehler nicht	selma123	PHP Tipps 2008	3	17.09.2008 11:57
Dateien erstellen oder schreiben funktioniert nicht im Web		PHP Tipps 2004-2	1	05.12.2004 19:39

09.12.2011, 11:57
Erdbeer Benutzer Registriert seit: 04.10.2011 Beiträge: 48 PHP-Kenntnisse: Anfänger	[Erledigt] Ip Sperre funktioniert bei einer 2 Ip nicht mehr Ich habe folgendes Problem, meine Ip Sperre funktioniert nicht mehr, wenn ich mehr als 1 eine Ip in die ipbans.data eintrage warum? index.php PHP-Code: // IP Validieren function isBanned( $ipAdressString, $banlistFile = 'doc/ipbans.data', $entrySep = "\r\n" ) { $inboundIPArray = explode('.', trim($ipAdressString)); if ( is_readable($banlistFile) ) { // load file $banlist = file_get_contents($banlistFile); // create array of contents $banlistArray = explode($entrySep, $banlist); // filter IPs, exclude corrupt definitions from banlist foreach ( $banlistArray as $key => &$entry ) { $entry = explode('.', trim($entry)); if ( count($entry) != 4 ) unset($banlistArray[$key]); } // process current ip foreach ( $banlistArray as $item ) { $current = array(); foreach ( $item as $boundPos => $ipToken ) { if ( (int)$ipToken == (int)$inboundIPArray[$boundPos] \|\| $ipToken == '' ) $current[$boundPos] = 1; else $current[$boundPos] = 0; } if ( array_sum($current) == 4 ) return true; } } return false; } if ( isBanned( $currentAddress ) ) echo 'Deine IP-Adresse wurde auf unserem System gesperrt!'; else 'Willkommen :)'; ipbans.data PHP-Code: `178.8.44.153 178.8..* 91.9.65.205 91.9..`


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

09.12.2011, 20:56
Wolla Moderator Registriert seit: 06.06.2008 Beiträge: 4.945 PHP-Kenntnisse: Fortgeschritten	file() nimmt die Zeilenschaltungen mit in den Array. Man sollte das mit dem zweiten Parameter unterdrücken: $banlistArray = file($banlistFile,FILE_IGNORE_NEW_LINES); __________________ Warum denkt mein Hund eigentlich immer dann, wenn es an der Tür klingelt, es sei für ihn?

09.12.2011, 20:59
ChrisvA Erfahrener Benutzer Registriert seit: 10.01.2010 Beiträge: 386 PHP-Kenntnisse: Fortgeschritten	Was vielleicht auch mal interessant wäre, ist die Ausgabe von print_r($banlistArray); Dann sieht man ja wo es hakt.

14.12.2011, 20:54
Erdbeer Benutzer Registriert seit: 04.10.2011 Beiträge: 48 PHP-Kenntnisse: Anfänger	\r habe ich herausgenommen scheint nun zufunktionieren vielen Dank.

14.12.2011, 22:13
Arne Drews Erfahrener Benutzer Registriert seit: 22.04.2009 Beiträge: 3.486 PHP-Kenntnisse: Anfänger	Du kommst mit einer foreach aus, wenn Du's anders angehst: PHP-Code: function isBanned( $ipAdressString, $banlistFile = 'doc/ipbans.data' ) { $inboundIPArray = explode( '.', trim($ipAdressString) ); if( is_readable($banlistFile) ) { // get valid ip addresses from file $banlistArray = preg_grep( '/^((\d{1,3}\|\)\.){3}(\d{1,3}\|\)$/', file($banlistFile, FILE_IGNORE_NEW_LINES) ); // process current ip foreach( $banlistArray as $item ) { // check for masked octets $ipOctets = explode( '.', $item ); $ipMasked = (int)array_search( '', $ipOctets ); // slicing unmasked octets $ipOctets = array_slice( $ipOctets, 0, count($ipOctets) - $ipMasked ); $ipCurrent = array_slice( $inboundIPArray, 0, count($ipOctets) ); // filter if( 0 == count(array_diff($ipCurrent, $ipOctets)) ) return true; } } return false; } __________________ Phantasie ist wichtiger als Wissen, denn Wissen ist begrenzt.* Probleme kann man niemals mit derselben Denkweise lösen, durch die sie entstanden sind. Albert Einstein