Junio C Hamano <gits...@pobox.com> wrote:
> @@ -156,7 +156,14 @@ void fill_stat_cache_info(struct cache_entry *ce, struct 
> stat *st)
>  static int ce_compare_data(const struct cache_entry *ce, struct stat *st)
>  {
>       int match = -1;
> -     int fd = open(ce->name, O_RDONLY);
> +     static int cloexec = O_CLOEXEC;
> +     int fd = open(ce->name, O_RDONLY | cloexec);
> +
> +     if ((cloexec & O_CLOEXEC) && fd < 0 && errno == EINVAL) {
> +             /* Try again w/o O_CLOEXEC: the kernel might not support it */
> +             cloexec &= ~O_CLOEXEC;
> +             fd = open(ce->name, O_RDONLY | cloexec);
> +     }

Seems fine, I prefer not using recursion so it's
easier-to-review.

But I have a _slight_ preference towards Dscho's version in
<alpine.DEB.2.20.1610251230150.3264@virtualbox> in case we
decide to start using another O_* flag in here.
(but I'm not usually a C programmer)

Reply via email to