commit 4ed7cd3ab07f7c721daf4241fe1dac306fefd1fb
author Brandon Casey <casey@nrlssc.navy.mil> Wed Jan 16 13:12:46 2008 -0600
committer Junio C Hamano <gitster@pobox.com> Wed Jan 16 15:35:35 2008 -0800
tree e3b03a59cd0bca192fb396522f813210451404b8
parent d6cf61bfd4bccffcc8b095f8469dbe749d70abdf

Improve use of lockfile API

Remove remaining double close(2)'s.  i.e. close() before
commit_locked_index() or commit_lock_file().

Signed-off-by: Junio C Hamano <gitster@pobox.com>

refs.c

diff --git a/refs.c b/refs.c
index c3ffe03..8a42d39 100644
--- a/refs.c
+++ b/refs.c
@@ -864,7 +864,6 @@
 			die("too long a refname '%s'", list->name);
 		write_or_die(fd, line, len);
 	}
-	close(fd);
 	return commit_lock_file(&packlock);
 }
 
@@ -1021,12 +1020,9 @@
 
 void unlock_ref(struct ref_lock *lock)
 {
-	if (lock->lock_fd >= 0) {
-		close(lock->lock_fd);
-		/* Do not free lock->lk -- atexit() still looks at them */
-		if (lock->lk)
-			rollback_lock_file(lock->lk);
-	}
+	/* Do not free lock->lk -- atexit() still looks at them */
+	if (lock->lk)
+		rollback_lock_file(lock->lk);
 	free(lock->ref_name);
 	free(lock->orig_ref_name);
 	free(lock);
@@ -1151,7 +1147,7 @@
 	}
 	if (write_in_full(lock->lock_fd, sha1_to_hex(sha1), 40) != 40 ||
 	    write_in_full(lock->lock_fd, &term, 1) != 1
-		|| close(lock->lock_fd) < 0) {
+		|| close_lock_file(lock->lk) < 0) {
 		error("Couldn't write %s", lock->lk->filename);
 		unlock_ref(lock);
 		return -1;