Merged in %S fixes and XX_NOT_CHANGED fixes from 2.2.

Jeremy.

diff --git a/source/include/trans2.h b/source/include/trans2.h
index 067607e1cb032307c9f22c507119e252722a1460..7d35e9a5b61dd85cb4022436920149734b07855c 100644 (file)
--- a/source/include/trans2.h
+++ b/source/include/trans2.h
@@ -332,10 +332,10 @@ Byte offset   Type     name                description
 #define SMB_QUERY_FILE_UNIX_BASIC      0x200   /* UNIX File Info*/
 #define SMB_SET_FILE_UNIX_BASIC        0x200
 
-#define MODE_NO_CHANGE                 0xFFFFFFFF     /* file mode value which */
+#define SMB_MODE_NO_CHANGE                 0xFFFFFFFF     /* file mode value which */
                                               /* means "don't change it" */
-#define UID_NO_CHANGE                  0xFFFFFFFF
-#define GID_NO_CHANGE                  0xFFFFFFFF
+#define SMB_UID_NO_CHANGE                  0xFFFFFFFF
+#define SMB_GID_NO_CHANGE                  0xFFFFFFFF
 
 /*
 Offset Size         Name

diff --git a/source/lib/substitute.c b/source/lib/substitute.c
index 8e2ee3c0523ccecd9ff7726e7c13ad2be778deae..deef085c9947490219c0410d9a842e100ec88bad 100644 (file)
--- a/source/lib/substitute.c
+++ b/source/lib/substitute.c
@@ -35,6 +35,7 @@ extern pstring global_myname;
  Based on code by Branko Cibej <branko.cibej@hermes.si>
  When this is called p points at the '%' character.
 ********************************************************************/
+
 static size_t expand_env_var(char *p, int len)
 {
        fstring envname;
@@ -87,14 +88,19 @@ static size_t expand_env_var(char *p, int len)
  Patch from jkf@soton.ac.uk
  Added this to implement %p (NIS auto-map version of %H)
 *******************************************************************/
-static char *automount_path(char *user_name)
+
+static char *automount_path(const char *user_name)
 {
        static pstring server_path;
+       struct passwd *pass;
 
        /* use the passwd entry as the default */
        /* this will be the default if WITH_AUTOMOUNT is not used or fails */
-       /* pstrcpy() copes with get_user_home_dir() returning NULL */
-       pstrcpy(server_path, get_user_home_dir(user_name));
+
+       if (((pass = Get_Pwnam(user_name))!=NULL) && (pass->pw_dir != NULL))
+               pstrcpy(server_path, pass->pw_dir );
+       else
+               *server_path = '\0';
 
 #if (defined(HAVE_NETGROUP) && defined (WITH_AUTOMOUNT))
 
@@ -111,9 +117,8 @@ static char *automount_path(char *user_name)
                        }
                } else {
                        /* NIS key lookup failed: default to user home directory from password file */
-          pstrcpy(server_path, get_user_home_dir(user_name));
-                 DEBUG(5, ("NIS lookup failed. Using Home path from passwd file. Home path is: %s\n",
-                       server_path ));
+                       pstrcpy(server_path, get_user_home_dir(user_name));
+                       DEBUG(5, ("NIS lookup failed. Using Home path from passwd file. Home path is: %s\n", server_path ));
                }
        }
 #endif
@@ -123,36 +128,32 @@ static char *automount_path(char *user_name)
        return server_path;
 }
 
-
 /*******************************************************************
  Patch from jkf@soton.ac.uk
  This is Luke's original function with the NIS lookup code
  moved out to a separate function.
 *******************************************************************/
+
 static char *automount_server(const char *user_name)
 {
        static pstring server_name;
 
        /* use the local machine name as the default */
        /* this will be the default if WITH_AUTOMOUNT is not used or fails */
-       if (*local_machine) {
+       if (*local_machine)
                pstrcpy(server_name, local_machine);
-       } else {
+       else
                pstrcpy(server_name, global_myname);
-       }
        
 #if (defined(HAVE_NETGROUP) && defined (WITH_AUTOMOUNT))
 
-       if (lp_nis_home_map())
-       {
+       if (lp_nis_home_map()) {
                int home_server_len;
                char *automount_value = automount_lookup(user_name);
                home_server_len = strcspn(automount_value,":");
                DEBUG(5, ("NIS lookup succeeded.  Home server length: %d\n",home_server_len));
                if (home_server_len > sizeof(pstring))
-               {
                        home_server_len = sizeof(pstring);
-               }
                strncpy(server_name, automount_value, home_server_len);
                 server_name[home_server_len] = '\0';
        }
@@ -166,6 +167,7 @@ static char *automount_server(const char *user_name)
 /****************************************************************************
  Do some standard substitutions in a string.
 ****************************************************************************/
+
 void standard_sub_basic(const char *smb_name, char *str)
 {
        char *p, *s;
@@ -226,10 +228,10 @@ void standard_sub_basic(const char *smb_name, char *str)
        }
 }
 
-
 /****************************************************************************
  Do some standard substitutions in a string.
 ****************************************************************************/
+
 void standard_sub_advanced(int snum, const char *user, const char *connectpath, gid_t gid, const char *smb_name, char *str)
 {
        char *p, *s, *home;
@@ -286,14 +288,47 @@ void standard_sub_advanced(int snum, const char *user, const char *connectpath,
 /****************************************************************************
  Do some standard substitutions in a string.
 ****************************************************************************/
+
 void standard_sub_conn(connection_struct *conn, char *str)
 {
        standard_sub_advanced(SNUM(conn), conn->user, conn->connectpath, conn->gid, current_user_info.smb_name, str);
 }
 
 /****************************************************************************
-like standard_sub but by snum
+ Like standard_sub but for a homes share where snum still points to the [homes]
+ share. No user specific snum created yet so servicename should be the username.
 ****************************************************************************/
+
+void standard_sub_home(int snum, const char *user, char *str)
+{
+       char *p, *s;
+
+       for (s=str; (p=strchr_m(s, '%'));s=p) {
+               int l = sizeof(pstring) - (int)(p-str);
+               
+               switch (*(p+1)) {
+               case 'S': 
+                       string_sub(p,"%S", user, l); 
+                       break;
+               case 'p': 
+                       string_sub(p,"%p", automount_path(user), l); 
+                       break;
+               case '\0': 
+                       p++; 
+                       break; /* don't run off the end of the string */
+                       
+               default: p+=2; 
+                       break;
+               }
+       }
+
+        standard_sub_advanced(snum, user, "", -1, current_user_info.smb_name, str);
+}
+
+/****************************************************************************
+ Like standard_sub but by snum.
+****************************************************************************/
+
 void standard_sub_snum(int snum, char *str)
 {
        extern struct current_user current_user;
@@ -313,6 +348,7 @@ void standard_sub_snum(int snum, char *str)
 /*******************************************************************
  Substitute strings with useful parameters.
 ********************************************************************/
+
 void standard_sub_vuser(char *str, user_struct *vuser)
 {
        standard_sub_advanced(-1, vuser->user.unix_name, "", -1, current_user_info.smb_name, str);
@@ -321,6 +357,7 @@ void standard_sub_vuser(char *str, user_struct *vuser)
 /*******************************************************************
  Substitute strings with useful parameters.
 ********************************************************************/
+
 void standard_sub_vsnum(char *str, user_struct *vuser, int snum)
 {
        standard_sub_advanced(snum, vuser->user.unix_name, "", -1, current_user_info.smb_name, str);

diff --git a/source/lib/username.c b/source/lib/username.c
index 8154a2b40e4932a662a0cc985eff704f00f86bf9..c7e25b7cd129c4e0af3f6487aeb059b5939511ed 100644 (file)
--- a/source/lib/username.c
+++ b/source/lib/username.c
@@ -45,6 +45,13 @@ char *get_user_home_dir(const char *user)
        static struct passwd *pass;
        int snum;
 
+       /* Ensure the user exists. */
+
+       pass = Get_Pwnam(user);
+
+       if (!pass)
+               return(NULL);
+
        /* If a path is specified in [homes] then use it instead of the
           user's home directory from struct passwd. */
 
@@ -52,18 +59,14 @@ char *get_user_home_dir(const char *user)
                static pstring home_dir;
                
                pstrcpy(home_dir, lp_pathname(snum));
-               standard_sub_snum(snum, home_dir);
+               standard_sub_home(snum, user, home_dir);
 
                if (home_dir[0])
                        return home_dir;
        }
 
-       /* Get home directory from struct passwd. */
-
-       pass = Get_Pwnam(user);
+       /* Return home directory from struct passwd. */
 
-       if (!pass)
-               return(NULL);
        return(pass->pw_dir);      
 }
 

diff --git a/source/libsmb/clifile.c b/source/libsmb/clifile.c
index e282c9a6ab6290a472dd6091f96c158648f7630d..4b7c173a63ddc611f7d0621b148095758e130419 100644 (file)
--- a/source/libsmb/clifile.c
+++ b/source/libsmb/clifile.c
@@ -175,7 +175,7 @@ static BOOL cli_unix_chmod_chown_internal(struct cli_state *cli, const char *fna
 BOOL cli_unix_chmod(struct cli_state *cli, const char *fname, mode_t mode)
 {
        return cli_unix_chmod_chown_internal(cli, fname, 
-               unix_perms_to_wire(mode), UID_NO_CHANGE, GID_NO_CHANGE);
+               unix_perms_to_wire(mode), SMB_UID_NO_CHANGE, SMB_GID_NO_CHANGE);
 }
 
 /****************************************************************************
@@ -184,7 +184,7 @@ BOOL cli_unix_chmod(struct cli_state *cli, const char *fname, mode_t mode)
 
 BOOL cli_unix_chown(struct cli_state *cli, const char *fname, uid_t uid, gid_t gid)
 {
-       return cli_unix_chmod_chown_internal(cli, fname, MODE_NO_CHANGE, (uint32)uid, (uint32)gid);
+       return cli_unix_chmod_chown_internal(cli, fname, SMB_MODE_NO_CHANGE, (uint32)uid, (uint32)gid);
 }
 
 /****************************************************************************