diff options
author | Guy Harris <guy@alum.mit.edu> | 2004-01-06 21:11:04 +0000 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2004-01-06 21:11:04 +0000 |
commit | 77e43b09ea6642290a03002374fac8a6a30aaaa3 (patch) | |
tree | 4f2cd194fe6b4e2d6fed182b1ab75b29feffdc50 /packet-fw1.c | |
parent | 0b82bd5123645ff16db59f6180f6430b91fd4aa4 (diff) |
Don't add an interface to the interface list if we already have
MAX_INTERFACES interfaces in the list.
Make "found" a gboolean.
svn path=/trunk/; revision=9576
Diffstat (limited to 'packet-fw1.c')
-rw-r--r-- | packet-fw1.c | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/packet-fw1.c b/packet-fw1.c index 5795e29b61..e928ec9f3f 100644 --- a/packet-fw1.c +++ b/packet-fw1.c @@ -2,7 +2,7 @@ * Routines for Ethernet header disassembly of FW1 "monitor" files * Copyright 2002,2003, Alfred Koebler <ako@icon.de> * - * $Id: packet-fw1.c,v 1.9 2003/10/01 07:11:44 guy Exp $ + * $Id: packet-fw1.c,v 1.10 2004/01/06 21:11:04 guy Exp $ * * Ethereal - Network traffic analyzer * By Alfred Koebler <ako@icon.de> @@ -128,7 +128,7 @@ dissect_fw1(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) char header[1000]; char *p_header; int i; - int found; + gboolean found; #define MAX_INTERFACES 20 static char *p_interfaces[MAX_INTERFACES]; @@ -171,36 +171,36 @@ dissect_fw1(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) } /* Known interface name - if not, remember it */ - found=1; + found=FALSE; for (i=0; i<interface_anzahl && i<MAX_INTERFACES; i++) { if ( strcmp(p_interfaces[i], interface_name) == 0 ) { - found=0; + found=TRUE; } } - if (found == 1 ) { + if (!found && interface_anzahl < MAX_INTERFACES) { p_interfaces[interface_anzahl] = strdup(interface_name); interface_anzahl++; } /* display all interfaces always in the same order */ for (i=0; i<interface_anzahl; i++) { - found=1; + found=FALSE; if ( strcmp(p_interfaces[i], interface_name) == 0 ) { - found=0; + found=TRUE; } p_header = header + strlen(header); if (!fw1_iflist_with_chain) { sprintf(p_header, " %c %s %c", - found==0 ? (direction[0]=='i' ? 'i' : (direction[0]=='O' ? 'O' : ' ')) : ' ', + found ? (direction[0]=='i' ? 'i' : (direction[0]=='O' ? 'O' : ' ')) : ' ', p_interfaces[i], - found==0 ? (direction[0]=='I' ? 'I' : (direction[0]=='o' ? 'o' : ' ')) : ' ' + found ? (direction[0]=='I' ? 'I' : (direction[0]=='o' ? 'o' : ' ')) : ' ' ); } else { sprintf(p_header, " %c%c %s %c%c", - found==0 ? (direction[0]=='i' ? 'i' : (direction[0]=='O' ? 'O' : ' ')) : ' ', - found==0 ? (direction[0]=='i' ? chain[0] : (direction[0]=='O' ? chain[0] : ' ')) : ' ', + found ? (direction[0]=='i' ? 'i' : (direction[0]=='O' ? 'O' : ' ')) : ' ', + found ? (direction[0]=='i' ? chain[0] : (direction[0]=='O' ? chain[0] : ' ')) : ' ', p_interfaces[i], - found==0 ? (direction[0]=='I' ? 'I' : (direction[0]=='o' ? 'o' : ' ')) : ' ', - found==0 ? (direction[0]=='I' ? chain[0] : (direction[0]=='o' ? chain[0] : ' ')) : ' ' + found ? (direction[0]=='I' ? 'I' : (direction[0]=='o' ? 'o' : ' ')) : ' ', + found ? (direction[0]=='I' ? chain[0] : (direction[0]=='o' ? chain[0] : ' ')) : ' ' ); } } |