ovsdb-types: Fix bug in ovsdb_base_type_is_ref().
authorBen Pfaff <blp@nicira.com>
Mon, 28 Mar 2011 17:48:36 +0000 (10:48 -0700)
committerBen Pfaff <blp@nicira.com>
Thu, 31 Mar 2011 23:43:01 +0000 (16:43 -0700)
This function only worked properly inside OVSDB itself, because that is
the only place where the 'refTable' member of ovsdb_base_type is set.
Both inside and outside OVSDB, 'refTableName' is set for reference types,
so it's better to check for that.

This doesn't fix any existing bug because this function was only used
inside OVSDB until now.

lib/ovsdb-types.h

index 6903aa83f165fb3eb8b42e3cdfb415cffd737beb..e852391ccd44bc834d33f21673b3154303f05be5 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (c) 2009, 2010 Nicira Networks
+/* Copyright (c) 2009, 2010, 2011 Nicira Networks
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -174,7 +174,7 @@ ovsdb_atomic_type_is_valid(enum ovsdb_atomic_type atomic_type)
 static inline bool
 ovsdb_base_type_is_ref(const struct ovsdb_base_type *base)
 {
-    return base->type == OVSDB_TYPE_UUID && base->u.uuid.refTable;
+    return base->type == OVSDB_TYPE_UUID && base->u.uuid.refTableName;
 }
 
 static inline bool