diff options
| author | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2012-05-03 14:00:56 +0000 |
|---|---|---|
| committer | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2012-05-03 14:00:56 +0000 |
| commit | 76f582c58ddd3ad1becb569c34ada6dc8199de5a (patch) | |
| tree | 7952d8107f20d9f9ac5fb78510836224ab4ef75f | |
| parent | 33646d34115e359cb4afb0afb89747823d98be22 (diff) | |
| download | wireshark-76f582c58ddd3ad1becb569c34ada6dc8199de5a.tar.gz wireshark-76f582c58ddd3ad1becb569c34ada6dc8199de5a.tar.bz2 wireshark-76f582c58ddd3ad1becb569c34ada6dc8199de5a.zip | |
Fix logic bug. I'm sure this was something Coverity found and I fixed but forgot to commit it. Unfortunately Coverity has upgraded to Scan 5.5 and things are different now so I can't cross reference which CID this was anymore.
svn path=/trunk/; revision=42395
| -rw-r--r-- | epan/dissectors/packet-rtp-midi.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/epan/dissectors/packet-rtp-midi.c b/epan/dissectors/packet-rtp-midi.c index 104a5b8849..719e784da3 100644 --- a/epan/dissectors/packet-rtp-midi.c +++ b/epan/dissectors/packet-rtp-midi.c @@ -3944,7 +3944,7 @@ decode_sysex_common_nrt_sd_ext( tvbuff_t *tvb, packet_info *pinfo _U_, proto_tre data_len -= 2; */ consumed += 2; - }; + } return consumed; } @@ -3971,7 +3971,7 @@ decode_sysex_common_nrt_gi( tvbuff_t *tvb, packet_info *pinfo _U_, proto_tree *t /* This is the only sub-command that we know that we can decode, * the Identity Request has already been decoded up to this point. */ - if ( sub_id != RTP_MIDI_SYSEX_COMMON_NRT_GI_IDENTITY_REQUEST ) { + if ( sub_id == RTP_MIDI_SYSEX_COMMON_NRT_GI_IDENTITY_REQUEST ) { /* nothing more to do... */ @@ -4026,7 +4026,7 @@ decode_sysex_common_nrt_gi( tvbuff_t *tvb, packet_info *pinfo _U_, proto_tree *t offset +=4 ; data_len -= 4;*/ consumed += 4; - }; + } return consumed; } |