Hi! > +static char **parse_skiplist(char *fs) > +{ > + char **skiplist = NULL; There is no point in initializing variables that are not read before they are assigned. Other than that: Reviewed-by: Cyril Hrubis <chrubis@suse.cz> -- Cyril Hrubis chrubis@suse.cz