Re: [PATCH 05/11] ipaccess: more robust error handling in the accept patch

On Wed, Aug 22, 2012 at 05:04:47PM +0200, pablo@gnumonks.org wrote:

+err: + close(bfd->fd); + e1inp_line_put(line); + return ret; }

one more thing. The code in ipaccess_drop appears to set bfd->fd = -1. Do we need that? You certainly don't have to do this in the current patchset but it would be nice if we at least know if we should (re)initialize to -1 or not. holger