jak zabezpieczyc ten skrypt zeby tylko mozna bylo podawac ip a nie ktos poda

122.122.122.122;ls -la

i ma wylistowane pliki :/

Stosuj wyrażenia regularne i odpowiednie regułki.

Przeczytaj sobie to.

A wiesz co to są wyrażenia regularne ? Jeśli nie to przede wszystkim poczytaj o tym w google, bo to jest zagadnienie pojawiające się w praktycznie każdym języku programowania, a regułki wyglądają tak samo. Przykładem wyrażeń regularnych w PHP jest preg_ match" title="Zobacz w manualu PHP" target="_manual. Cała masa informacji znajduje się w google i manualu.

EDIT: @webdicepl, ubiegłeś mnie o kilka sekund

A po co Tobie to przy sprawdzeniu IP ? wystarczą reguły [0-9] i \. a jak masz tego użyć to sam pomyśl.

a Co ma REQUEST_URI do adresu IP?

Z takim podejściem do nauki proponuje Ci żeby znalazł sobie inne zajęcie niż programowanie.



P.S. Słyszałeś kiedyś o znakach interpunkcyjnych?

EDIT: Forum ucięło backslashe, poprawiłem.

[PHP] pobierz, plaintext 
<?php
preg_match ('#^d{1,3}.d{1,3}.d{1,3}.d{1,3}$#', $ipek) // co jest w $ipek? gdzie jest srednik? czemu nie sprawdzasz co zwraca preg_match?
?>
[PHP] pobierz, plaintext 

Programowanie nie polega na zapamietywaniu i odtwarzaniu wiedzy, rozwiazan, tylko na wymyslaniu ich, bo nie wszystko [a ledwie, powiedzmy, 5% rzeczy juz bylo, i widziales je]


-------
edit: widze ze post icetaba zniknal:

Wybacz, ale niektóre Twoje wypowiedzi i posty są irytujące i emanują niechęcią do google i manuala, tyle mojego offtop'u.

Pobierasz sobie GET'em dane, potem puszczasz je przez preg_match, który zwraca true jak wyrażenie pasuje do Twojej reguł. Tworzysz odpowiednią regułę i sprawdzasz czy wyrażenie do niej pasuje.

Przed chwilą usunąłeś swojego ostatniego posta z kodem. Brakowało Ci w instrukcji warunkowej klamer. Proponuję Ci naukę PHP od postaw, naprawdę.

[PHP] pobierz, plaintext 
<?php
preg_match ('#^d{1,3}.d{1,3}.d{1,3}.d{1,3}$#', $ipek)
$ipek = $_GET['ipek'];
		exec('ping '.$ipek, $pok);
				echo $pok[1];
?>
[PHP] pobierz, plaintext 

1. W funkcji preg_match" title="Zobacz w manualu PHP" target="_manual używasz jako argumentu zmiennej ipek, ale zastanów się przez chwilę czy ona istnieje? Deklarujesz ją dopiero później.

2. Jak słusznie zauważył ~dr_bozno brakuje Ci średnika.

3. Funkcja preg_match" title="Zobacz w manualu PHP" target="_manual zostanie wywołana, a dalsza część kodu wykona się tak samo jakby jej nie było.

Programowanie polega na analitycznym myśleniu, jeśli sam nie możesz sobie poradzić z takimi problemami, to programowanie widocznie nie jest dla Ciebie, i nie obrażaj się tu na mnie za to co pisze.

Poprawny kod:

[PHP] pobierz, plaintext 
<?php
$ipek = $_GET['ipek'];
 
if (preg_match ('#^d{1,3}.d{1,3}.d{1,3}.d{1,3}$#', $ipek))
{
  exec ('ping '.$ipek, $pok);
  echo $pok[1];
}
else
{
  echo 'Nieprawidłowe IP';
}
?>
[PHP] pobierz, plaintext 

icetab: nic tu nikomu nie ściska, poza zastraszająco niskim poziomem Twoich pytań. Trochę samodzielności.