Fejl i mail()

prøv:
mail("min@mail.dk", "".$emne."", "".$msg."", "From:".$afsender."");

Du mener ikke:

  mail("min@mail.dk", $emne, $msg, "From: $afsender");

??

Og hvis det virker så har du lige åbnet for mail-spam til helt andre brugere fra de søde hackere rnudt i verden. De vil nemlig prøve at bruge din mail-formular og sætte mere end linie ind i feltet emne. Det kan man nok ikke gøre i en browser, men det bruger de heller ikke. Det de sætter ind er et linieskift efterfulgt af en Bcc: en@email.adresse, og da PHP blot sætter dit emne ind i HTTP-headeren kommer den Bcc også ind, og så kan de sandelig sende spammail til hele verden - indtil du bliver træt af det.

Man skal, og skal og skal, og jeg har ikke sagt det før, så lad man gentage det nu: SKAL checke input fra formularer. Der må kun være en linie i $emne, og i $afsender, og er der antydning af et linieskift, så skal man ikke sende en mail.

hvilken funktion tæller antal linjer ?

Led efter \n  - men få det lige til at virke først. Hackerne kommer nok først i morgen ;)

okay..
hehe.. ja det gør de sku nok først ;-)

Hvis jeg nu hæver pointene til det dobbelte.. så får i begge to point..
Jeg brugte Jaw's svar fordi han var hurtigst (og fordi han også er bornholmer ;-))
men du har forklaret mig så meget om det erik, så du skal også have nogen :-D

Så hvis i begge smider et svar :-)

Nåh, men jeg samler slet ikke på point, tak.

okay :-D
Så må mig og jaw dele :-)

Ja, sikke en bornholmermafia  *g*
(det ord findes ikke på google - men det gør det nu ...)

Ohh, var det nu dig igen :) Jamn så tak herfra, men Erik's er så en smule mere syntax-mæssigt korrekt end min. Jeg rettede kun funktionaliteten, ikke syntaxen.

Erik: Du har lige tilføjet et helt nyt ord til den danske retstavningsordbog ;-)

Jaw: Ja det er mig..... Igen :-D

Super info Erik. Går ud fra det her skulle kune gøre det

if( eregi ("\n", $_POST["afsender")){
// fy du!
}

Tæller \r i øvrigt med i den her sammenhæng?

Jah, så for en god ordens skyld, for at være helt sikker ;)

  if( ereg ("[\r\n]", $_POST["afsender")){

Men hold op med at bruge ereg-funktionerne. De udfases på et tidspunkt i fremtiden fra standard PHP.

Er det mig, eller skal den ikke se sådan her ud:
if( ereg ("[\r\n]", $_POST["afsender"])){
??
Eller var det med vilje at der manglede ']' i $_POST["afsender" ??

Ja, der manglede en ]