converted a couple more functions to use a fd instead of a FILE*

added a new utility fn file_lines_slashcont() which is used to handle
files that treat a \ followed by a newline as a blank
(This used to be commit 384ecd9d66ccd31ee85000c0ca55d413d8f2cc53)

2 files changed, 17 insertions, 15 deletions

diff --git a/source3/smbd/groupname.c b/source3/smbd/groupname.c
index 4dadfaa939..fcbf5dd778 100644
--- a/source3/smbd/groupname.c
+++ b/source3/smbd/groupname.c
@@ -71,9 +71,8 @@ void load_groupname_map(void)
   static BOOL initialized = False;
   char *groupname_map_file = lp_groupname_map();
   SMB_STRUCT_STAT st;
-  FILE *fp;
-  char *s;
-  pstring buf;
+  char **lines;
+  int i;
   groupname_map_entry *new_ep;
 
   if(!initialized) {
@@ -102,12 +101,13 @@ void load_groupname_map(void)
    * Load the file.
    */
 
-  fp = sys_fopen(groupname_map_file,"r");
-  if (!fp) {
+  lines = file_lines_load(groupname_map_file,NULL);
+  if (!lines) {
     DEBUG(0,("load_groupname_map: can't open groupname map %s. Error was %s\n",
           groupname_map_file, strerror(errno)));
     return;
   }
+  file_lines_slashcont(lines);
 
   /*
    * Throw away any previous list.
@@ -116,11 +116,12 @@ void load_groupname_map(void)
 
   DEBUG(4,("load_groupname_map: Scanning groupname map %s\n",groupname_map_file));
 
-  while((s=fgets_slash(buf,sizeof(buf),fp))!=NULL) {
+  for (i=0; lines[i]; i++) {
     pstring unixname;
     pstring windows_name;
     struct group *gptr;
     DOM_SID tmp_sid;
+    char *s = lines[i];
 
     DEBUG(10,("load_groupname_map: Read line |%s|\n", s));
 
@@ -202,7 +203,7 @@ Error was %s.\n", unixname, strerror(errno) ));
   DEBUG(10,("load_groupname_map: Added %ld entries to groupname map.\n",
 	    ubi_slCount(&groupname_map_list)));
            
-  fclose(fp);
+  file_lines_free(lines);
 }
 
 /***********************************************************
diff --git a/source3/smbd/password.c b/source3/smbd/password.c
index 208dbd2bff..f92d31718c 100644
--- a/source3/smbd/password.c
+++ b/source3/smbd/password.c
@@ -828,15 +828,16 @@ allows this user from this machine
 ****************************************************************************/
 static BOOL check_user_equiv(char *user, char *remote, char *equiv_file)
 {
-  pstring buf;
   int plus_allowed = 1;
   char *file_host;
   char *file_user;
-  FILE *fp = sys_fopen(equiv_file, "r");
+  char **lines = file_lines_load(equiv_file, NULL);
+  int i;
+
   DEBUG(5, ("check_user_equiv %s %s %s\n", user, remote, equiv_file));
-  if (! fp) return False;
-  while(fgets(buf, sizeof(buf), fp)) 
-  {
+  if (! lines) return False;
+  for (i=0; lines[i]; i++) {
+    char *buf = lines[i];
     trim_string(buf," "," ");
 
     if (buf[0] != '#' && buf[0] != '\n') 
@@ -857,7 +858,7 @@ static BOOL check_user_equiv(char *user, char *remote, char *equiv_file)
 	  {
 	    /* a bare plus means everbody allowed */
 	    DEBUG(6, ("check_user_equiv everybody allowed\n"));
-	    fclose(fp);
+	    file_lines_free(lines);
 	    return True;
 	  }
 	}
@@ -912,17 +913,17 @@ static BOOL check_user_equiv(char *user, char *remote, char *equiv_file)
 	  /* is it this user */
 	  if (file_user == 0 || strequal(user, file_user)) 
 	    {
-	      fclose(fp);
 	      DEBUG(5, ("check_user_equiv matched %s%s %s\n",
 			(plus ? "+" : "-"), file_host,
 			(file_user ? file_user : "")));
+	      file_lines_free(lines);
 	      return (plus ? True : False);
 	    }
 	}
       }
     }
   }
-  fclose(fp);
+  file_lines_free(lines);
   return False;
 }