Changeset 6296 for trunk/psLib/src/db

Timestamp:

Feb 1, 2006, 6:48:47 PM (20 years ago)

Author:

desonia

Message:

patch from Bug 680 -- psDB behavior fixes.

File:

• trunk/psLib/src/db/psDB.c (modified) (4 diffs)

trunk/psLib/src/db/psDB.c

@@ -12 +12 @@
  *  @author Joshua Hoblitt
  *
- *  @version $Revision: 1.45 $ $Name: not supported by cvs2svn $
- *  @date $Date: 2005-12-24 00:33:15 $
+ *  @version $Revision: 1.46 $ $Name: not supported by cvs2svn $
+ *  @date $Date: 2006-02-02 04:48:47 $
  *
  *  Copyright 2005 Joshua Hoblitt, University of Hawaii
@@ -308 +308 @@
     // Get number of rows returned in result
     rowCount = mysql_num_rows(result);
+    // XXX has mysql's semantics changed?  If this is zero then why was a
+    // result set returned?
+    if (!rowCount) {
+        mysql_free_result(result);
+        return NULL;
+    }
 
     // pre-allocate enough elements to hold the complete result set
@@ -476 +482 @@
 
     rowCount = mysql_num_rows(result);
+    // XXX has mysql's semantics changed?  If this is zero then why was a
+    // result set returned?
+    if (rowCount == 0) {
+        mysql_free_result(result);
+        return NULL;
+    }
 
     // pre-allocate enough elements to hold the complete result set
@@ -1258 +1270 @@
         } else if (item->type == PS_DATA_STRING) {
             // + column name + _ + like + _ + ' + value + '
-            if (*(char *)item->data.V == '\0') {
+            // check for NULL and empty ("") strings
+            if (item->data.V == NULL || *(char *)item->data.V == '\0') {
                 psStringAppend(&query, "%s IS NULL", item->name);
             } else {