On 2011/02/14 21:39, Claudio Jeker wrote:
> 
> After a bit more thinking I like this version a lot better. This correctly
> prints the neighbor AS number in both cases 16bit and 32bit AS Number.

Ok. 

Personally the only time I've hit this is when connecting to   
an ixp route-server running with transparent-as, so logging
the actual path is of limited use, but doing so doesn't hurt 
nd might be useful in some cases.



> Comments?
> -- 
> :wq Claudio
> 
> Index: rde.c
> ===================================================================
> RCS file: /cvs/src/usr.sbin/bgpd/rde.c,v
> retrieving revision 1.306
> diff -u -p -r1.306 rde.c
> --- rde.c     29 Jan 2011 17:10:45 -0000      1.306
> +++ rde.c     14 Feb 2011 19:48:11 -0000
> @@ -922,8 +922,8 @@ rde_update_dispatch(struct imsg *imsg)
>                       fas = aspath_neighbor(asp->aspath);
>                       if (peer->conf.remote_as != fas) {
>                           log_peer_warnx(&peer->conf, "bad path, "
> -                             "starting with %i, "
> -                             "enforce neighbor-as enabled", fas);
> +                             "starting with %s, "
> +                             "enforce neighbor-as enabled", log_as(fas));
>                           rde_update_err(peer, ERR_UPDATE, ERR_UPD_ASPATH,
>                                   NULL, 0);
>                           goto done;

Reply via email to