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)); }