diff options
| author | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2013-07-16 16:23:31 +0000 |
|---|---|---|
| committer | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2013-07-16 16:23:31 +0000 |
| commit | 91ecc2334f2a0de5eb3fb2b6cc97c2948e9533f9 (patch) | |
| tree | 4d4712ddf3b3953e9bee3f69c853b4cb68c998a9 /epan/dissectors/packet-infiniband_sdp.c | |
| parent | bb058987208733aaaf5aaeac74b0fadcbcd9918c (diff) | |
"man inet_pton" only says that it returns a negative value, zero or a postive value, and not specifically -1, 0, or 1, so even though we know that's what wsutil's inet_pton will return, we shouldn't test against those exact values.
svn path=/trunk/; revision=50670
Diffstat (limited to 'epan/dissectors/packet-infiniband_sdp.c')
| -rw-r--r-- | epan/dissectors/packet-infiniband_sdp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-infiniband_sdp.c b/epan/dissectors/packet-infiniband_sdp.c index cfa269ff66..4bb5142e06 100644 --- a/epan/dissectors/packet-infiniband_sdp.c +++ b/epan/dissectors/packet-infiniband_sdp.c @@ -551,7 +551,7 @@ proto_reg_handoff_ib_sdp(void) SET_ADDRESS(&manual_addr[i], AT_IB, sizeof(guint16), manual_addr_data[i]); } } else { /* GID */ - if (! inet_pton(AF_INET6, gPREF_ID[i], manual_addr_data[i]) ) { + if (inet_pton(AF_INET6, gPREF_ID[i], manual_addr_data[i]) <= 0) { error_occured = TRUE; } else { SET_ADDRESS(&manual_addr[i], AT_IB, GID_SIZE, manual_addr_data[i]); |