Hi All,
  I notice the origin attribute in function bgp_update_attrs , if route source  
is BGP, I think it should set as ORIGIN_IGP, not INCOMPLETE, according to RFC 
or Cisco device . It seems there is mistake on Line 1443 in attrs.c .

Line 1443:
bgp_set_attr_u32(&attrs, pool, BA_ORIGIN, 0, src ? ORIGIN_INCOMPLETE : 
ORIGIN_IGP);

ð  bgp_set_attr_u32(&attrs, pool, BA_ORIGIN, 0, src ? ORIGIN_IGP : 
ORIGIN_INCOMPLETE ); // correct?

Thanks
Arvin

Reply via email to