Error: eregi() [function.eregi]: REG_ECTYPE

Kellimus

Hello there. I continue to have this error, even after I have entered a valid e-mail address and username.. Here is the errors:

An error occured in script /home/mindflux/public_html/member/includes/config.inc on line 13: eregi() [function.eregi]: REG_ECTYPE
Please enter a valid email address!

An error occured in script /home/mindflux/public_html/member/includes/config.inc on line 13: eregi() [function.eregi]: REG_ECTYPE
Please enter a valid username!

Please try again.

Now here is the code for my form (sorry if its sloppy, I'm in a rush):

<?php//This is the registration page for the site.

//Include the configuration file for error management and such.

require_once ('includes/config.inc'); //Set the page title and include the HTML header.

$page_title = 'Register';

include ('includes/header.html');

if(isset($_POST['submit'])) 
{ //Handle the form		
require_once('includes/mysql_connect.php'); // Connect to the database.	

// Check for a first name.
if(eregi ("^[[:alpha:].' -]{2,15}$", stripslashes(trim($_POST['first_name']))))
{	
$fn = escape_data($_POST['first_name']);
	} else {
		$fn = FALSE;
echo '<p><font color="red" size="+1">Please enter your first name!</font></p>';
}

// Check for a last name.

if (eregi ("^[[:alpha:].' -]{2,30}$", stripslashes(trim($_POST['last_name']))))
{	
$ln = escape_data($_POST['last_name']);
	} else {
	$ln = FALSE;
echo '<p><font color="red" size="+1">Please enter your last name!</font></p>';
}

// Check for an email address.	

if (eregi ("^[[:alum:]][a-z0-9_.-]*@[a-z0-9.-]+\.[a-z]{2,4}$", stripslashes(trim($_POST['email']))))
{
$e = escape_data($_POST['email']);
} else {
$e = FALSE;
echo '<p><font color="red" size="+1">Please enter a valid email address!</font></p>';
}

// Check for a username.

if (eregi ("^[[:alum:]_]{4, 20}$", stripslashes(trim($_POST['username'])))) 
{
	$u = escape_data($_POST['username']);
} else {
	$u = FALSE;
	echo '<p><font color="red" size="+1">Please enter a valid username!</font></p>';
}	
}

// Check for a password and match against the confirmed password.

if (eregi ("^[[:alnum:]]{4,20}$", stripslashes(trim($_POST['password1']))))
{		
if($_POST['password1'] == $_POST['password2'])	
{	
$p = escape_data($_POST['password1']);
} else {		
$p = FALSE;
echo '<p><font color="red" size="+1">Your password did not match the confirmed password!</font></p>';
}
} else {	
$p = FALSE;
echo '<p><font color="red" size="+1">Please enter a valid password!</font></p>';
}

if ($fn && $ln && $e && $u && $p)
{		

//Make sure the username is available.
$query = "SELECT user_id FROM users WHERE username='$u'";
$result = @mysql_query ($query);

if (mysql_num_rows($result) == 0) //Available.
{

// Add the user.	

$query = "INSERT INTO users (username, first_name, last_name, email, password, registration_date) VALUES ('$u', '$fn', '$ln', '$e', PASSWORD('$p'), NOW() )";

$result = @mysql_query ($query); //Run the query.

if ($result) // If it ran OK.
{		

// Send an email, if desired.

echo '<h3>Thank you for registering!</h3>';
include ('includes/footer.html'); // Include the HTML footer.

exit();
} else // If it did not run OK.
{	

//Send a message to the error log, if desired.					
echo '<p><font color="red" size="+1">You could not be registered due to a system error.  We apologize for any inconvenience.</font></p>';
}
} else // The username is not available.	
{

echo '<p><font color="red" size="+1">That username is already taken.</font></p>';	
}	

mysql_close(); // Close the database connection.	

} else // If one of the data tests failed.
{	
echo '<p><font color="red" size="+1">Please try again.</font></p>';	
} // End of the main Submit conditional

?>
<h1>Register</h1>
<form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post">
<fieldset>

<p><b>First Name:</b> <input type="text" name="first_name" size="15" maxlength="15" value="<?php if (isset($_POST['first_name'])) echo $_POST['first_name']; ?>" /></p>

<p><b>Last Name:</b> <input type="text" name="last_name" size="30" maxlength="30" value="<?php if (isset($_POST['last_name'])) echo $_POST['last_name']; ?>" /></p>

<p><b>Email Address:</b> <input type="text" name="email" size="40" maxlength="40" value="<?php if (isset($_POST['email'])) echo $_POST['email']; ?>" /></p>

<p><b>User Name:</b> <input type="text" name="username" size="10" maxlength="20" value="<?php if (isset($_POST['username'])) echo $_POST['username']; ?>" /> <small>Use only letters, numbers, and the underscore.  Must be between 4 and 20 characters long.</small></p>

<p><b>Password:</b> <input type="password" name="password1" size="20" maxlength="20" /> <small>Use only letters and numbers.  Must be between 4 and 20 characters long.</small></p>

<p><b>Confirm Password:</b> <input type="password" name="password2" size="20" maxlength="20" /></p></fieldset><div align="center"><input type="submit" name="submit" value="Register" /></div>

</form><!-- End of Form -->

<?php // Include the HTML footer.

include ('includes/footer.html');?>

I don't understand why it continues to error out, even after I have entered a valid email and username..

Help would be appreciated 🙂

wilku

REG_ECTYPE means invalid character class - this means your regex pattern is faulty (I haven't used ereg so I really can't say what is wrong).
Edit: I'm guessing "-" may be the culprit - it has a special meaning in character class like [a-z]. Try escaping it with \

Weedpacket

"alum"?

(And I don't know if you've heard or not, but a few years ago PHP's developers decided to put more support into the PCRE functions (preg_match(), etc.); the POSIX regular expression library is slated to be moved out of the core PHP distribution in PHP 6).

Kellimus

Weedpacket wrote:
"alum"?

(And I don't know if you've heard or not, but a few years ago PHP's developers decided to put more support into the PCRE functions (preg_match(), etc.); the POSIX regular expression library is slated to be moved out of the core PHP distribution in PHP 6).

Hahaha, thanks for pointing that out to me 🙂

I just started learning PHP about six months to a year ago (I have many years of C++ programming, and I've noticed the two are similar in syntax and logic) so I really don't know all that much... Got any places I could check up (learn 😛) on PCRE functions?

I still have one error though...

eregi() [function.eregi]: REG_BADBR

It only happens when I input my username.. Here is the code for the username:

// Check for a username.	
if (eregi ("^[[:alnum:]_]{4, 20}$", stripslashes(trim($_POST['username']))))
{

$u = escape_data($_POST['username']);

} else {

$u = FALSE;

echo '<p><font color="red" size="+1">Please enter a valid username!</font></p>';
}

I don't get what's wrong with it now.. Please help?

Weedpacket

Kellimus wrote:
Got any places I could check up (learn 😛) on PCRE functions?

The functions are listed in the manual under [man]PCRE[/man]. For the regular expression syntax, there is a reference page in that section, and most web sites that discuss regular expressions use the same syntax.

Your error: You have a space between the "," and the "20".

More generally, it would be faster if you checked the length of the string before throwing regexps around:

$username = stripslashes(trim($_POST['username']));
if(strlen($username)>=4 && strlen($username<=20) && !preg_match('/[^a-z0-9_]/i', $username))

Where I used preg_match, and decided to check that there weren't any characters that are not in the set [a-zA-Z0-9_] instead of checking that all of them were.

Kellimus

This is my error now:

mysql_num_rows(): supplied argument is not a valid MySQL result resource
You could not be registered due to a system error. We apologize for any inconvenience.

I'm pretty sure this is the section of code that this error is reffering to:

if ($fn && $ln && $e && $u && $p)
{
//Make sure the username is available.

$query = "SELECT user_id FROM users WHERE username='$u'";
$result = @mysql_query ($query);

if (mysql_num_rows($result) == 0) //Available.
{
	// Add the user.

$query = "INSERT INTO users (username, first_name, last_name, email, password, registration_date) VALUES ('$u', '$fn', '$ln', '$e', PASSWORD('$p'), NOW() )";

$result = @mysql_query ($query); //Run the query.

if ($result) // If it ran OK.
{	

// Send an email, if desired.

echo '<h3>Thank you for registering!</h3>';

include ('includes/footer.html'); // Include the HTML footer.

exit();

} else // If it did not run OK.
{
	//Send a message to the error log, if desired.

echo '<p><font color="red" size="+1">You could not be registered due to a system error.  We apologize for any inconvenience.</font></p>';

}
} else // The username is not available.
{

echo '<p><font color="red" size="+1">That username is already taken.</font></p>';
}

mysql_close(); // Close the database connection.

} else // If one of the data tests failed.
{

echo '<p><font color="red" size="+1">Please try again.</font></p>';

Weedpacket

$result = @mysql_query ($query);

While developing, it's best to have the script fail as quickly and as noisily as possible. That way you have a better chance to work out what is failing.

In this case, lose the '@' from the above line and add the line

if(!$result) die(mysql_error());

That way you'll get an error message with information in it (something that, not incidentally, you don't want when you go public; then you can start suppressing errors and putting out generic messages of your own).