This change will avoid having the keyword "invalid" twice as:
[invalid MgmtD Len nnn] (invalid)
and
(invalid TLV length nnn != 1) (invalid)
ND_PRINT("\n\tDomain name: ");
mgmtd_len = GET_U_1(tptr + 3);
if (mgmtd_len < 1 || mgmtd_len > VTP_DOMAIN_NAME_LEN) {
ND_PRINT("\n\tDomain name: ");
mgmtd_len = GET_U_1(tptr + 3);
if (mgmtd_len < 1 || mgmtd_len > VTP_DOMAIN_NAME_LEN) {
- ND_PRINT(" [invalid MgmtD Len %u]", mgmtd_len);
+ ND_PRINT(" [MgmtD Len %u]", mgmtd_len);
goto invalid;
}
nd_printjnp(ndo, tptr + 4, mgmtd_len);
goto invalid;
}
nd_printjnp(ndo, tptr + 4, mgmtd_len);
* in units of 16-bit words.
*/
if (tlv_len != 1) {
* in units of 16-bit words.
*/
if (tlv_len != 1) {
- ND_PRINT(" (invalid TLV length %u != 1)", tlv_len);
+ ND_PRINT(" [TLV length %u != 1]", tlv_len);
goto invalid;
} else {
tlv_value = GET_BE_U_2(tptr + 2);
goto invalid;
} else {
tlv_value = GET_BE_U_2(tptr + 2);