On Sat, Mar 02, 2013 at 12:58:48AM -0600, Doug Goldstein wrote: > @@ -152,6 +199,14 @@ static int parse_acl_file(const char *filename, ACLList > *acl_list) > failure: > fclose(f); > > + if (include_list) { > + for (i = 0; i < include_count; i++) { > + if (include_list[i]) > + free(include_list[i]);
free(NULL) is a nop so the if isn't necessary - you can free(include_list[i]) unconditionally.