keep a pointer to the CVSENTRIES in the CVSFILE structure.

ok xsa@

1 files changed, 24 insertions, 19 deletions

diff --git a/usr.bin/cvs/file.c b/usr.bin/cvs/file.c
index bd3c98c0165..5774b78515b 100644
--- a/usr.bin/cvs/file.c
+++ b/usr.bin/cvs/file.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: file.c,v 1.114 2005/08/17 08:35:53 xsa Exp $	*/
+/*	$OpenBSD: file.c,v 1.115 2005/08/17 16:23:19 joris Exp $	*/
 /*
  * Copyright (c) 2004 Jean-Francois Brousseau <jfb@openbsd.org>
  * All rights reserved.
@@ -114,7 +114,8 @@ static int	 cvs_file_sort(struct cvs_flist *, u_int);
 static int	 cvs_file_cmp(const void *, const void *);
 static int	 cvs_file_cmpname(const char *, const char *);
 static CVSFILE	*cvs_file_alloc(const char *, u_int);
-static CVSFILE	*cvs_file_lget(const char *, int, CVSFILE *, struct cvs_ent *);
+static CVSFILE	*cvs_file_lget(const char *, int, CVSFILE *, CVSENTRIES *,
+		    struct cvs_ent *);
 
 
 /*
@@ -392,7 +393,7 @@ cvs_file_getspec(char **fspec, int fsn, int flags, int (*cb)(CVSFILE *, void *),
 	 * Fetch the needed information about ".", so we can setup a few
 	 * things to get ourselfs going.
 	 */
-	cf = cvs_file_lget(".", 0, NULL, NULL);
+	cf = cvs_file_lget(".", 0, NULL, NULL, NULL);
 	if (cf == NULL) {
 		cvs_log(LP_ERR, "arrrr i failed captain!");
 		return (-1);
@@ -523,7 +524,7 @@ cvs_file_loadinfo(char *path, int flags, int (*cb)(CVSFILE *, void *),
 	 * There might not be an Entries file, so do not fail if there
 	 * is none available to get the info from.
 	 */
-	entf = cvs_ent_open(parent, O_RDONLY);
+	entf = cvs_ent_open(parent, O_RDWR);
 
 	/*
 	 * Load the Entry if we successfully opened the Entries file.
@@ -552,9 +553,9 @@ cvs_file_loadinfo(char *path, int flags, int (*cb)(CVSFILE *, void *),
 	 * <path> for a directory.
 	 */
 	if (type == DT_DIR)
-		base = cvs_file_lget(path, flags, NULL, ent);
+		base = cvs_file_lget(path, flags, NULL, entf, ent);
 	else
-		base = cvs_file_lget(parent, flags, NULL, NULL);
+		base = cvs_file_lget(parent, flags, NULL, entf, NULL);
 
 	if (base == NULL) {
 		cvs_log(LP_ERR, "failed to obtain directory info for '%s'",
@@ -582,7 +583,7 @@ cvs_file_loadinfo(char *path, int flags, int (*cb)(CVSFILE *, void *),
 	 * to the base.
 	 */
 	if (type != DT_DIR) {
-		cf = cvs_file_lget(path, flags, base, ent);
+		cf = cvs_file_lget(path, flags, base, entf, ent);
 		if (cf == NULL) {
 			cvs_error = CVS_EX_DATA;
 			goto fail;
@@ -591,11 +592,6 @@ cvs_file_loadinfo(char *path, int flags, int (*cb)(CVSFILE *, void *),
 		cvs_file_attach(base, cf);
 	}
 
-	if (entf != NULL) {
-		cvs_ent_close(entf);
-		entf = NULL;
-	}
-
 	/*
 	 * Always pass the base directory, unless:
 	 * - we are running in server or local mode and the path is not "."
@@ -608,6 +604,11 @@ cvs_file_loadinfo(char *path, int flags, int (*cb)(CVSFILE *, void *),
 	    ((cvs_error = cb(base, arg)) != CVS_EX_OK))
 		goto fail;
 
+	if (entf != NULL) {
+		cvs_ent_close(entf);
+		entf = NULL;
+	}
+
 	/*
 	 * If we have a normal file, pass it as well.
 	 */
@@ -830,7 +831,7 @@ cvs_file_getdir(CVSFILE *cf, int flags, int (*cb)(CVSFILE *, void *),
 	}
 
 	ret = -1;
-	entf = cvs_ent_open(fpath, O_RDONLY);
+	entf = cvs_ent_open(fpath, O_RDWR);
 	while ((de = readdir(dp)) != NULL) {
 		if (!strcmp(de->d_name, ".") ||
 		    !strcmp(de->d_name, ".."))
@@ -863,7 +864,7 @@ cvs_file_getdir(CVSFILE *cf, int flags, int (*cb)(CVSFILE *, void *),
 		else
 			ent = NULL;
 
-		cfp = cvs_file_lget(pbuf, flags, cf, ent);
+		cfp = cvs_file_lget(pbuf, flags, cf, entf, ent);
 		if (cfp == NULL) {
 			cvs_log(LP_ERR, "failed to get '%s'", pbuf);
 			goto done;
@@ -923,7 +924,7 @@ cvs_file_getdir(CVSFILE *cf, int flags, int (*cb)(CVSFILE *, void *),
 		if (len >= sizeof(pbuf))
 			goto done;
 
-		cfp = cvs_file_lget(pbuf, flags, cf, ent);
+		cfp = cvs_file_lget(pbuf, flags, cf, entf, ent);
 		if (cfp == NULL) {
 			cvs_log(LP_ERR, "failed to fetch '%s'", pbuf);
 			goto done;
@@ -1180,7 +1181,8 @@ cvs_file_alloc(const char *path, u_int type)
  * failure.
  */
 static CVSFILE *
-cvs_file_lget(const char *path, int flags, CVSFILE *parent, struct cvs_ent *ent)
+cvs_file_lget(const char *path, int flags, CVSFILE *parent, CVSENTRIES *pent,
+    struct cvs_ent *ent)
 {
 	int ret;
 	u_int type;
@@ -1195,6 +1197,7 @@ cvs_file_lget(const char *path, int flags, CVSFILE *parent, struct cvs_ent *ent)
 	if ((cfp = cvs_file_alloc(path, type)) == NULL)
 		return (NULL);
 	cfp->cf_parent = parent;
+	cfp->cf_entry = pent;
 
 	if ((cfp->cf_type == DT_DIR) && (cfp->cf_parent == NULL))
 		cfp->cf_flags |= CVS_DIRF_BASE;
@@ -1276,9 +1279,11 @@ cvs_file_lget(const char *path, int flags, CVSFILE *parent, struct cvs_ent *ent)
 		}
 	}
 
-	if ((cfp->cf_type == DT_DIR) && (cvs_load_dirinfo(cfp, flags) < 0)) {
-		cvs_file_free(cfp);
-		return (NULL);
+	if (cfp->cf_type == DT_DIR) {
+		if (cvs_load_dirinfo(cfp, flags) < 0) {
+			cvs_file_free(cfp);
+			return (NULL);
+		}
 	}
 
 	if ((cfp->cf_repo != NULL) && (cfp->cf_type == DT_DIR) &&