lport: Tolerate null pointers in destroy functions.

The coding style says to do this. The actual caller doesn't pass a null pointer. Signed-off-by: Ben Pfaff <[email protected]> Acked-by: Mickey Spiegel <[email protected]>
grizax · Dec 20, 2016 · 69d21a9 · 69d21a9
1 parent 4fbc56c
commit 69d21a9
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/ovn/controller/lport.c b/ovn/controller/lport.c
@@ -68,6 +68,10 @@ lport_index_init(struct lport_index *lports, struct ovsdb_idl *ovnsb_idl)
 void
 lport_index_destroy(struct lport_index *lports)
 {
+    if (!lports) {
+        return;
+    }
+
     /* Destroy all of the "struct lport"s.
      *
      * We don't have to remove the node from both indexes. */
@@ -141,6 +145,10 @@ mcgroup_index_init(struct mcgroup_index *mcgroups, struct ovsdb_idl *ovnsb_idl)
 void
 mcgroup_index_destroy(struct mcgroup_index *mcgroups)
 {
+    if (!mcgroups) {
+        return;
+    }
+
     struct mcgroup *mcgroup, *next;
     HMAP_FOR_EACH_SAFE (mcgroup, next, dp_name_node, &mcgroups->by_dp_name) {
         hmap_remove(&mcgroups->by_dp_name, &mcgroup->dp_name_node);