On Tue, Dec 15, 2015 at 2:12 PM, Bernd Eckenfels <e...@zusammenkunft.net> wrote: > Hello, > > I always like it when access() is used instead of stat() magic. > > I noticed that the new ProgramExists in java_md_common.c does not > anymore reject directories (which are typically executable). Not sure > it this matters or is intentional, it is a change in semantic.
Right. This is a small loss of robustness. Changing stat into access here isn't really right because not stat has a superset of functionality, and (like I keep saying) all the stat calls in the JDK are potentially broken and need fixing.