From d21dd16789458c07e63abe021f93f656dba4e52c Mon Sep 17 00:00:00 2001 From: Jeffrey Altman Date: Thu, 28 Oct 2010 00:40:32 -0400 Subject: [PATCH] vol: Always use INVALID_FD to indicate an invalid fd file descriptors on Windows are not ints and therefore cannot be safely compared against -1. Always use INVALID_FD which is -1 on UNIX and INVALID_HANDLE_VALUE on Windows. Change-Id: I8788f95817ee45367f04f4b90b4e6625c5a39dca Reviewed-on: http://gerrit.openafs.org/3178 Reviewed-by: Derrick Brashear Reviewed-by: Andrew Deason Tested-by: Derrick Brashear --- src/vol/partition.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/vol/partition.c b/src/vol/partition.c index 6f304bb4c..bc7fe761c 100644 --- a/src/vol/partition.c +++ b/src/vol/partition.c @@ -1227,7 +1227,7 @@ VLockPartition_r(char *name) if (!dp) return; /* no partition, will fail later */ - if (dp->lock_fd != -1) + if (dp->lock_fd != INVALID_FD) return; #if defined(AFS_SUN5_ENV) || defined(AFS_AIX41_ENV) @@ -1250,7 +1250,7 @@ VLockPartition_r(char *name) else dp->lock_fd = afs_open(partitionName, code); - if (dp->lock_fd != -1) + if (dp->lock_fd != INVALID_FD) break; if (errno == ENOENT) code |= O_CREAT; @@ -1314,7 +1314,7 @@ VUnlockPartition_r(char *name) if (!dp) return; /* no partition, will fail later */ close(dp->lock_fd); - dp->lock_fd = -1; + dp->lock_fd = INVALID_FD; } #endif /* AFS_NT40_ENV */ -- 2.39.5