s3:torture: Fix uint64_t comparsion in if-clause
authorAndreas Schneider <asn@samba.org>
Fri, 17 Feb 2017 08:45:33 +0000 (09:45 +0100)
committerJeremy Allison <jra@samba.org>
Thu, 23 Feb 2017 02:18:10 +0000 (03:18 +0100)
Found by covscan.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=12592

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
source3/torture/torture.c

index 846b67526611dd8d6013a1351bd499d5d8790b48..30621222db792672f8a663bac370355c7e9dad86 100644 (file)
@@ -5066,7 +5066,7 @@ static bool run_rename_access(int dummy)
        }
 
        if (cli) {
-               if (fnum != -1) {
+               if (fnum != (uint64_t)-1) {
                        cli_close(cli, fnum);
                }
                cli_unlink(cli, src,
@@ -5457,7 +5457,7 @@ static bool run_owner_rights(int dummy)
   fail:
 
        if (cli) {
-               if (fnum != -1) {
+               if (fnum != (uint16_t)-1) {
                        cli_close(cli, fnum);
                }
                cli_unlink(cli, fname,