Revision: 5956 http://ipcop.svn.sourceforge.net/ipcop/?rev=5956&view=rev Author: owes Date: 2011-10-16 13:10:14 +0000 (Sun, 16 Oct 2011) Log Message: ----------- Add sanity checks to better warn against layer 8 error.
Modified Paths: -------------- ipcop/trunk/html/cgi-bin/email.cgi Modified: ipcop/trunk/html/cgi-bin/email.cgi =================================================================== --- ipcop/trunk/html/cgi-bin/email.cgi 2011-10-16 11:27:28 UTC (rev 5955) +++ ipcop/trunk/html/cgi-bin/email.cgi 2011-10-16 13:10:14 UTC (rev 5956) @@ -99,6 +99,15 @@ } # end unless ($saveerror) if ($cgiparams{'ACTION'} eq $Lang::tr{'send test mail'}) { + if ($settings{'EMAIL_SERVER'} eq '') { + $errormessage .= "$Lang::tr{'email server can not be empty'}<br/>"; + } + if ($settings{'EMAIL_TO'} eq '' ) { + $errormessage .= "$Lang::tr{'to email bad'}<br/>"; + } + if ($errormessage) { + goto ERROR; + } my $template = "/var/ipcop/email/templates/test"; @@ -135,6 +144,7 @@ $errormessage = "$Lang::tr{'test email could not be sent'}:<br/>"; $errormessage .= "$return <br />"; } +ERROR: } # end if ($cgiparams{'ACTION'} eq $Lang::tr{'send test mail'}) &Header::openpage($Lang::tr{'email settings'}, 1, ''); This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. ------------------------------------------------------------------------------ All the data continuously generated in your IT infrastructure contains a definitive record of customers, application performance, security threats, fraudulent activity and more. Splunk takes this data and makes sense of it. Business sense. IT sense. Common sense. http://p.sf.net/sfu/splunk-d2d-oct _______________________________________________ Ipcop-svn mailing list Ipcop-svn@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ipcop-svn