Claudio Jeker([email protected]) on 2019.11.09 15:32:39 +0100:
> On Sat, Nov 09, 2019 at 03:27:31PM +0100, Denis Fondras wrote:
> > Fix function name in error message.
> > 
> > Index: kroute.c
> > ===================================================================
> > RCS file: /cvs/src/usr.sbin/ospfd/kroute.c,v
> > retrieving revision 1.112
> > diff -u -p -r1.112 kroute.c
> > --- kroute.c        28 Dec 2018 19:25:10 -0000      1.112
> > +++ kroute.c        9 Nov 2019 14:11:03 -0000
> > @@ -1501,7 +1501,7 @@ rtmsg_process(char *buf, size_t len)
> >                             if ((mpath || prio == kr_state.fib_prio) &&
> >                                 (kr = kroute_matchgw(okr, nexthop)) ==
> >                                 NULL) {
> > -                                   log_warnx("dispatch_rtmsg "
> > +                                   log_warnx("rtmsg_process "
> >                                         "mpath route not found");
> >                                     /* add routes we missed out earlier */
> >                                     goto add;
> > @@ -1536,7 +1536,7 @@ rtmsg_process(char *buf, size_t len)
> >  add:
> >                             if ((kr = calloc(1,
> >                                 sizeof(struct kroute_node))) == NULL) {
> > -                                   log_warn("dispatch calloc");
> > +                                   log_warn("rtmsg_process calloc");
> >                                     return (-1);
> >                             }
> >  
> > @@ -1580,7 +1580,7 @@ add:
> >                     okr = kr;
> >                     if (mpath &&
> >                         (kr = kroute_matchgw(kr, nexthop)) == NULL) {
> > -                           log_warnx("dispatch_rtmsg "
> > +                           log_warnx("rtmsg_process "
> >                                 "mpath route not found");
> >                             return (-1);
> >                     }
> > 
> 
> OK. Another option is to use __func__ which is always correct.

Please definatly use __func__. With that ok benno too.

Reply via email to