[PATCH v4 01/16] add util/search-path.{c, h} to test for executables in $PATH
David Bremner
david at tethera.net
Thu Aug 11 22:51:16 PDT 2016
Daniel Kahn Gillmor <dkg at fifthhorseman.net> writes:
> This is a utility function we can use to see whether an executa>
> + if (strchr (exename, '/')) {
> + if (0 == access (exename, X_OK))
> + return TRUE;
> + else
> + return FALSE;
> + }
Should we distinguish between relative and absolute paths here? I can't
think of any security implications, but I'm wondering if a relative path
is likely just a user error.
> + path = (char *) malloc (n);
> + if (! path)
> + return FALSE;
I kindof hate hiding the error here, although I agree it's
unlikely. What about the unixy return 0 ok, 1 not found -1 error?
> + confstr (_CS_PATH, path, n);
> + }
> +
> + tok = strtok_r (path, ":", &save);
> + while (tok) {
I guess it's fine to modify path here, but another option is
strtok_len (in string-util.h)
More information about the notmuch
mailing list