The Tcpdump Group git mirrors - tcpdump/blobdiff

index cbc5868e25bce7147f75949d24b2bab983809f94..854bc49d3d297ed78b4371f658750fcc4ed105eb 100644 (file)

--- a/print-vtp.c

+++ b/print-vtp.c

@@ -26,15 +26,11 @@

#include "config.h"

#endif

-#include <tcpdump-stdinc.h>

+#include <netdissect-stdinc.h>

-#include <stdio.h>

-#include <string.h>

-#include "interface.h"

+#include "netdissect.h"

#include "addrtoname.h"

#include "extract.h"

-#include "nlpid.h"

#define VTP_HEADER_LEN 36

#define VTP_DOMAIN_NAME_LEN 32

@@ -48,13 +44,13 @@

#define VTP_JOIN_MESSAGE 0x04

struct vtp_vlan_ {

- u_int8_t len;

- u_int8_t status;

- u_int8_t type;

- u_int8_t name_len;

- u_int16_t vlanid;

- u_int16_t mtu;

- u_int32_t index;

+ uint8_t len;

+ uint8_t status;

+ uint8_t type;

+ uint8_t name_len;

+ uint16_t vlanid;

+ uint16_t mtu;

+ uint32_t index;

};

static const struct tok vtp_message_type_values[] = {

@@ -121,7 +117,8 @@ static const struct tok vtp_stp_type_values[] = {

};

void

-vtp_print (const u_char *pptr, u_int length)

+vtp_print (netdissect_options *ndo,

+ const u_char *pptr, u_int length)

{

int type, len, tlv_len, tlv_value;

const u_char *tptr;

@@ -132,26 +129,26 @@ vtp_print (const u_char *pptr, u_int length)

tptr = pptr;

- if (!TTEST2(*tptr, VTP_HEADER_LEN))

- goto trunc;

+ ND_TCHECK2(*tptr, VTP_HEADER_LEN);

type = *(tptr+1);

- printf("VTPv%u, Message %s (0x%02x), length %u",

+ ND_PRINT((ndo, "VTPv%u, Message %s (0x%02x), length %u",

*tptr,

tok2str(vtp_message_type_values,"Unknown message type", type),

- *(tptr+1),

- length);

+ type,

+ length));

/* In non-verbose mode, just print version and message type */

- if (vflag < 1) {

+ if (ndo->ndo_vflag < 1) {

return;

}

/* verbose mode print all fields */

- printf("\n\tDomain name: %s, %s: %u",

- (tptr+4),

- tok2str(vtp_header_values,"Unknown",*(tptr+1)),

- *(tptr+2));

+ ND_PRINT((ndo, "\n\tDomain name: "));

+ fn_printzp(ndo, tptr + 4, *(tptr + 3), NULL);

+ ND_PRINT((ndo, ", %s: %u",

+ tok2str(vtp_header_values, "Unknown", type),

+ *(tptr+2)));

tptr += VTP_HEADER_LEN;

@@ -166,7 +163,7 @@ vtp_print (const u_char *pptr, u_int length)

* +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

* +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

- * | Management Domain Name |

+ * | Management Domain Name (zero-padded to 32 bytes) |

* +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

* | Configuration revision number |

* +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

@@ -179,20 +176,23 @@ vtp_print (const u_char *pptr, u_int length)

- printf("\n\t Config Rev %x, Updater %s",

+ ND_TCHECK2(*tptr, 8);

+ ND_PRINT((ndo, "\n\t Config Rev %x, Updater %s",

EXTRACT_32BITS(tptr),

- ipaddr_string(tptr+4));

+ ipaddr_string(ndo, tptr+4)));

tptr += 8;

- printf(", Timestamp 0x%08x 0x%08x 0x%08x",

+ ND_TCHECK2(*tptr, VTP_UPDATE_TIMESTAMP_LEN);

+ ND_PRINT((ndo, ", Timestamp 0x%08x 0x%08x 0x%08x",

EXTRACT_32BITS(tptr),

EXTRACT_32BITS(tptr + 4),

- EXTRACT_32BITS(tptr + 8));

+ EXTRACT_32BITS(tptr + 8)));

tptr += VTP_UPDATE_TIMESTAMP_LEN;

- printf(", MD5 digest: %08x%08x%08x%08x",

+ ND_TCHECK2(*tptr, VTP_MD5_DIGEST_LEN);

+ ND_PRINT((ndo, ", MD5 digest: %08x%08x%08x%08x",

EXTRACT_32BITS(tptr),

EXTRACT_32BITS(tptr + 4),

EXTRACT_32BITS(tptr + 8),

- EXTRACT_32BITS(tptr + 12));

+ EXTRACT_32BITS(tptr + 12)));

tptr += VTP_MD5_DIGEST_LEN;

break;

@@ -205,7 +205,7 @@ vtp_print (const u_char *pptr, u_int length)

* +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

* +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

- * | Management Domain Name |

+ * | Management Domain Name (zero-padded to 32 bytes) |

* +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

* | Configuration revision number |

* +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

@@ -218,7 +218,7 @@ vtp_print (const u_char *pptr, u_int length)

- printf(", Config Rev %x", EXTRACT_32BITS(tptr));

+ ND_PRINT((ndo, ", Config Rev %x", EXTRACT_32BITS(tptr)));

* VLAN INFORMATION

@@ -242,17 +242,17 @@ vtp_print (const u_char *pptr, u_int length)

if (len == 0)

break;

- if (!TTEST2(*tptr, len))

- goto trunc;

+ ND_TCHECK2(*tptr, len);

- vtp_vlan = (struct vtp_vlan_*)tptr;

- printf("\n\tVLAN info status %s, type %s, VLAN-id %u, MTU %u, SAID 0x%08x, Name %s",

+ vtp_vlan = (const struct vtp_vlan_*)tptr;

+ ND_TCHECK(*vtp_vlan);

+ ND_PRINT((ndo, "\n\tVLAN info status %s, type %s, VLAN-id %u, MTU %u, SAID 0x%08x, Name ",

tok2str(vtp_vlan_status,"Unknown",vtp_vlan->status),

tok2str(vtp_vlan_type_values,"Unknown",vtp_vlan->type),

EXTRACT_16BITS(&vtp_vlan->vlanid),

EXTRACT_16BITS(&vtp_vlan->mtu),

- EXTRACT_32BITS(&vtp_vlan->index),

- (tptr + VTP_VLAN_INFO_OFFSET));

+ EXTRACT_32BITS(&vtp_vlan->index)));

+ fn_printzp(ndo, tptr + VTP_VLAN_INFO_OFFSET, vtp_vlan->name_len, NULL);

* Vlan names are aligned to 32-bit boundaries.

@@ -271,9 +271,9 @@ vtp_print (const u_char *pptr, u_int length)

type = *tptr;

tlv_len = *(tptr+1);

- printf("\n\t\t%s (0x%04x) TLV",

+ ND_PRINT((ndo, "\n\t\t%s (0x%04x) TLV",

tok2str(vtp_vlan_tlv_values, "Unknown", type),

- type);

+ type));

* infinite loop check

@@ -282,38 +282,37 @@ vtp_print (const u_char *pptr, u_int length)

return;

}

- if (!TTEST2(*tptr, tlv_len*2 +2))

- goto trunc;

+ ND_TCHECK2(*tptr, tlv_len * 2 +2);

tlv_value = EXTRACT_16BITS(tptr+2);

switch (type) {

case VTP_VLAN_STE_HOP_COUNT:

- printf(", %u", tlv_value);

+ ND_PRINT((ndo, ", %u", tlv_value));

break;

case VTP_VLAN_PRUNING:

- printf(", %s (%u)",

+ ND_PRINT((ndo, ", %s (%u)",

tlv_value == 1 ? "Enabled" : "Disabled",

- tlv_value);

+ tlv_value));

break;

case VTP_VLAN_STP_TYPE:

- printf(", %s (%u)",

+ ND_PRINT((ndo, ", %s (%u)",

tok2str(vtp_stp_type_values, "Unknown", tlv_value),

- tlv_value);

+ tlv_value));

break;

case VTP_VLAN_BRIDGE_TYPE:

- printf(", %s (%u)",

+ ND_PRINT((ndo, ", %s (%u)",

tlv_value == 1 ? "SRB" : "SRT",

- tlv_value);

+ tlv_value));

break;

case VTP_VLAN_BACKUP_CRF_MODE:

- printf(", %s (%u)",

+ ND_PRINT((ndo, ", %s (%u)",

tlv_value == 1 ? "Backup" : "Not backup",

- tlv_value);

+ tlv_value));

break;

@@ -327,7 +326,7 @@ vtp_print (const u_char *pptr, u_int length)

case VTP_VLAN_TRANS_BRIDGED_VLAN:

case VTP_VLAN_ARP_HOP_COUNT:

default:

- print_unknown_data(gndo,tptr, "\n\t\t ", 2 + tlv_len*2);

+ print_unknown_data(ndo, tptr, "\n\t\t ", 2 + tlv_len*2);

break;

}

len -= 2 + tlv_len*2;

@@ -345,14 +344,15 @@ vtp_print (const u_char *pptr, u_int length)

* +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

* +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

- * | Management Domain Name |

+ * | Management Domain Name (zero-padded to 32 bytes) |

* +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

* | Start value |

* +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

- printf("\n\tStart value: %u", EXTRACT_32BITS(tptr));

+ ND_TCHECK2(*tptr, 4);

+ ND_PRINT((ndo, "\n\tStart value: %u", EXTRACT_32BITS(tptr)));

break;

case VTP_JOIN_MESSAGE:

@@ -367,7 +367,7 @@ vtp_print (const u_char *pptr, u_int length)

return;

trunc:

- printf("[|vtp]");

+ ND_PRINT((ndo, "[|vtp]"));

}