Complementando...
Por: Arnaldo,
31/08/2009 13:50:09
Acho que os colegas já enumeraram todos os problemas referenteas ao código acima. Eu gostaria apenas de completar dizendo que a senha cadastrada na base de dados deve estar criptografada com com algum algorítimo assimétrico + salt password e que também é uma boa prática para evitar o sql injection o uso de bindParam do PDO. Abraço galera!
Não Faria pensando apenas no login
Por: Mário,
13/03/2009 08:59:08
Na minha opinião, a proteção não deve estar apenas na hipótese de uma pessoa querer se logar sem ter o cadastro.
Creio que deve-se pensar mais a fundo no que o usuário malicioso queira fazer.
Existem muitos casos onde o usuário malicioso envia instruções SQL nos campos de digitação que a página disponibiliza. Creio que a segurança deve se extender até este ponto ou mais além.
Com certeza , addslashes é mais seguro que sua proposta.
Fraco
Por: Julio,
11/09/2008 19:02:46
Muito trabalho pra pouco resultado, o mais correto é o mysql_real_escape_string()
http://www.php.net/manual/en/function.mysql-real-escape-string.php
fraco esse post
Por: Carlos,
22/08/2008 11:54:16
Cara seguinte a melhor forma de se proteger é usar a funçao do php mesmo que é mysql_real_escape_string() pois é mais completa que a addslashes
como o rafael falou acima esse seu codigo nao faz nada apenas deu mais trabalho em verificar uma entrada de dados que pode acabar com o seu banco sem contar
que se voce colocar
' or 1='1 ja ta dentro do sistema
um abraço
os codigos postados aqui poderiam ser avaliados antes de postar porque senao vira terra de ninguem e codigos como esse ensinao coisas erradas, sem contar que é um codigo bem maior do que podemos usar para uma verificação simples de senha
abraçoo
falho demais
Por: Micael,
05/08/2008 13:57:06
1º: se resolve muito mais facilmente usando o addslashes
2º como o Rafael disse, tu não evita o sql injection, soh valida os dados depois. Pode deletar o banco, validar de varias formas.
acho o conteudo deste site muito fraco, cheio de falhas de segurança. o pessoal tem que cuidar mais antes de publicar material.
Segurança falha
Por: Rafael,
31/07/2008 06:45:49
Este seu metodo de proteção é extremamente falho. é possivel até mesmo deletar seu banco de dados. colocando uma aspa simples no campo "txt_usu", qualquer comando apos , será executado, por exemplo
' and drop database sqlmal --
nao testei, mas isso, ou algo proximo, fura seu sistema facilmente