Print this page
6659 nvlist_free(NULL) is a no-op

@@ -279,21 +279,15 @@
 
         moderr = err ? EMOD_NOMEM : 0;
 out:
         if (moderr == 0)
                 *out = fmri;
-
-        if (moderr != 0 && fmri)
+        else
                 nvlist_free(fmri);
 
-        if (obj)
                 nvlist_free(obj);
-
-        if (site)
                 nvlist_free(site);
-
-        if (ctxt)
                 nvlist_free(ctxt);
 
         return (moderr == 0 ? 0 : topo_mod_seterrno(mod, moderr));
 }