[patch] Minor O_CLOEXEC optimization, "regression" fix

Jan Kratochvil jan.kratochvil@redhat.com
Wed Oct 9 13:10:00 GMT 2013


On Tue, 08 Oct 2013 21:44:33 +0200, Tom Tromey wrote:
> >>>>> "Jan" == Jan Kratochvil <jan.kratochvil@redhat.com> writes:
> Jan> -  if (result == NULL)
> Jan> +  if (result != NULL)
> Jan> +    fopen_e_ever_succeeded = 1;
> Jan> +  else if (!fopen_e_ever_succeeded)
> 
> What if we have it check for EINVAL instead?

May one rely on MS-Windows fopen("","re") will fail with EINVAL if it fails
because of the "e" flag, Kai?  It is in GDB function gdb_fopen_cloexec.

original post:
	[patch] Minor O_CLOEXEC optimization, "regression" fix
	https://sourceware.org/ml/gdb-patches/2013-10/msg00233.html


Thanks,
Jan



More information about the Gdb-patches mailing list