From: Andrew Deason <adeason@sinenomine.net>
Date: Thu, 18 Feb 2010 20:41:24 +0000 (-0600)
Subject: VLockFile: init fd to INVALID_FD, not zero
X-Git-Tag: openafs-devel-1_5_73~137
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=c43625d91bc104759666629e5a78dff7f74a14fd;p=packages%2Fo%2Fopenafs.git

VLockFile: init fd to INVALID_FD, not zero

Zero is a valid file descriptor; initialize lockfile fd to INVALID_FD
instead, so we know it's invalid.

Change-Id: Iacb549ec4ed821763d20cb0e4f25882b4151bf3a
Reviewed-on: http://gerrit.openafs.org/1345
Tested-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
---

diff --git a/src/vol/vutil.c b/src/vol/vutil.c
index 8b28f9c95..8f7aefe2b 100644
--- a/src/vol/vutil.c
+++ b/src/vol/vutil.c
@@ -850,8 +850,9 @@ void
 VLockFileInit(struct VLockFile *lf, const char *path)
 {
     memset(lf, 0, sizeof(*lf));
-    assert(pthread_mutex_init(&lf->mutex, NULL) == 0);
     lf->path = strdup(path);
+    lf->fd = INVALID_FD;
+    assert(pthread_mutex_init(&lf->mutex, NULL) == 0);
 }
 
 # ifdef AFS_NT40_ENV