diff options
author | Pascal Quantin <pascal.quantin@gmail.com> | 2015-06-09 22:05:55 +0200 |
---|---|---|
committer | Pascal Quantin <pascal.quantin@gmail.com> | 2015-06-09 20:20:55 +0000 |
commit | 0d471c3d882a20ad41a6d52bd0e7b210b2f92334 (patch) | |
tree | e8ca8ad604660bf27c41b978ef463ec2d54c1d15 /ui/gtk | |
parent | 4e60e8fb390005db6c69e9b79fa9a01140fa0e44 (diff) |
GTK: fix a V610 warning reported by PVS-Studio
Unspecified behavior. Check the shift operator '>>'. The left operand '(- 1)' is negative.
Change-Id: I508d451736830a150b8839aa6353ee4c7bcf848f
Reviewed-on: https://code.wireshark.org/review/8859
Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
Diffstat (limited to 'ui/gtk')
-rw-r--r-- | ui/gtk/rtp_analysis.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ui/gtk/rtp_analysis.c b/ui/gtk/rtp_analysis.c index 4f6fbb9e29..02d3014029 100644 --- a/ui/gtk/rtp_analysis.c +++ b/ui/gtk/rtp_analysis.c @@ -2523,7 +2523,7 @@ copy_file(gchar *dest, gint channels, gint format, user_data_t *user_data) if (nchars != 4) goto copy_file_err; /* total length; it is permitted to set this to 0xffffffff */ - phton32(pd, -1); + phton32(pd, 0xffffffff); nchars = fwrite(pd, 1, 4, to_stream); if (nchars != 4) goto copy_file_err; |