[PATCH 1/2] Close file descriptors when running ntpdate (#1083978)

Colin Walters walters at verbum.org
Thu Dec 4 14:29:42 UTC 2014


On Thu, Dec 4, 2014, at 09:23 AM, Vratislav Podzimek wrote:

> +    # using iutil.execWithRedirect here would mean that if this function was run
> +    # multiple times from multiple threads in the same time, the execution would
> +    # be serialized because of the program_log lock
> +    proc = subprocess.Popen(["ntpdate", "-q", server], close_fds=True)

- Why not subprocess.call()?  (You should be able to pass close_fds=True to it)

- Is it intentional that you're dropping the equivalent of stdout=open('/dev/null', 'w'), stderr=subprocess.STDOUT ?

- Also worth mentioning this fixes a bug where if the provided server name had shell metacharacters it would fail by executing code instead of returning a clean error


More information about the anaconda-patches mailing list