Minor change to bring smbmount in the main branch in line with some bzero
[samba.git] / source / client / smbmount.c
index 505552997b371cb5e5d468022872da2c25bccad6..00331f36819e1518d9bb1b58b1337870d295dc79 100644 (file)
@@ -19,9 +19,7 @@
    Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 */
 
-#ifdef SYSLOG
-#undef SYSLOG
-#endif
+#define NO_SYSLOG
 
 #include <linux/version.h>
 #define LVERSION(major,minor,patch) (((((major)<<8)+(minor))<<8)+(patch))
@@ -30,6 +28,8 @@
 #endif
 
 #include "includes.h"
+
+#include <asm/types.h>
 #include <linux/smb_fs.h>
 static struct smb_conn_opt conn_options;
 
@@ -38,7 +38,7 @@ static struct smb_conn_opt conn_options;
 #endif
 
 /* Uncomment this to allow debug the smbmount daemon */
-#define SMBFS_DEBUG 1
+/* #define SMBFS_DEBUG 1 */
 
 pstring cur_dir = "\\";
 pstring cd_path = "";
@@ -71,7 +71,6 @@ extern int name_type;
 extern int max_protocol;
 int port = SMB_PORT;
 
-
 time_t newer_than = 0;
 int archive_level = 0;
 
@@ -80,12 +79,10 @@ extern int DEBUGLEVEL;
 
 BOOL translation = False;
 
-extern int cnum;
-extern int mid;
-extern int pid;
-extern int tid;
-extern int gid;
-extern int uid;
+extern uint16 cnum;
+extern uint16 mid;
+extern uint16 pid;
+extern uint16 vuid;
 
 extern BOOL have_ip;
 extern int max_xmit;
@@ -97,7 +94,7 @@ extern BOOL tar_reset;
 /* clitar bits end */
  
 
-int myumask = 0755;
+mode_t myumask = 0755;
 
 extern pstring scope;
 
@@ -152,7 +149,7 @@ static BOOL chkpath(char *path,BOOL report)
   trim_string(path2,NULL,"\\");
   if (!*path2) *path2 = '\\';
 
-  bzero(outbuf,smb_size);
+  memset(outbuf,'\0',smb_size);
   set_message(outbuf,0,4 + strlen(path2),True);
   SCVAL(outbuf,smb_com,SMBchkpth);
   SSVAL(outbuf,smb_tid,cnum);
@@ -183,20 +180,51 @@ static BOOL chkpath(char *path,BOOL report)
   return(CVAL(inbuf,smb_rcls) == 0);
 }
 
+static void
+exit_parent( int sig )
+{
+       /* parent simply exits when child says go... */
+       exit(0);
+}
+
 static void
 daemonize(void)
 {
-       int i;
-       if ((i = fork()) < 0)
+       int j, status;
+       pid_t child_pid;
+
+       signal( SIGTERM, exit_parent );
+
+       if ((child_pid = fork()) < 0)
        {
                DEBUG(0, ("could not fork\n"));
        }
-       if (i > 0)
+       if (child_pid > 0)
        {
-               /* parent simply exits */
-               exit(0);
+               while( 1 ) {
+                       j = waitpid( child_pid, &status, 0 );
+                       if( j < 0 ) {
+                               if( EINTR == errno ) {
+                                       continue;
+                               }
+                               status = errno;
+                       }
+                       break;
+               }
+               /* If we get here - the child exited with some error status */
+               exit(status);
        }
-       setsid();
+       /* Programmers Note:
+               Danger Will Robinson!  Danger!
+
+               There use to be a call to setsid() here.  This does no
+               harm to normal mount operations, but it broke automounting.
+               The setsid call has been moved to just before the child
+               sends the SIGTERM to the parent.  All of our deadly embrace
+               conditions with autofs will have been cleared by then...
+               -mhw-
+       */
+       signal( SIGTERM, SIG_DFL );
        chdir("/");
 }
 
@@ -233,7 +261,7 @@ static BOOL mount_send_login(char *inbuf, char *outbuf)
   conn_options.protocol = opt.protocol;
   conn_options.case_handling = CASE_LOWER;
   conn_options.max_xmit = opt.max_xmit;
-  conn_options.server_uid = opt.server_uid;
+  conn_options.server_uid = opt.server_vuid;
   conn_options.tid = opt.tid;
   conn_options.secmode = opt.sec_mode;
   conn_options.maxmux = opt.max_mux;
@@ -258,6 +286,8 @@ send_fs_socket(char *mount_point, char *inbuf, char *outbuf)
 {
        int fd, closed = 0, res = 1;
 
+       pid_t parentpid = getppid();
+
        while (1)
        {
                if ((fd = open(mount_point, O_RDONLY)) < 0)
@@ -279,6 +309,15 @@ send_fs_socket(char *mount_point, char *inbuf, char *outbuf)
                        DEBUG(0, ("smbmount: ioctl failed, res=%d\n", res));
                }
 
+               if( parentpid ) {
+                       /* Ok...  We are going to kill the parent.  Now
+                               is the time to break the process group... */
+                       setsid();
+                       /* Send a signal to the parent to terminate */
+                       kill( parentpid, SIGTERM );
+                       parentpid = 0;
+               }
+
                close_sockets();
                close(fd);
                /*
@@ -295,7 +334,7 @@ send_fs_socket(char *mount_point, char *inbuf, char *outbuf)
                /*
                 * Wait for a signal from smbfs ...
                 */
-               signal(SIGUSR1, &usr1_handler);
+               CatchSignal(SIGUSR1, &usr1_handler);
                pause();
                DEBUG(0, ("smbmount: got signal, getting new socket\n"));
 
@@ -328,7 +367,7 @@ static void cmd_mount(char *inbuf,char *outbuf)
        int retval;
        char mount_point[MAXPATHLEN+1];
 
-       if (!next_token(NULL, mpoint, NULL))
+       if (!next_token(NULL, mpoint, NULL, sizeof(mpoint)))
        {
                DEBUG(0,("You must supply a mount point\n"));
                return;
@@ -352,7 +391,7 @@ static void cmd_mount(char *inbuf,char *outbuf)
 
        slprintf(mount_command, sizeof(mount_command)-1,"smbmnt %s -s %s", mount_point, share_name);
 
-       while(next_token(NULL, buf, NULL))
+       while(next_token(NULL, buf, NULL, sizeof(buf)))
        {
                pstrcat(mount_command, " ");
                pstrcat(mount_command, buf);
@@ -361,17 +400,17 @@ static void cmd_mount(char *inbuf,char *outbuf)
        DEBUG(3, ("mount command: %s\n", mount_command));
 
        /*
-        * Create the background process before trying the mount.
-        * (We delay closing files to allow diagnostic messages.)
-        */
+               Set up to return as a daemon child and wait in the parent
+               until the child say it's ready...
+       */
        daemonize();
 
-       /* The parent has exited here, the child handles the connection: */
        if ((retval = system(mount_command)) != 0)
        {
                DEBUG(0,("mount failed\n"));
                exit(1);
        }
+
        send_fs_socket(mount_point, inbuf, outbuf);
 }      
 
@@ -431,7 +470,7 @@ void cmd_help(char *dum_in, char *dum_out)
   int i=0,j;
   fstring buf;
 
-  if (next_token(NULL,buf,NULL))
+  if (next_token(NULL,buf,NULL,sizeof(buf)))
     {
       if ((i = process_tok(buf)) >= 0)
        DEBUG(0,("HELP %s:\n\t%s\n\n",commands[i].name,commands[i].description));                   
@@ -450,61 +489,25 @@ void cmd_help(char *dum_in, char *dum_out)
 /****************************************************************************
 wait for keyboard activity, swallowing network packets
 ****************************************************************************/
-#ifdef CLIX
-static char wait_keyboard(char *buffer)
-#else
 static void wait_keyboard(char *buffer)
-#endif
 {
   fd_set fds;
   int selrtn;
   struct timeval timeout;
   
-#ifdef CLIX
-  int delay = 0;
-#endif
-  
   while (1) 
     {
       extern int Client;
       FD_ZERO(&fds);
       FD_SET(Client,&fds);
-#ifndef CLIX
       FD_SET(fileno(stdin),&fds);
-#endif
 
       timeout.tv_sec = 20;
       timeout.tv_usec = 0;
-#ifdef CLIX
-      timeout.tv_sec = 0;
-#endif
-      selrtn = sys_select(&fds,&timeout);
+      selrtn = sys_select(MAX(Client,fileno(stdin))+1,&fds,&timeout);
       
-#ifndef CLIX
       if (FD_ISSET(fileno(stdin),&fds))
        return;
-#else
-      {
-       char ch;
-       int readret;
-
-    set_blocking(fileno(stdin), False);        
-       readret = read_data( fileno(stdin), &ch, 1);
-       set_blocking(fileno(stdin), True);
-       if (readret == -1)
-         {
-           if (errno != EAGAIN)
-             {
-               /* should crash here */
-               DEBUG(1,("readchar stdin failed\n"));
-             }
-         }
-       else if (readret != 0)
-         {
-           return ch;
-         }
-      }
-#endif
 
       /* We deliberately use receive_smb instead of
          client_receive_smb as we want to receive
@@ -513,16 +516,7 @@ static void wait_keyboard(char *buffer)
       if (FD_ISSET(Client,&fds))
        receive_smb(Client,buffer,0);
       
-#ifdef CLIX
-      delay++;
-      if (delay > 100000)
-       {
-         delay = 0;
-         chkpath("\\",False);
-       }
-#else
       chkpath("\\",False);
-#endif
     }  
 }
 
@@ -542,7 +536,7 @@ static BOOL process(char *base_directory)
   if ((InBuffer == NULL) || (OutBuffer == NULL)) 
     return(False);
   
-  bzero(OutBuffer,smb_size);
+  memset(OutBuffer,'\0',smb_size);
 
   if (!mount_send_login(InBuffer,OutBuffer))
     return(False);
@@ -574,7 +568,7 @@ static BOOL process(char *base_directory)
       /* and get the first part of the command */
       {
        char *ptr = line;
-       if (!next_token(&ptr,tok,NULL)) continue;
+       if (!next_token(&ptr,tok,NULL,sizeof(tok))) continue;
       }
 
       if ((i = process_tok(tok)) >= 0)
@@ -589,28 +583,17 @@ static BOOL process(char *base_directory)
       fstring tok;
       int i;
 
-      bzero(OutBuffer,smb_size);
+      memset(OutBuffer,'\0',smb_size);
 
       /* display a prompt */
       DEBUG(0,("smb: %s> ", CNV_LANG(cur_dir)));
-      fflush(dbf);
+      dbgflush();
 
-#ifdef CLIX
-      line[0] = wait_keyboard(InBuffer);
-      /* this might not be such a good idea... */
-      if ( line[0] == EOF)
-       break;
-#else
       wait_keyboard(InBuffer);
-#endif
   
       /* and get a response */
-#ifdef CLIX
-      fgets( &line[1],999, stdin);
-#else
       if (!fgets(line,1000,stdin))
        break;
-#endif
 
       /* input language code to internal one */
       CNV_INPUT (line);
@@ -625,7 +608,7 @@ static BOOL process(char *base_directory)
       /* and get the first part of the command */
       {
        char *ptr = line;
-       if (!next_token(&ptr,tok,NULL)) continue;
+       if (!next_token(&ptr,tok,NULL,sizeof(tok))) continue;
       }
 
       if ((i = process_tok(tok)) >= 0)
@@ -698,9 +681,8 @@ static void usage(char *pname)
   TimeInit();
   charset_initialise();
 
-  pid = getpid();
-  uid = getuid();
-  gid = getgid();
+  pid = (uint16)getpid();
+  vuid = (uint16)getuid();
   mid = pid + 100;
   myumask = umask(0);
   umask(myumask);
@@ -861,7 +843,7 @@ static void usage(char *pname)
     }
 
 
-  DEBUG(3,("%s client started (version %s)\n",timestring(),VERSION));
+  DEBUG( 3, ( "Client started (version %s)\n", VERSION ) );
 
   if(!get_myname(myhostname,NULL))
   {