Do the comparision between argv[0] and argv[vo->index] in a meaningful

way.

Fixes the bug wherein add_list or update_list gets passed ace_type =
'LIST' and ace_name = '', causing the list to end up being owned by the
next value of list_id from the numvals table, which will end up being
the next list to be created.

diff --git a/server/qvalidate.pc b/server/qvalidate.pc
index 29a66897cb603466732fb971f8a869205e830cb8..f9500511693166aade4db91759af0674eedefab4 100644 (file)
--- a/server/qvalidate.pc
+++ b/server/qvalidate.pc
@@ -375,7 +375,7 @@ int validate_typedata(struct query *q, char *argv[], struct valobj *vo)
           * that we're trying to resolve, we should get the value of
           * numvalues.[idfield] for the id.
           */
-         if (vo->idfield && !strcmp(argv[0], argv[vo->index]))
+         if (vo->idfield && (*(int *)argv[0] == *(int *)argv[vo->index]))
            {
              set_next_object_id(q->validate->object_id, q->rtable, 0);
              name = vo->idfield;