Re: [PATCH v2 1/1] write_queue: Avoid possible use-after-free if fd is read-/writable

[Holger Hans Peter Freyther]

On Tue, Jun 10, 2014 at 10:27:52AM +0200, Pablo Neira Ayuso wrote:

> > +#define HANDLE_BAD_FD(rc, label) \
> > +   do { \
> > +           if (rc == -EBADFD) \
> > +                   goto label; \
> > +   } while (0);
> 
> Do we really get anything good with this macro? This checking is only
> required in three places in this patch.

What is the argument against having this macro? readability?