diff options
-rw-r--r-- | asn1/h225/h225.cnf | 8 | ||||
-rw-r--r-- | asn1/h245/h245.cnf | 14 | ||||
-rw-r--r-- | epan/dissectors/packet-h225.c | 8 | ||||
-rw-r--r-- | epan/dissectors/packet-h245.c | 14 | ||||
-rw-r--r-- | epan/dissectors/packet-per.c | 9 | ||||
-rw-r--r-- | epan/proto.h | 2 |
6 files changed, 29 insertions, 26 deletions
diff --git a/asn1/h225/h225.cnf b/asn1/h225/h225.cnf index 86b373b53a..142f648d28 100644 --- a/asn1/h225/h225.cnf +++ b/asn1/h225/h225.cnf @@ -213,7 +213,7 @@ CallIdentifier/guid guid if (h225_pi->is_destinationInfo == TRUE) { if (value_tvb) { len = tvb_length(value_tvb); - tvb_memcpy(value_tvb, h225_pi->dialedDigits, 0, len); + tvb_memcpy(value_tvb, (guint8*)h225_pi->dialedDigits, 0, len); } h225_pi->dialedDigits[len] = '\0'; h225_pi->is_destinationInfo = FALSE; @@ -223,10 +223,12 @@ CallIdentifier/guid guid #.FN_PARS H245TransportAddress/h245ipAddress/h245ipv4 VAL_PTR = &value_tvb #.FN_BODY H245TransportAddress/h245ipAddress/h245ipv4 - tvbuff_t *value_tvb; + tvbuff_t *value_tvb; + ipv4_address = 0; %(DEFAULT_BODY)s - tvb_memcpy(value_tvb, (char *)&ipv4_address, 0, 4); + if (value_tvb) + tvb_memcpy(value_tvb, (guint8*)&ipv4_address, 0, 4); #.END #---------------------------------------------------------------------------------------- #.FN_PARS H245TransportAddress/h245ipAddress/h245ipv4port diff --git a/asn1/h245/h245.cnf b/asn1/h245/h245.cnf index 162c8b4a21..1b7f77e942 100644 --- a/asn1/h245/h245.cnf +++ b/asn1/h245/h245.cnf @@ -276,14 +276,14 @@ OpenLogicalChannel VAL_PTR = &value_tvb #.FN_BODY UnicastAddress/iPAddress/network - tvbuff_t *value_tvb; + tvbuff_t *value_tvb; %(DEFAULT_BODY)s if ( media_channel ) - tvb_memcpy(value_tvb, (char *)&ipv4_address, 0, 4); + tvb_memcpy(value_tvb, (guint8*)&ipv4_address, 0, 4); if ( media_control_channel ) - tvb_memcpy(value_tvb, (char *)&rtcp_ipv4_address, 0, 4); + tvb_memcpy(value_tvb, (guint8*)&rtcp_ipv4_address, 0, 4); #.END #---------------------------------------------------------------------------------------- @@ -364,7 +364,7 @@ OpenLogicalChannel src_addr.type=AT_IPv4; src_addr.len=4; - src_addr.data=(char *)&ipv4_address; + src_addr.data=(guint8*)&ipv4_address; rtp_add_address(pinfo, &src_addr, ipv4_port, 0, "H245", pinfo->fd->num, NULL); } @@ -373,7 +373,7 @@ OpenLogicalChannel src_addr.type=AT_IPv4; src_addr.len=4; - src_addr.data=(char *)&rtcp_ipv4_address; + src_addr.data=(guint8*)&rtcp_ipv4_address; rtcp_add_address(pinfo, &src_addr, rtcp_ipv4_port, 0, "H245", pinfo->fd->num); } @@ -394,7 +394,7 @@ OpenLogicalChannel src_addr.type=AT_IPv4; src_addr.len=4; - src_addr.data=(char *)&ipv4_address; + src_addr.data=(guint8*)&ipv4_address; rtp_add_address(pinfo, &src_addr, ipv4_port, 0, "H245", pinfo->fd->num, NULL); } @@ -403,7 +403,7 @@ OpenLogicalChannel src_addr.type=AT_IPv4; src_addr.len=4; - src_addr.data=(char *)&rtcp_ipv4_address; + src_addr.data=(guint8*)&rtcp_ipv4_address; rtcp_add_address(pinfo, &src_addr, rtcp_ipv4_port, 0, "H245", pinfo->fd->num); } diff --git a/epan/dissectors/packet-h225.c b/epan/dissectors/packet-h225.c index d3966d321b..027715961e 100644 --- a/epan/dissectors/packet-h225.c +++ b/epan/dissectors/packet-h225.c @@ -1212,12 +1212,14 @@ static int dissect_protocolIdentifier(tvbuff_t *tvb, int offset, packet_info *pi static int dissect_h225_T_h245ipv4(tvbuff_t *tvb, int offset, packet_info *pinfo _U_, proto_tree *tree, int hf_index) { - tvbuff_t *value_tvb; + tvbuff_t *value_tvb; + ipv4_address = 0; offset = dissect_per_octet_string(tvb, offset, pinfo, tree, hf_index, 4, 4, &value_tvb); - tvb_memcpy(value_tvb, (char *)&ipv4_address, 0, 4); + if (value_tvb) + tvb_memcpy(value_tvb, (guint8*)&ipv4_address, 0, 4); return offset; } @@ -2244,7 +2246,7 @@ dissect_h225_DialedDigits(tvbuff_t *tvb, int offset, packet_info *pinfo _U_, pro if (h225_pi->is_destinationInfo == TRUE) { if (value_tvb) { len = tvb_length(value_tvb); - tvb_memcpy(value_tvb, h225_pi->dialedDigits, 0, len); + tvb_memcpy(value_tvb, (guint8*)h225_pi->dialedDigits, 0, len); } h225_pi->dialedDigits[len] = '\0'; h225_pi->is_destinationInfo = FALSE; diff --git a/epan/dissectors/packet-h245.c b/epan/dissectors/packet-h245.c index 6a717f6671..748b175f65 100644 --- a/epan/dissectors/packet-h245.c +++ b/epan/dissectors/packet-h245.c @@ -9587,16 +9587,16 @@ static int dissect_v76LogicalChannelParameters(tvbuff_t *tvb, int offset, packet static int dissect_h245_Ipv4_network(tvbuff_t *tvb, int offset, packet_info *pinfo _U_, proto_tree *tree, int hf_index) { - tvbuff_t *value_tvb; + tvbuff_t *value_tvb; offset = dissect_per_octet_string(tvb, offset, pinfo, tree, hf_index, 4, 4, &value_tvb); if ( media_channel ) - tvb_memcpy(value_tvb, (char *)&ipv4_address, 0, 4); + tvb_memcpy(value_tvb, (guint8*)&ipv4_address, 0, 4); if ( media_control_channel ) - tvb_memcpy(value_tvb, (char *)&rtcp_ipv4_address, 0, 4); + tvb_memcpy(value_tvb, (guint8*)&rtcp_ipv4_address, 0, 4); return offset; @@ -10176,7 +10176,7 @@ dissect_h245_OLC_rev_multiplexParameters(tvbuff_t *tvb, int offset, packet_info src_addr.type=AT_IPv4; src_addr.len=4; - src_addr.data=(char *)&ipv4_address; + src_addr.data=(guint8*)&ipv4_address; rtp_add_address(pinfo, &src_addr, ipv4_port, 0, "H245", pinfo->fd->num, NULL); } @@ -10185,7 +10185,7 @@ dissect_h245_OLC_rev_multiplexParameters(tvbuff_t *tvb, int offset, packet_info src_addr.type=AT_IPv4; src_addr.len=4; - src_addr.data=(char *)&rtcp_ipv4_address; + src_addr.data=(guint8*)&rtcp_ipv4_address; rtcp_add_address(pinfo, &src_addr, rtcp_ipv4_port, 0, "H245", pinfo->fd->num); } @@ -12959,7 +12959,7 @@ dissect_h245_T_forwardMultiplexAckParameters(tvbuff_t *tvb, int offset, packet_i src_addr.type=AT_IPv4; src_addr.len=4; - src_addr.data=(char *)&ipv4_address; + src_addr.data=(guint8*)&ipv4_address; rtp_add_address(pinfo, &src_addr, ipv4_port, 0, "H245", pinfo->fd->num, NULL); } @@ -12968,7 +12968,7 @@ dissect_h245_T_forwardMultiplexAckParameters(tvbuff_t *tvb, int offset, packet_i src_addr.type=AT_IPv4; src_addr.len=4; - src_addr.data=(char *)&rtcp_ipv4_address; + src_addr.data=(guint8*)&rtcp_ipv4_address; rtcp_add_address(pinfo, &src_addr, rtcp_ipv4_port, 0, "H245", pinfo->fd->num); } diff --git a/epan/dissectors/packet-per.c b/epan/dissectors/packet-per.c index 14adc2a7a4..af33564de9 100644 --- a/epan/dissectors/packet-per.c +++ b/epan/dissectors/packet-per.c @@ -292,7 +292,7 @@ dissect_per_restricted_character_string_sorted(tvbuff_t *tvb, guint32 offset, pa { guint32 length; gboolean byte_aligned; - char *buf; + guint8 *buf; guint char_pos; int bits_per_char; guint32 old_offset; @@ -405,7 +405,7 @@ DEBUG_ENTRY("dissect_per_restricted_character_string"); } } buf[char_pos]=0; - proto_tree_add_string(tree, hf_index, tvb, (old_offset>>3), (offset>>3)-(old_offset>>3), buf); + proto_tree_add_string(tree, hf_index, tvb, (old_offset>>3), (offset>>3)-(old_offset>>3), (char*)buf); if (value_tvb) { *value_tvb = tvb_new_real_data(buf, length, length); tvb_set_free_cb(*value_tvb, g_free); @@ -542,7 +542,6 @@ DEBUG_ENTRY("dissect_per_constrained_sequence_of"); /* 19.6 ub>=64k or unset */ if(max_len>=65536){ - guint32 start_offset=offset; /* semi-constrained whole number for number of elements */ /* each element encoded as 10.9 */ offset=dissect_per_length_determinant(tvb, offset, pinfo, parent_tree, hf_per_sequence_of_length, &length); @@ -1473,11 +1472,11 @@ DEBUG_ENTRY("dissect_per_octet_string"); it = proto_tree_add_uint(tree, hf_index, tvb, val_start, val_length, val_length); else it = proto_tree_add_int(tree, hf_index, tvb, val_start, val_length, val_length); - proto_item_append_text(it, (val_length == 1) ? " octet" : " octets"); + proto_item_append_text(it, plurality(val_length, " octet", " octets")); } else { if (pbytes) { if(IS_FT_STRING(hfi->type)){ - proto_tree_add_string(tree, hf_index, tvb, val_start, val_length, pbytes); + proto_tree_add_string(tree, hf_index, tvb, val_start, val_length, (char*)pbytes); } else if (hfi->type==FT_BYTES) { proto_tree_add_bytes(tree, hf_index, tvb, val_start, val_length, pbytes); } else { diff --git a/epan/proto.h b/epan/proto.h index f6944be66d..950687810f 100644 --- a/epan/proto.h +++ b/epan/proto.h @@ -143,7 +143,7 @@ typedef enum { BASE_HEX, /**< hexadecimal */ BASE_OCT, /**< octal */ BASE_DEC_HEX, /**< decimal (hexadecimal) */ - BASE_HEX_DEC, /**< hexadecimal (decimal) */ + BASE_HEX_DEC /**< hexadecimal (decimal) */ } base_display_e; #define IS_BASE_DUAL(b) ((b)==BASE_DEC_HEX||(b)==BASE_HEX_DEC) |