Fixed mausb valid bit & added invalid_valid TFS

Fixed valid bit check for MAUSB EPHandleResp packet. the valid bit
is active low (ie: valid = 0, invalid = 1).

I noticed Some other dissectors also defined an tfs_invalid_valid,
so got rid of their local definitions & added it to epan/tfs.c.

Change-Id: Ifd78325f9c7c046224073fc9b29e0cc60dc5c286
Reviewed-on: https://code.wireshark.org/review/1766
Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>

5 files changed, 3 insertions, 3 deletions

diff --git a/epan/dissectors/packet-iec104.c b/epan/dissectors/packet-iec104.c
index 3c84bce3c5..9afac8d5d2 100644
--- a/epan/dissectors/packet-iec104.c
+++ b/epan/dissectors/packet-iec104.c
@@ -515,7 +515,6 @@ static const value_string qoi_r_types[] = {
 static const true_false_string tfs_blocked_not_blocked = { "Blocked", "Not blocked" };
 static const true_false_string tfs_substituted_not_substituted = { "Substituted", "Not Substituted" };
 static const true_false_string tfs_not_topical_topical = { "Not Topical", "Topical" };
-static const true_false_string tfs_invalid_valid = { "Invalid", "Valid" };
 static const true_false_string tfs_transient_not_transient = { "Transient", "Not Transient" };
 static const true_false_string tfs_overflow_no_overflow = { "Overflow", "No overflow" };
 static const true_false_string tfs_select_execute = { "Select", "Execute" };
diff --git a/epan/dissectors/packet-mausb.c b/epan/dissectors/packet-mausb.c
index d4f536930d..80f1dc0af6 100644
--- a/epan/dissectors/packet-mausb.c
+++ b/epan/dissectors/packet-mausb.c
@@ -1372,7 +1372,7 @@ proto_register_mausb(void)
         },
         { &hf_mausb_ep_handle_resp_valid,
             { "Valid", "mausb.ep_valid", FT_BOOLEAN, 6,
-              TFS(&tfs_valid_not_valid), MAUSB_EP_HANDLE_RESP_VALID_MASK,
+              TFS(&tfs_invalid_valid), MAUSB_EP_HANDLE_RESP_VALID_MASK,
               NULL, HFILL
             }
         },
diff --git a/epan/dissectors/packet-ppi.c b/epan/dissectors/packet-ppi.c
index 5237b7c68f..e4bb771926 100644
--- a/epan/dissectors/packet-ppi.c
+++ b/epan/dissectors/packet-ppi.c
@@ -343,7 +343,6 @@ static dissector_handle_t ppi_fnet_handle;
 static const true_false_string tfs_ppi_head_flag_alignment = { "32-bit aligned", "Not aligned" };
 static const true_false_string tfs_tsft_ms = { "milliseconds", "microseconds" };
 static const true_false_string tfs_ht20_40 = { "HT40", "HT20" };
-static const true_false_string tfs_invalid_valid = { "Invalid", "Valid" };
 static const true_false_string tfs_phy_error = { "PHY error", "No errors"};
 
 static const value_string vs_ppi_field_type[] = {
diff --git a/epan/tfs.c b/epan/tfs.c
index 8dbe252b99..d9b878b1a0 100644
--- a/epan/tfs.c
+++ b/epan/tfs.c
@@ -66,6 +66,7 @@ const true_false_string tfs_detected_not_detected = { "Detected", "Not Detected"
 const true_false_string tfs_available_not_available = { "Available", "Not available" };
 const true_false_string tfs_shared_independent = { "Shared", "Independent" };
 const true_false_string tfs_valid_invalid = { "Valid", "Invalid" };
+const true_false_string tfs_invalid_valid = { "Invalid", "Valid" };
 const true_false_string tfs_group_unique_name = { "Group name", "Unique name" };
 const true_false_string tfs_inuse_not_inuse = { "In use", "Not in use" };
 const true_false_string tfs_critical_not_critical = { "Critical", "Not critical" };
diff --git a/epan/tfs.h b/epan/tfs.h
index 23ff9ab9fc..6afebb4e28 100644
--- a/epan/tfs.h
+++ b/epan/tfs.h
@@ -77,6 +77,7 @@ WS_DLL_PUBLIC const true_false_string tfs_detected_not_detected;
 WS_DLL_PUBLIC const true_false_string tfs_available_not_available;
 WS_DLL_PUBLIC const true_false_string tfs_shared_independent;
 WS_DLL_PUBLIC const true_false_string tfs_valid_invalid;
+WS_DLL_PUBLIC const true_false_string tfs_invalid_valid;
 WS_DLL_PUBLIC const true_false_string tfs_group_unique_name;
 WS_DLL_PUBLIC const true_false_string tfs_inuse_not_inuse;
 WS_DLL_PUBLIC const true_false_string tfs_critical_not_critical;