free_holder_data(): check whether member metric in non-NULL
free_holder_data(): check whether member metric in non-NULL

If metric is NULL, we must not dereference it
Freeing metric members that might be NULL is well-defined

allinurl/goaccess,allinurl/goaccess,allinurl/goaccess,allinurl/goaccess