Jonathan Tan <jonathantanmy@xxxxxxxxxx> writes: >> +static void filter_trees_free(void *filter_data) { >> + struct filter_trees_depth_data* d = filter_data; >> + oidmap_free(&d->seen_at_depth, 1); >> + free(d); >> +} > > Check for NULL-ness of filter_data too, to match the usual behavior of > free functions. Also, the asterisk sticks to the variable, not type, i.e. struct filter_trees_depth_data *d = filter_data;