From 0d471c3d882a20ad41a6d52bd0e7b210b2f92334 Mon Sep 17 00:00:00 2001 From: Pascal Quantin Date: Tue, 9 Jun 2015 22:05:55 +0200 Subject: 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 --- ui/gtk/rtp_analysis.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'ui/gtk/rtp_analysis.c') 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; -- cgit v1.2.3