Probleme mit IP Sperre bei Tresorknackerscript

Diese Seite verwendet Cookies. Durch die Nutzung unserer Seite erklären Sie sich damit einverstanden, dass wir Cookies setzen. Weitere Informationen

  • Probleme mit IP Sperre bei Tresorknackerscript

    Also ich habe aus Spaß mal so nen kleines Tresorknacker Script geschrieben!
    Unter anderem gibts da auch eine IP Sperre (24 Stunden) welche gesetzt wird wenn ein User versucht hat denn Tresor zu knacken es aber nicht geschaft hat oder es geschaft hat!

    Ich habe deshalb einen Ordner mit Namen ips in diesen wird nach dem die Funktion write_ipsperre(); aufgerufen wurde eine Datei mit Namen md5(IP).txt angelegt, was auch funktionier!

    Dann habe ich noch eine Funktion check_ipsperre() die bei vorhandensein der Ip Datei true sonst false zurück gibt! Das geht auch...

    Jetzt kommt das Problem!

    Ich hab ne schöne Abfrage gemacht die im Falle des Falles den User hinweist das er schon mitgemacht aber nicht gewonnen hat & das Eingabeformular sperrt!

    Hier:

    Quellcode

    1. <?php
    2. if(check_ipsperre() == true){ $message = "4"; }
    3. ?>


    Sollte doch eigendlich gehen oder? :shock:

  • Inhalt der Datei ip_sperre.php

    Quellcode

    1. <?php
    2. function check_ipsperre(){
    3. if(file_exists("./ips/".md5($_SERVER['REMOTE_ADDR']).".txt")){ return true; } else { return false; }
    4. }
    5. function write_ipsperre(){
    6. $fp = fopen("./ips/".md5($_SERVER['REMOTE_ADDR']).".txt", "a");
    7. fclose($fp);
    8. }
    9. function clear_ipsperre(){
    10. $ipfolder = "./ips/";
    11. // Dateien älter als n Minuten werden gelöscht
    12. $handle = @opendir($ipfolder);
    13. while ($ipfile = @readdir($handle)) {
    14. $ipdate = @filectime($ipfolder.$ipfile);
    15. if ($ipdate < (@time() - (60*$ip_sperrzeit))) {
    16. @unlink($ipfolder.$ipfile);
    17. }
    18. }
    19. @closedir($handle);
    20. }
    21. ?>
    Alles anzeigen


    Die Message halt so:

    Quellcode

    1. <?php
    2. if($message == "4"){
    3. ?>
    4. <div class="text" style="height:30px; width:450px; background-color:#FF1F00; border:#D41F00 solid 1px;"><font color="
    5. ">Du hast heute schonmal versucht unseren Tresor zu knacken! Versuche es morgen nochmal!</font></div>
    6. <?php
    7. }
    8. ?>

  • $ip_sperrzeit steht in der config.inc.php die includiert wird. Die @ hatte ich alle schon weg, hat aber nichts gebracht!
    Ich mach aus dem - mal ebend ein +!

    Code index.php:

    Quellcode

    1. <?php
    2. // Einstellungen
    3. $code = "05641"; // Der code der genackt werden soll
    4. $weiterleitung = "win.php"; // Da soll der Tresorknacker hin
    5. $ip_sperrzeit = 1500; // 24 Stunden
    6. // Funktion ip_sperre includen
    7. include_once 'ip_sperre.php';
    8. // IP Sperre säubern
    9. clear_ipsperre();
    10. // Ab hier bitte nichts mehr ändern
    11. if($code == ""){ $message = "1"; } // Es wird eine Meldung ausgegeben wenn kein Tresorcode vorgeben wurde
    12. if(check_ipsperre() == true){ $message = "4"; } // Wenn IP Sperre aktiv dann Meldung ausgeben!
    13. if($_POST['Submit'] == true AND check_ipsperre() != true AND $_POST['code'] != ""){
    14. write_ipsperre();
    16. if($_POST["code"] == ""){
    17. $message = "2";
    18. }
    19. elseif($_POST["code"] != $code){
    20. // IP Sperre
    21. write_ipsperre();
    22. $message = "3";
    23. }
    24. elseif($_POST["code"] == $code){
    25. // IP Sperre
    26. write_ipsperre();
    27. echo "Gewonnen";
    28. }
    29. }
    30. ?>
    31. <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
    32. <html xmlns="http://www.w3.org/1999/xhtml">
    33. <head>
    34. <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
    35. <title>Tresorknacker Script</title>
    36. <style type="text/css">
    37. <!--
    38. .ueberschrift {
    39. font-size: 12px;
    40. font-weight: bold;
    41. font-family: Verdana, Arial, Helvetica, sans-serif;
    42. }
    43. .text {
    44. font-size: 12px;
    45. font-family: Verdana, Arial, Helvetica, sans-serif;
    46. }
    47. -->
    48. </style>
    49. </head>
    51. <body>
    52. <div align="center"><img src="img/logo.jpg" alt="Logo" width="320" height="216" /></div>
    53. <br /><br />
    54. <div align="center"><br />
    55. <?php
    56. if($message == "1"){
    57. ?>
    58. <div class="text" style="height:15px; width:450px; background-color:#FF1F00; border:#D41F00 solid 1px;"><font color="
    59. ">Es wurde noch kein Tresorcode eingestellt!</font></div>
    60. <?php
    61. }
    62. ?>
    63. <?php
    64. if($message == "2"){
    65. ?>
    66. <div class="text" style="height:15px; width:450px; background-color:#FF1F00; border:#D41F00 solid 1px;"><font color="
    67. ">Du hast keinen Code eingegeben!</font></div>
    68. <?php
    69. }
    70. ?>
    71. <?php
    72. if($message == "3"){
    73. ?>
    74. <div class="text" style="height:15px; width:450px; background-color:#FF1F00; border:#D41F00 solid 1px;"><font color="
    75. ">Schade. Du hast denn Tresor leider nicht geknackt!</font></div>
    76. <?php
    77. }
    78. ?>
    79. <?php
    80. if($message == "4"){
    81. ?>
    82. <div class="text" style="height:30px; width:450px; background-color:#FF1F00; border:#D41F00 solid 1px;"><font color="
    83. ">Du hast heute schonmal versucht unseren Tresor zu knacken! Versuche es morgen nochmal!</font></div>
    84. <?php
    85. }
    86. ?>
    87. <br />
    88. <form id="tresor" name="tresor" method="post" action="index.php" class="text">
    89. <label>Hier Code eingeben:
    90. <input name="code" type="text" id="code" <?php if($message == "4") echo "disabled=\"disabled\""; ?>/>
    91. </label>
    92. <label>
    93. <input type="submit" name="Submit" value="Tresor knacken" <?php if($message == "4") echo "disabled=\"disabled\""; ?> />
    94. </label>
    95. </form>
    96. </div>
    97. </body>
    98. </html>
    Alles anzeigen


    Ich weiß nicht einen kurzen Moment lief es aber dann auf einmal nicht mehr!!! :?

  • bist du dir sicher, dass die bedingungen aus 12 bis 14 nicht gleichzeitig passieren?
    Sollten sie zumindest nicht, oder? Arbeite besser mal mit elseifs.
    Und bau dir zur Fehlersuche überall Ausgaben ein, damit du weißt in welcher if du dich befindest.

    Ansonsten hast du innerhalb von Funktionen keinen Zugriff auf externe Variablen.
    Es sei denn du importierst sie mit global.

    Quellcode

    1. function clear_ipsperre(){
    2. global $ip_sperrzeit;

  • Also http://tinyurl.com/yq3o9t sagt das die IP Datei existiert...
    http://tinyurl.com/ytwctl aber sagt Formular wurde gesendet, IP Sperre nicht gesetzt!

    Komisch!

    Code jetzt:

    ip_sperre.php

    Quellcode

    1. <?php
    2. function check_ipsperre(){
    3. if(file_exists("./ips/".md5($_SERVER['REMOTE_ADDR']).".txt")){ return true; } else { return false; }
    4. }
    5. function write_ipsperre(){
    6. $fp = fopen("./ips/".md5($_SERVER['REMOTE_ADDR']).".txt", "a");
    7. fclose($fp);
    8. }
    9. function clear_ipsperre(){
    10. global $ip_sperrzeit;
    11. $ipfolder = "./ips/";
    12. // Dateien älter als n Minuten werden gelöscht
    13. $handle = opendir($ipfolder);
    14. while ($ipfile = readdir($handle)) {
    15. $ipdate = filectime($ipfolder.$ipfile);
    16. if ($ipdate < (time() + (60*$ip_sperrzeit)) AND $ipfile != "." AND $ipfile != "..") {
    17. unlink($ipfolder.$ipfile);
    18. }
    19. }
    20. closedir($handle);
    21. }
    22. ?>
    Alles anzeigen


    index.php

    Quellcode

    1. <?php
    2. // Einstellungen
    3. $code = "05641"; // Der code der genackt werden soll
    4. $weiterleitung = "win.php"; // Da soll der Tresorknacker hin
    5. $ip_sperrzeit = 1500; // 24 Stunden
    6. // Funktion ip_sperre includen
    7. include_once 'ip_sperre.php';
    8. // IP Sperre säubern
    9. clear_ipsperre();
    10. // Ab hier bitte nichts mehr ändern
    11. if($code == ""){ $message = "1"; } // Es wird eine Meldung ausgegeben wenn kein Tresorcode vorgeben wurde
    12. elseif(check_ipsperre() == true){ $message = "4"; echo "<br />Ip sperre schon gesetzt"; } // Wenn IP Sperre aktiv dann Meldung ausgeben!
    13. elseif($_POST['Submit'] == true AND check_ipsperre() != true AND $_POST['code'] != ""){
    14. echo "<br /><font color=\"red\">Formular wurde gesendet, Ip Sperre ist nich gesetzt & das Input Feld ist nicht leer</font><br />";
    15. write_ipsperre();
    17. if($_POST["code"] == ""){
    18. $message = "2";
    19. echo "<br /><font color=\"red\">Kein Code angegeben</font>";
    20. }
    21. elseif($_POST["code"] != $code){
    22. // IP Sperre
    23. write_ipsperre();
    24. echo "<br /><font color=\"red\">Falscher Code</font><br />";
    25. $message = "3";
    26. }
    27. elseif($_POST["code"] == $code){
    28. // IP Sperre
    29. write_ipsperre();
    30. echo "<br />Gewonnen";
    31. }
    32. }
    33. ?>
    34. <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
    35. <html xmlns="http://www.w3.org/1999/xhtml">
    36. <head>
    37. <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
    38. <title>Tresorknacker Script</title>
    39. <style type="text/css">
    40. <!--
    41. .ueberschrift {
    42. font-size: 12px;
    43. font-weight: bold;
    44. font-family: Verdana, Arial, Helvetica, sans-serif;
    45. }
    46. .text {
    47. font-size: 12px;
    48. font-family: Verdana, Arial, Helvetica, sans-serif;
    49. }
    50. -->
    51. </style>
    52. </head>
    54. <body>
    55. <div align="center"><img src="img/logo.jpg" alt="Logo" width="320" height="216" /></div>
    56. <br /><br />
    57. <div align="center"><br />
    58. <?php
    59. if($message == "1"){
    60. ?>
    61. <div class="text" style="height:15px; width:450px; background-color:#FF1F00; border:#D41F00 solid 1px;"><font color="
    62. ">Es wurde noch kein Tresorcode eingestellt!</font></div>
    63. <?php
    64. }
    65. ?>
    66. <?php
    67. if($message == "2"){
    68. ?>
    69. <div class="text" style="height:15px; width:450px; background-color:#FF1F00; border:#D41F00 solid 1px;"><font color="
    70. ">Du hast keinen Code eingegeben!</font></div>
    71. <?php
    72. }
    73. ?>
    74. <?php
    75. if($message == "3"){
    76. ?>
    77. <div class="text" style="height:15px; width:450px; background-color:#FF1F00; border:#D41F00 solid 1px;"><font color="
    78. ">Schade. Du hast denn Tresor leider nicht geknackt!</font></div>
    79. <?php
    80. }
    81. ?>
    82. <?php
    83. if($message == "4"){
    84. ?>
    85. <div class="text" style="height:30px; width:450px; background-color:#FF1F00; border:#D41F00 solid 1px;"><font color="
    86. ">Du hast heute schonmal versucht unseren Tresor zu knacken! Versuche es morgen nochmal!</font></div>
    87. <?php
    88. }
    89. ?>
    90. <br />
    91. <form id="tresor" name="tresor" method="post" action="index.php" class="text">
    92. <label>Hier Code eingeben:
    93. <input name="code" type="text" id="code" <?php if($message == "4") echo "disabled=\"disabled\""; ?>/>
    94. </label>
    95. <label>
    96. <input type="submit" name="Submit" value="Tresor knacken" <?php if($message == "4") echo "disabled=\"disabled\""; ?> />
    97. </label>
    98. </form>
    99. </div>
    100. </body>
    101. </html>
    Alles anzeigen


    Hm langsam hab ich auch kein Plan mehr^^...

  • also wie gesagt.. bau dir ein paar Ausgaben ein. Wenn ich verzweifelt nach einem Fehler suchen würde, würde ich meine Datei so aufbauen:

    Quellcode

    1. $limit = (time() + (60*$ip_sperrzeit));
    2. echo 'es werden alle eintrage vor '.date('d.m.Y H:i:s', $limit).' geloescht<br />';
    3. while ($ipfile = readdir($handle)) {
    4. $ipdate = filectime($ipfolder.$ipfile);
    5. if ($ipdate < $limit AND $ipfile != "." AND $ipfile != "..") {
    6. unlink($ipfolder.$ipfile);
    7. echo "$ipfile geloescht<br />";
    8. } else {
    9. echo "$ipfile nicht geloescht<br />";
    10. }
    11. }
    Alles anzeigen


    Dein Code ist außerdem ziemlich unübersichtlich - so habe mir noch nicht die Mühe gemacht alles anzuschauen. Zumal das nicht alles ist, oder? Zumindest hast du oben von einer config geredet die eingebunden wird.