php/mySQL error: mysql_num_rows(): supplied argument is not a valid MySQL result

Question

I'm trying to INSERT INTO a mySQL database and I'm getting this error on:

if (mysql_num_rows($login) == 1){

Here is the php, The php does add the user to the database. I can't figure it out.

<?
session_start();
require("config.php");

$u = $_GET['username'];
$pw = $_GET['password'];
$pwh = $_GET['passwordhint'];
$em = $_GET['email'];
$zc = $_GET['zipcode'];


$check = "INSERT INTO family (loginName, email, password, passwordhint, location) VALUES ('$u', '$pw', '$pwh', '$em', '$zc')";


$login = mysql_query($check, $link) or die(mysql_error());


if (mysql_num_rows($login) == 1) {
$row = mysql_fetch_assoc($login);
echo 'Yes';exit;


} else {
echo 'No';exit;
}

mysql_close($link);
?>

Thanks,

BoltClock · Accepted Answer · 2010-06-09 17:19:05Z

4

You're doing an INSERT query, so $login won't contain any result set, it's either true or false.

So instead of this:

if (mysql_num_rows($login) == 1) {

You can either do this:

if (mysql_affected_rows($link) == 1) {

Or this:

if ($login == true) {

Also, you don't need this line anymore:

$row = mysql_fetch_assoc($login);

Unrelated to your question, but a very important thing to remember when doing SQL queries: you'll need to escape all your $_GET data with mysql_real_escape_string() just before inserting, so as to prevent SQL injection.

answered Jun 9, 2010 at 17:19

BoltClock

728k165 gold badges1.4k silver badges1.4k bronze badges

Sign up to request clarification or add additional context in comments.

Comments

thetaiko · Accepted Answer · 2010-06-09 17:20:00Z

0

mysql_query() returns True/False for INSERT, UPDATE, DELETE, DROP...

answered Jun 9, 2010 at 17:20

thetaiko

7,8342 gold badges35 silver badges49 bronze badges

Collectives™ on Stack Overflow

php/mySQL error: mysql_num_rows(): supplied argument is not a valid MySQL result

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related