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