diff options
author | Pascal Quantin <pascal.quantin@gmail.com> | 2015-06-07 16:01:30 +0200 |
---|---|---|
committer | Pascal Quantin <pascal.quantin@gmail.com> | 2015-06-07 14:35:07 +0000 |
commit | 83351a9996b04a5b3b2e7f3f705ffc63d13a0b53 (patch) | |
tree | d0cdfdc1935607792408ad318a00ab09bd981f44 /ui/gtk/iax2_analysis.c | |
parent | 6e58ae3dee5f4e389f4f915c89a790cd09c4dd3a (diff) | |
download | wireshark-83351a9996b04a5b3b2e7f3f705ffc63d13a0b53.tar.gz |
iax2_analysis.c: fix V610 warning reported by PVS-Studio
Unspecified behavior. Check the shift operator '>>'. The left operand '(- 1)' is negative.
Change-Id: Icdb999c2df0e7843086ec811ca4747876fbf2424
Reviewed-on: https://code.wireshark.org/review/8809
Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
Diffstat (limited to 'ui/gtk/iax2_analysis.c')
-rw-r--r-- | ui/gtk/iax2_analysis.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ui/gtk/iax2_analysis.c b/ui/gtk/iax2_analysis.c index 3297601d55..2894f0430b 100644 --- a/ui/gtk/iax2_analysis.c +++ b/ui/gtk/iax2_analysis.c @@ -2420,7 +2420,7 @@ static gboolean copy_file(gchar *dest, gint channels, gint format, user_data_t * return FALSE; } /* total length, it is permited to set this to 0xffffffff */ - phton32(pd, -1); + phton32(pd, 0xffffffff); fwritten = ws_write(to_fd, pd, 4); if ((fwritten < 4) || (fread_cnt == (size_t)-1)) { ws_close(forw_fd); |