Y a-t-il un risque d'injection dans la fonction mail() de PHP?

Question

Je me demande s'il y a un risque de sécurité potentiel du code suivant. Je l'utiliser sur mon blog chaque fois qu'un utilisateur soumet un commentaire, il me envoyer un message texte:

mail('cellnumber@messaging.sprintpcs.com', '', 
    "Comment posted by $name: $comment", 
    "From: comments@example.com"); 

Où $name et $comment sont des valeurs entrées par l'utilisateur qui n'ont pas vraiment été assainies en aucune façon. Est-il possible qu'un utilisateur puisse faire quelque chose de malveillant ici? La documentation de mail() ne dit rien à ce sujet, mais il est juste incorrect de coller des valeurs entrées par l'utilisateur directement dans une chaîne. Y a-t-il un risque réel ou suis-je simplement parano?

Answer 1

Tant que tout le matériel de l'utilisateur est conservé dans le corps , il n'y a pas de risque d'injection. Toutefois, dès qu'un utilisateur peut affecter les en-têtes de courrier, il peut injecter des en-têtes supplémentaires et faire des choses telles que l'envoi d'e-mails ou l'ajout d'un message complètement différent, y compris des pièces jointes.

Si vous recherchez des retours à la ligne dans les parties qui affectent les en-têtes et que vous les rejetez si cela apparaît, cela devrait suffire. La norme SMTP utilise CRLF pour séparer les lignes d'en-tête, mais AFAIK, de nombreux serveurs basés sur Unix veulent que vous n'utilisiez que LF (parce que c'est le séparateur de ligne natif sur ces systèmes). Le relayer le courrier le convertit alors en l'envoyant plus loin en amont.

Answer 2

PHP est plein de trucs dangereux, mais je ne suis pas sûr qu'il y ait beaucoup de risques ici. Il est certain qu'un commentaire trop long pourrait déborder la limite que vous pourriez avoir sur la taille du message. Cela ne vous fera probablement pas de mal.

La syntaxe de $XXX de PHP étend la chaîne, mais ne fait aucune évaluation supplémentaire du XXX le fait-il? Si $XXX fonctionnait comme une sorte de eval(), vous seriez dans un monde d'exécution de code à distance, mais je ne pense pas que ce soit le cas. Si cela fait SMTP à l'intérieur, alors un . sur une ligne suivie par une ligne vide signalerait la fin d'un courrier, et théoriquement un attaquant pourrait suivre cela avec plus de texte dans les lignes successives pour détourner votre mailer à mail un autre message à quelqu'un d'autre. C'est si PHP ne reconnaît pas et échappe à une telle chaîne en premier lieu. C'est quelque chose que je regarderais.

Answer 3

Soyez conscient du caractère de nouvelle ligne à l'intérieur du nom. Deux conséquents (erm .. pardonnez mon anglais, deux newlines l'un après l'autre) newlines signifient "fin des en-têtes" dans SMTP. La séquence "\ n. \ N" (ligne vide avec seulement un point) signifie également "fin du message".

EDIT: Oui, il y a un risque de spammeurs! Que se passe-t-il si le nom $ contient réellement:

someonesName 
CC: spamsubject@domain.com; anotherspamsubject@domain.com 

(c'est-à-dire quelque chose pour injecter plus d'en-têtes)?

EDIT2: n'a pas remarqué que "name" et "comment" étaient tous deux dans la partie du corps du message. Alors mon idée sur les spammeurs n'est pas valide.