Fix bug where binary in cwd would trump binary in PATH with test_exec

Fix memory leak by freeing iterator.
Simplify logic to be more intuitive.
Separate out _check_exec() into two logically different portions.
Get rid of unnecessary goto's.

Bug 5848