Jonathan Tan <[email protected]> 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;

Reply via email to