mail() error - help!

[xorion]

Hi guys,

I recently upgraded to php5 with register_globals=off on my directadmin.

my mail() script stopped working, how do i get it to work again? thanks!

code:

if (mail($to, $subject, $body)) {
echo("<p>Message successfully sent!</p>");
} else {
echo("<p>Message delivery failed...</p>");
}

[Teach]

Hi there,

Have you checked you've got a mail daemon running, such as sendmail/qmail?

mail() uses this to send email

[xorion]

I'm using exim

mail 2105 0.0 0.0 8720 1044 ? Ss 00:52 0:01 /usr/sbin/exim -bd -q15m -oP /var/run/exim.pid

[Teach]

Hi there,

As your MTA path is /usr/sbin/exim:

You will probably need to update "sendmail_path" in your php config 🙂

[xorion]

Currently it is commented out, should i enable it and put:

sendmail_path = /usr/sbin/exim

--
[mail function]
; For Win32 only.
SMTP = localhost
smtp_port = 25

; For Win32 only.
;sendmail_from = [e-mail]

; For Unix only. You may supply arguments as well (default: "sendmail -t -i").

;sendmail_path =

ps aux|grep exim

mail 2105 0.0 0.0 8720 1044 ? Ss 00:52 0:01 /usr/sbin/exim -bd -q15m -oP /var/run/exim.pid
mail 11263 0.0 0.0 8752 1700 ? S 22:21 0:00 /usr/sbin/exim -bd -q15m -oP /var/run/exim.pid
mail 11364 0.0 0.0 8752 1412 ? S 22:22 0:00 /usr/sbin/exim -bd -q15m -oP /var/run/exim.pid
root 11382 0.3 0.1 8716 2080 ? S 22:22 0:00 /usr/sbin/exim -q
root 12552 0.0 0.1 11228 3060 ? S 22:22 0:00 /usr/sbin/exim -q
mail 12553 0.0 0.0 11420 1856 ? S 22:22 0:00 /usr/sbin/exim -q
mail 12632 0.0 0.0 8752 1412 ? S 22:23 0:00 /usr/sbin/exim -bd -q15m -oP /var/run/exim.pid
mail 12634 0.0 0.0 8752 1412 ? S 22:23 0:00 /usr/sbin/exim -bd -q15m -oP /var/run/exim.pid

[xorion]

tried the above, doesn't work

[bradgrafelman]

Some more code might help. Where are the variables that you pass to mail() defined? Are you logging all PHP errors and have error_reporting set to E_ALL? By "stopped working," do you mean that you're seeing the failed error message, getting a blank e-mail, ...?

[xorion]

Hey, thanks 🙂

It stopped working after upgrading to php5 (register_globals=off too), and now the script output is: Message delivery failed...

"Are you logging all PHP errors and have error_reporting set to E_ALL? " - How do i log the errors of the script?

[xorion]

bradgrafelman;10911923 wrote:

Some more code might help. Where are the variables that you pass to mail() defined? Are you logging all PHP errors and have error_reporting set to E_ALL? By "stopped working," do you mean that you're seeing the failed error message, getting a blank e-mail, ...?

you are right, could be my code. i tested a simple test mail script and it works fine. could you what's wrong with:

<?php
session_start();
include 'config.php';
include 'opendb.php';

$_POST['nric'] = $nric;
$_POST['email'] = $email;

$query = "SELECT * FROM members WHERE member_nric='$nric' AND member_email='$email'";
$result = mysql_query($query) or die(mysql_error());

if (mysql_num_rows($result)==0) {
        header("Refresh: 2; url= forgotpassword.php");
        echo "NRIC and Email does not match";
}

else {
        while($row = mysql_fetch_array($result)){
                $to = $row['member_email'];
                $password = rand(000001,999999);
                $updatepassword = "UPDATE members SET member_password='$password' WHERE member_email='$to'";
                $doupdatepassword = mysql_query($updatepassword);
        }
}

$subject = "Forgot Password";
$body = "Your new password is " .$password;

if (mail($to, $subject, $body)) {
  echo("<p>Message successfully sent!</p>");
 } else {
  echo("<p>Message delivery failed...</p>");
 }

include 'closedb.php';
?>

[bradgrafelman]

This looks backwards:

$_POST['nric'] = $nric;
$_POST['email'] = $email;

You're changing the values of submitted data and setting them equal to variables that don't exist. I think what you mean to do is the opposite - set variables equal to submitted values.

Also note that the script is vulnerable to SQL injection attacks. User-supplied data should never be placed directly into a query string. Instead, you must properly escape it (e.g. using [man]mysql_real_escape_string/man).