Tighten the dump-glossary test.
1. The only indication we get of an out-of-order value string is a message on STDERR, so check that and fail the test if STDERR wasn't empty. 2. This exposes an out-of-order value string in packet-stun.c; fix it. 3. This triggered the pre-commit hook on packet-stun.c, which noticed an API error (ENC_ASCII -> ENC_ASCII|ENC_NA); fix that too. Change-Id: I36f87a2a87b40537119562f22a7e3012716ff239 Lesson: automated testing/tooling is both wonderful and scary. Reviewed-on: https://code.wireshark.org/review/2682 Reviewed-by: Evan Huus <eapache@gmail.com>
This commit is contained in:
parent
8d2eaaab51
commit
a87e292272
|
@ -325,8 +325,8 @@ static const value_string attributes[] = {
|
|||
{ICE_CONTROLLING , "ICE-CONTROLLING"},
|
||||
{RESPONSE_ORIGIN , "RESPONSE-ORIGIN"},
|
||||
{OTHER_ADDRESS , "OTHER-ADDRESS"},
|
||||
{MS_CANDIDATE_IDENTIFIER, "MS-CANDIDATE-IDENTIFIER"},
|
||||
{MS_SEQUENCE_NUMBER , "MS-SEQUENCE-NUMBER"},
|
||||
{MS_CANDIDATE_IDENTIFIER, "MS-CANDIDATE-IDENTIFIER"},
|
||||
{MS_SERVICE_QUALITY , "MS-SERVICE-QUALITY"},
|
||||
{BANDWIDTH_ACM , "Bandwidth Admission Control Message"},
|
||||
{BANDWIDTH_RSV_ID , "Bandwidth Reservation Identifier"},
|
||||
|
@ -1242,7 +1242,7 @@ dissect_stun_message(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, gboole
|
|||
proto_tree_add_item(att_tree, hf_stun_att_reserved, tvb, offset+3, 1, ENC_NA);
|
||||
break;
|
||||
case MS_CANDIDATE_IDENTIFIER:
|
||||
proto_tree_add_item(att_tree, hf_stun_att_ms_foundation, tvb, offset, 4, ENC_ASCII);
|
||||
proto_tree_add_item(att_tree, hf_stun_att_ms_foundation, tvb, offset, 4, ENC_ASCII|ENC_NA);
|
||||
break;
|
||||
|
||||
default:
|
||||
|
|
|
@ -278,10 +278,13 @@ clopts_step_valid_name_resolving() {
|
|||
}
|
||||
|
||||
test_dump_glossary() {
|
||||
$TSHARK -G $1 > /dev/null
|
||||
$TSHARK -G $1 > /dev/null 2> ./testout.txt
|
||||
RETURNVALUE=$?
|
||||
if [ ! $RETURNVALUE -eq $EXIT_OK ]; then
|
||||
test_step_failed "exit status: $RETURNVALUE"
|
||||
elif [ -s ./testout.txt ]; then
|
||||
test_step_output_print ./testout.txt
|
||||
test_step_failed "Error messages on stderr"
|
||||
else
|
||||
test_step_ok
|
||||
fi
|
||||
|
|
Loading…
Reference in New Issue