Ack: looks good. On Tue, Sep 8, 2015 at 4:02 PM, Donald Sharp <[email protected]> wrote:
> When caling pim_sock_open if the failure cause happens, however > unlikely, don't leak the fd on failure. > > Signed-off-by: Donald Sharp <[email protected]> > --- > pimd/pim_pim.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/pimd/pim_pim.c b/pimd/pim_pim.c > index 41c26a8..cd8a290 100644 > --- a/pimd/pim_pim.c > +++ b/pimd/pim_pim.c > @@ -371,6 +371,7 @@ static int pim_sock_open(struct in_addr ifaddr, int > ifindex) > return -1; > > if (pim_socket_join(fd, qpim_all_pim_routers_addr, ifaddr, ifindex)) { > + close(fd); > return -2; > } > > -- > 1.7.10.4 > > > _______________________________________________ > Quagga-dev mailing list > [email protected] > https://lists.quagga.net/mailman/listinfo/quagga-dev >
_______________________________________________ Quagga-dev mailing list [email protected] https://lists.quagga.net/mailman/listinfo/quagga-dev
