php-resource



Zurück   PHP-Scripte PHP-Tutorials PHP-Jobs und vieles mehr > Entwicklung > PHP Developer Forum
 

Login

 
eingeloggt bleiben
star Jetzt registrieren   star Passwort vergessen
 

 

 


PHP Developer Forum Hier habt ihr die Möglichkeit, eure Skriptprobleme mit anderen Anwendern zu diskutieren. Seid so fair und beantwortet auch Fragen von anderen Anwendern. Dieses Forum ist sowohl für ANFÄNGER als auch für PHP-Profis! Fragen zu Laravel, YII oder anderen PHP-Frameworks.

Antwort
 
LinkBack (1) Themen-Optionen Thema bewerten
  1 links from elsewhere to this Post. Click to view. #1 (permalink)  
Alt 12-05-2013, 16:13
Truncate
 Registrierter Benutzer
Links : Onlinestatus : Truncate ist offline
Registriert seit: Nov 2005
Beiträge: 419
Truncate kann nur auf Besserung hoffen
Standard Include von Dateien sicher machen

Hallo,

bei nachfolgenden Beispiel kann man sicher mittels Nullbyte (?) etc. am Server/Scripten rumpfuschen:

PHP-Code:
<?

// Aufruf z.B. mittels index.php?impressum

$query=trim($_SERVER['QUERY_STRING']);

if(isset(
$query))

    {

        if(
is_file("data/".$query.".htm"))

            {
                
                include(
"data/".$query.".htm");
                
            }
            
            else
            
            {
                
                include(
"data/startseite.htm");
                
            }

    }

    else
        
    {
            
        include(
"data/startseite.htm");
            
    }

?>
Alternative wäre natürlich alle erlaubten Dateien in ein Whitelist-Array zu stecken.

Aber wie baut man obige Abfrage "sicher" um? Per RegEx z.B. nur a-z durchlassen?

Danke,
Truncate
Mit Zitat antworten
  #2 (permalink)  
Alt 12-05-2013, 16:49
Melewo
 Registrierter Benutzer
Links : Onlinestatus : Melewo ist offline
Registriert seit: Jan 2013
Beiträge: 365
Melewo befindet sich auf einem aufstrebenden Ast
Standard

Zitat:
Zitat von Truncate Beitrag anzeigen
Alternative wäre natürlich alle erlaubten Dateien in ein Whitelist-Array zu stecken.
Lässt sich vielleicht nicht verallgemeinern, doch ich speichere den Namen der Seite mit in einem Feld für den Datensatz der Seite ab. Somit wird nicht nur der Titel und Artikel aus der DB geholt, sondern auch der Name der Seite abgefragt. Wenn der Name der Seite in der Datenbank nicht existiert oder nicht mit der Id übereinstimmt, erfolgt eine Weiterleitung auf eine Übersichtsseite.
Mit Zitat antworten
  #3 (permalink)  
Alt 12-05-2013, 17:06
Truncate
 Registrierter Benutzer
Links : Onlinestatus : Truncate ist offline
Registriert seit: Nov 2005
Beiträge: 419
Truncate kann nur auf Besserung hoffen
Standard

So umständlich will ich es aber nicht machen, da die Seite lediglich 20 Seiten umfasst

Sprich die Abfrage nach der Existenz des Dateinamens soll bleiben, allerdings eben "sicher" werden.
Mit Zitat antworten
  #4 (permalink)  
Alt 12-05-2013, 17:37
Melewo
 Registrierter Benutzer
Links : Onlinestatus : Melewo ist offline
Registriert seit: Jan 2013
Beiträge: 365
Melewo befindet sich auf einem aufstrebenden Ast
Standard

Deshalb schrieb ich ja, es lässt sich nicht verallgemeinern. Allerdings ist es einfacher, als das was Du da vorhast, gerade dann, wenn die Seitenzahl begrenzt ist. In der DB genügt doch dann:

Code:
Id | Name_der_Seite | Titel | Text | Description

Erster Eintrag:
1 | schoenes-wetter-heute | Schönes Wetter heute | ...
Dazu ein Formular, dass diese Eingaben ermöglicht.

Aufgerufen mit /schoenes-wetter-heute-01.html und dazu eine einfache Regel in der htaccess und was Du da vorhast, das könntest Du Dir dann einsparen bei 20 Seiten. Doch das ist nur meine Ansicht, andere brauchen die nicht zu teilen, da es sich nicht verallgemeinern lässt.
Mit Zitat antworten
  #5 (permalink)  
Alt 12-05-2013, 17:42
Truncate
 Registrierter Benutzer
Links : Onlinestatus : Truncate ist offline
Registriert seit: Nov 2005
Beiträge: 419
Truncate kann nur auf Besserung hoffen
Standard

Argh... ich will weder eine DB-Lösung noch ein Array

Lediglich die aktuelle Prüfung soll "dicht" gemacht werden.
Mit Zitat antworten
  #6 (permalink)  
Alt 12-05-2013, 17:53
combie
 PHP Expert
Links : Onlinestatus : combie ist offline
Registriert seit: May 2006
Beiträge: 3.296
combie wird schon bald berühmt werden
Standard

Ein Array, mit 20 Einträgen und die Bude ist dicht!
Einfacher gehts nicht.
__________________
Wir werden alle sterben
Mit Zitat antworten
  #7 (permalink)  
Alt 12-05-2013, 17:56
Truncate
 Registrierter Benutzer
Links : Onlinestatus : Truncate ist offline
Registriert seit: Nov 2005
Beiträge: 419
Truncate kann nur auf Besserung hoffen
Standard

Wenn der User dann aber ne neue Datei anlegt, versagt das Script weil die Datei im Array nicht bekannt ist

Ich lass jz per RegEx einfach nur Buchstaben im Query zu, wird wohl ausreichen.
Mit Zitat antworten
  #8 (permalink)  
Alt 14-05-2013, 17:47
ezkimo
 Registrierter Benutzer
Links : Onlinestatus : ezkimo ist offline
Registriert seit: Apr 2005
Ort: Beckum / Westf.
Beiträge: 279
ezkimo befindet sich auf einem aufstrebenden Ast
ezkimo eine Nachricht über ICQ schicken
Standard

Zitat:
Zitat von Truncate Beitrag anzeigen
Wenn der User dann aber ne neue Datei anlegt, versagt das Script weil die Datei im Array nicht bekannt ist

Ich lass jz per RegEx einfach nur Buchstaben im Query zu, wird wohl ausreichen.
Wenn die aufzurufenden Dateien mit den Dateien in dem besagten Verzeichnis identisch sind, macht es vielleicht Sinn den Verzeichnissinhalt gegen den übergebenenen Parameter zu checken.

Ich würde ebenfalls das Array vorziehen ...
__________________
MM Newmedia | MeinBlog
Mit Zitat antworten
  #9 (permalink)  
Alt 14-05-2013, 18:30
Truncate
 Registrierter Benutzer
Links : Onlinestatus : Truncate ist offline
Registriert seit: Nov 2005
Beiträge: 419
Truncate kann nur auf Besserung hoffen
Standard

Zitat:
Zitat von ezkimo Beitrag anzeigen
Wenn die aufzurufenden Dateien mit den Dateien in dem besagten Verzeichnis identisch sind, macht es vielleicht Sinn den Verzeichnissinhalt gegen den übergebenenen Parameter zu checken.
Genau das macht doch mein Script (siehe 1. Posting).

Zitat:
Zitat von ezkimo Beitrag anzeigen
Ich würde ebenfalls das Array vorziehen ...
Ich nicht, weil es doppelt Arbeit macht.

Übrigens hatte ich konkret nach einer Lösung für ein Problem gefragt und gewisse Alternativen ausgeschlossen, trotzdem "empfiehlt" man mir diese trotzdem immer wieder
Mit Zitat antworten
  #10 (permalink)  
Alt 14-05-2013, 20:46
Melewo
 Registrierter Benutzer
Links : Onlinestatus : Melewo ist offline
Registriert seit: Jan 2013
Beiträge: 365
Melewo befindet sich auf einem aufstrebenden Ast
Standard

Zitat:
Zitat von Truncate Beitrag anzeigen
Ich nicht, weil es doppelt Arbeit macht.
Lieber zu Anfang doppelte Arbeit, als später die dreifache Arbeit und den Ärger dazu, falls Hacker eine Lücke finden.

Noch eine Alternative, ich sage mal so, die wurde einst (und zuweilen noch heute) nicht ganz grundlos für User-Generated-Content benutzt.

PHP-Code:
<?php
/*-------------------------------------------------------------
 Inhalt (von verzeichnis/test.htm):
 Nur ein [b]Kurztext[/b] zur [em]Probe[/em]!
 --------------------------------------------------------------
*/
$query "test";
$query preg_replace("/[^a-z0-9_-]/"""$query);

function 
nehmeBBC($tags) {

    
$tags preg_replace("/\[b\](.+?)\[\/b\]/""<b>$1</b>"$tags);
    
$tags preg_replace("/\[em\](.+?)\[\/em\]/""<em>$1</em>"$tags);

    return 
$tags;
}

if(isset(
$query)) {

    if(
is_file("verzeichnis/".$query.".htm")) {

        
$data file_get_contents("verzeichnis/".$query.".htm");
        
/* -- unerlaubte Benutzereingaben ausfiltern -- */
        
$data preg_replace("/[^a-zA-Z0-9.,:;!?+\/[\]_-]/"" "$data);
        
/* -- erlaubte Benutzereingaben verwenden ----- */
        
$data nehmeBBC($data);

        echo 
$data;
    }
    else {
         echo 
"Nein!\n";
    }
}
/*-------------------------------------------------------------
 Ausgabe Quelltext: Nur ein <b>Kurztext</b> zur <em>Probe</em>!
 --------------------------------------------------------------
*/
?>
Mit Zitat antworten
  #11 (permalink)  
Alt 14-05-2013, 20:54
Truncate
 Registrierter Benutzer
Links : Onlinestatus : Truncate ist offline
Registriert seit: Nov 2005
Beiträge: 419
Truncate kann nur auf Besserung hoffen
Standard

Zitat:
Zitat von Melewo Beitrag anzeigen
Noch eine Alternative, ich sage mal so, die wurde einst (und zuweilen noch heute) nicht ganz grundlos für User-Generated-Content benutzt.
Ähm... wieso postet Du jetzt mein (leicht geändertes) Script und schlägst es als "Alternative" vor?

Irgendwie liest hier keiner mehr wirklich mit...
Mit Zitat antworten
  #12 (permalink)  
Alt 14-05-2013, 21:02
Melewo
 Registrierter Benutzer
Links : Onlinestatus : Melewo ist offline
Registriert seit: Jan 2013
Beiträge: 365
Melewo befindet sich auf einem aufstrebenden Ast
Standard

Zitat:
Zitat von Truncate Beitrag anzeigen
Ähm... wieso postet Du jetzt mein (leicht geändertes) Script und schlägst es als "Alternative" vor?
Statt "leicht geändertes" würde es eher als eine etwas sichere Variante bezeichnen.
Mit Zitat antworten
  #13 (permalink)  
Alt 14-05-2013, 21:07
Truncate
 Registrierter Benutzer
Links : Onlinestatus : Truncate ist offline
Registriert seit: Nov 2005
Beiträge: 419
Truncate kann nur auf Besserung hoffen
Standard

Zitat:
Zitat von Melewo Beitrag anzeigen
Statt "leicht geändertes" würde es eher als eine etwas sichere Variante bezeichnen.
Du hast lediglich die von mir oben vorgeschlagene (und bereits umgesetzte) RegEx eingebaut.
Mit Zitat antworten
  #14 (permalink)  
Alt 14-05-2013, 21:22
Melewo
 Registrierter Benutzer
Links : Onlinestatus : Melewo ist offline
Registriert seit: Jan 2013
Beiträge: 365
Melewo befindet sich auf einem aufstrebenden Ast
Standard

Zitat:
Zitat von Truncate Beitrag anzeigen
Du hast lediglich die von mir oben vorgeschlagene (und bereits umgesetzte) RegEx eingebaut.
Da haben sich wohl noch ein paar Zeilen mehr verändert. Aber gut, Du möchtest Dich streiten, ich nicht, deshalb bin draußen.
Mit Zitat antworten
Antwort

Lesezeichen


LinkBacks (?)
LinkBack to this Thread: https://www.php-resource.de/forum/php-developer-forum/104007-include-von-dateien-sicher-machen.html
Erstellt von For Type Datum
Zockbuy.de die faire Rückwärtsauktion! - RSS Feed URL - Blog RSS Feed This thread Refback 13-05-2013 16:34

Aktive Benutzer in diesem Thema: 1 (Registrierte Benutzer: 0, Gäste: 1)
 

Ähnliche Themen
Thema Autor Forum Antworten Letzter Beitrag
Userdaten sicher machen RamonaS PHP Developer Forum 17 02-07-2009 00:10
include von dateien FragmasterIII PHP Developer Forum 3 02-08-2006 01:54
Webformulare sicher machen? em-!x PHP Developer Forum 4 02-08-2004 13:10
Bilder sicher machen flo221 HTML, JavaScript, AJAX, jQuery, CSS, Bootstrap, LESS 12 27-07-2004 13:55
Formular sicher machen zzet PHP Developer Forum 10 07-04-2003 14:32

Themen-Optionen
Thema bewerten
Thema bewerten:

Forumregeln
Es ist Ihnen nicht erlaubt, neue Themen zu verfassen.
Es ist Ihnen nicht erlaubt, auf Beiträge zu antworten.
Es ist Ihnen nicht erlaubt, Anhänge hochzuladen.
Es ist Ihnen nicht erlaubt, Ihre Beiträge zu bearbeiten.

BB-Code ist an.
Smileys sind an.
[IMG] Code ist an.
HTML-Code ist aus.
Trackbacks are an
Pingbacks are an
Refbacks are an


PHP News

ebiz-trader 7.5.0 mit PHP7 Unterstützung veröffentlicht
ebiz-trader 7.5.0 mit PHP7 Unterstützung veröffentlichtDie bekannte Marktplatzsoftware ebiz-trader ist in der Version 7.5.0 veröffentlicht worden.

28.05.2018 | Berni

Wissensbestand in Unternehmen
Wissensbestand in UnternehmenLebenslanges Lernen und Weiterbilden sichert Wissensbestand in Unternehmen

25.05.2018 | Berni


 

Aktuelle PHP Scripte

ADSMAN V3 - Werbe-Manager ansehen ADSMAN V3 - Werbe-Manager

ADSMAN V3 - mehr als nur ein Bannermanager! Banner, Textanzeigen und PagePeel Manager! Mit ADSMAN PRO haben Sie die Marketinglösung für eine effektive und effiziente Werbeschaltung mit messbaren Ergebnissen. Unterstützt werden Bannerformate in beliebi

25.10.2018 virtualsystem | Kategorie: PHP/ Bannerverwaltung
PHP News und Artikel Script V2

News schreiben, verwalten, veröffentlichen. Dies ist jetzt mit dem neuen PHP News & Artikel System von virtualsystem.de noch einfacher. Die integrierte Multi-User-Funktion und der WYSIWYG-Editor (MS-Office ähnliche Bedienung) ermöglichen...

25.10.2018 virtualsystem | Kategorie: PHP/ News
Top-Side Guestbook

Gästebuch auf Textbasis (kein MySQL nötig) mit Smilies, Ip Sperre (Zeit selbst einstellbar), Spamschutz, Captcha (Code-Eingabe), BB-Code, Hitcounter, Löschfunktion, Editierfunktion, Kommentarfunktion, Kürzung langer Wörter, Seiten- bzw. Blätterfunktion, V

22.10.2018 webmaster10 | Kategorie: PHP/ Gaestebuch
 Alle PHP Scripte anzeigen

Alle Zeitangaben in WEZ +2. Es ist jetzt 07:37 Uhr.