From 83351a9996b04a5b3b2e7f3f705ffc63d13a0b53 Mon Sep 17 00:00:00 2001 From: Pascal Quantin Date: Sun, 7 Jun 2015 16:01:30 +0200 Subject: 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 --- ui/gtk/iax2_analysis.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'ui') 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); -- cgit v1.2.3