diff options
| author | ISHIDA Wataru <ishida.wataru@lab.ntt.co.jp> | 2015-08-25 11:29:24 +0900 |
|---|---|---|
| committer | ISHIDA Wataru <ishida.wataru@lab.ntt.co.jp> | 2015-08-25 11:45:23 +0900 |
| commit | 1d8c71e22b9c974db66ff41cba81ae601ea14dd7 (patch) | |
| tree | 25e458da32ddc543352452741119aa295dd9a5f5 /packet | |
| parent | 64ddb52890451da4c9877f8b3273fb9fe35f878c (diff) | |
*: better log msg
Signed-off-by: ISHIDA Wataru <ishida.wataru@lab.ntt.co.jp>
Diffstat (limited to 'packet')
| -rw-r--r-- | packet/validate.go | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/packet/validate.go b/packet/validate.go index e82113ee..73697c4d 100644 --- a/packet/validate.go +++ b/packet/validate.go @@ -139,7 +139,7 @@ func ValidateAttribute(a PathAttributeInterface, rfs map[RouteFamily]bool, doCon case *PathAttributeUnknown: if p.getFlags()&BGP_ATTR_FLAG_OPTIONAL == 0 { - eMsg := "unrecognized well-known attribute" + eMsg := fmt.Sprintf("unrecognized well-known attribute %s", p.GetType()) data, _ := a.Serialize() return false, NewMessageError(eCode, eSubCodeUnknown, data, eMsg) } @@ -156,7 +156,7 @@ func ValidateFlags(t BGPAttrType, flags BGPAttrFlag) (bool, string) { * RFC 4271 P.17 For well-known attributes, the Transitive bit MUST be set to 1. */ if flags&BGP_ATTR_FLAG_OPTIONAL == 0 && flags&BGP_ATTR_FLAG_TRANSITIVE == 0 { - eMsg := "well-known attribute must have transitive flag 1" + eMsg := fmt.Sprintf("well-known attribute %s must have transitive flag 1", t) return false, eMsg } /* @@ -164,11 +164,11 @@ func ValidateFlags(t BGPAttrType, flags BGPAttrFlag) (bool, string) { * the Partial bit MUST be set to 0. */ if flags&BGP_ATTR_FLAG_OPTIONAL == 0 && flags&BGP_ATTR_FLAG_PARTIAL != 0 { - eMsg := "well-known attribute must have partial bit 0" + eMsg := fmt.Sprintf("well-known attribute %s must have partial bit 0", t) return false, eMsg } if flags&BGP_ATTR_FLAG_OPTIONAL != 0 && flags&BGP_ATTR_FLAG_TRANSITIVE == 0 && flags&BGP_ATTR_FLAG_PARTIAL != 0 { - eMsg := "optional non-transitive attribute must have partial bit 0" + eMsg := fmt.Sprintf("optional non-transitive attribute %s must have partial bit 0", t) return false, eMsg } |