3 replies to this topic

[Sayfor]

Boa tarde, sou novato em PHP, tentei incluir este código no meu site só que mesmo assim ele da erro você digita o endereço www.blablabla.com e aparece o seguinte:
Notice: Undefined index: p in C:\Arquivos de programas\EasyPHP 2.0b1\www\index.php on line 9
<conteudo pag principal>
Pagina Principal Sobre a Empresa Fale Conosco
© 2004/2005 Todos os direitos reservados

O code é:

<?php
$sessao = $_GET["p"];
$pag = $sessao.".php";
if (empty($_SERVER['QUERY_STRING'])) {
	include  "principal.php"; }
	else {
if(isset($pag) && file_exists($pag)){
 include($pag);
} else {
include ("erro.php");
}
}
?>

Alguém sabe me dizer se há algo errado?

Edição feita por: Sayfor, 27/01/2009, 15:02.

[Paulo André]

Na verdade não é um erro, é só um aviso lhe informando que o índice p que você está procurando na primeira linha, com o $_GET, não existe.

Uma boa prática é sempre você tratar este tipo de coisa, já que, pelo que eu entendi, na página principal não é utilizado querystring então não há necessidade deste.

Tente:

<?php

if ( !isset($_GET["p"]) )
{
	include  "principal.php";
}
else
{
	$pag = $_GET['p'] . ".php";
	
	if( file_exists($pag) )
	{
		include($pag);
	}
	else
	{
		include ("erro.php");
	}
}

?>

PS. provavelmente o aviso desapareceu, mas aconselho manipular querystring com arquivos de uma maneira mais segura.

[]'sAté mais

[Sayfor]

hehe por ser novato não estou 100% ainda, pode me dizer um code mais seguro então? oO
Eu tinha colocado error_reporting(E_ERROR | E_WARNING | E_PARSE); para esconder o aviso
Desde já agradeço.

[Guilherme Luiz]

Questão de segurança da query string é tão citado quanto a segurança contra gravidez e DST's...
Todu mundoi sabe que existe, sabe como fazer(alguns realmente não sabem) mas sempre na melhor hora, na hora da empolgação (aprendendo) esquece!

Então como o Paulo disse, o modo como vc pretende usar query string manipulando diretamente arquivos colocando a extenção deles sem nenhum preg_replace fica quase nula a segurança do seu site...

Portanto, sempre procure novos metodos de prevenção e proteção, tanto no php como na vida