Update the modules to use talloc() for their instance handles
Duh.  If we have a destructor on detach, don't call detach
Set destructor on module_entry_t
Reparent the client when added to the list
Shouldn't be any config when we boot