[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Findutils-patches] [PATCH] Avoid segfault in -execdir when PATH is
From: |
Eric Blake |
Subject: |
Re: [Findutils-patches] [PATCH] Avoid segfault in -execdir when PATH is unset. |
Date: |
Mon, 20 Aug 2007 06:26:47 -0600 |
User-agent: |
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.6) Gecko/20070728 Thunderbird/2.0.0.6 Mnenhy/0.7.5.666 |
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
According to James Youngman on 8/19/2007 9:29 AM:
> 2007-08-19 James Youngman <address@hidden>
>
> find/parser.c (check_path_safety): Assume the path is safe is
> $PATH is not set. This avoids a segfault in that situation
> and thus fixes Savannmah bug #20834.
^^^^^^^^^
Fix the typo, then this looks good to commit.
Definitely safe for 4.2.x; a stable branch should never crash.
> + /* $PATH is not set. Assume the OS default is safe.
> + * That may not be true on Windows, but I'm not aware
> + * of a way to get Windows to avoid searching the
> + * current directory anyway.
> + */
The above comment does not affect Cygwin, but may be true for mingw. But
I agree with your approach - there is no need to special case a
proprietary OS with its propensity to search the current directory first
regardless of whether $PATH is set or what it contains.
- --
Don't work too hard, make some time for fun as well!
Eric Blake address@hidden
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Cygwin)
Comment: Public key at home.comcast.net/~ericblake/eblake.gpg
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFGyYiH84KuGfSFAYARAsE6AJ9HS6iey/oTkPrnWoGezTvuKMFMsACgpEH4
yiwp/0fLhZnGZiUWbDBlbBo=
=nqnn
-----END PGP SIGNATURE-----