Acked-by: Donald Sharp <sha...@cumulusnetworks.com> On Tue, Jun 14, 2016 at 2:07 PM, Christian Franke <ch...@opensourcerouting.org> wrote: > From: Christian Franke <nob...@nowhere.ws> > > While the DEFUN should match the list of clients registered in > vtysh, it seems better to handle the case explicitly instead of > relying on the client list and the DEFUN signature being in sync. > > Signed-off-by: Christian Franke <ch...@opensourcerouting.org> > --- > vtysh/vtysh.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/vtysh/vtysh.c b/vtysh/vtysh.c > index 63b596a..625ab1c 100644 > --- a/vtysh/vtysh.c > +++ b/vtysh/vtysh.c > @@ -1840,6 +1840,9 @@ DEFUN (vtysh_write_terminal_daemon, > break; > } > > + if (i == array_size(vtysh_client)) > + return CMD_ERR_NO_MATCH; > + > ret = vtysh_client_execute(&vtysh_client[i], "show running-config\n", > stdout); > > return ret; > -- > 2.8.0 > > > _______________________________________________ > Quagga-dev mailing list > Quagga-dev@lists.quagga.net > https://lists.quagga.net/mailman/listinfo/quagga-dev
_______________________________________________ Quagga-dev mailing list Quagga-dev@lists.quagga.net https://lists.quagga.net/mailman/listinfo/quagga-dev