sync'ing up for 3.0alpha20 release
[abartlet/samba.git/.git] / source3 / smbd / process.c
index 1f575e2a462ee1d3719f3b5d185e6b5c9bca5458..0f7cfd0e9cb0e78ba0a19d2f7b228f8c9e1ec09e 100644 (file)
@@ -1,6 +1,5 @@
 /* 
-   Unix SMB/Netbios implementation.
-   Version 1.9.
+   Unix SMB/CIFS implementation.
    process incoming packets - main loop
    Copyright (C) Andrew Tridgell 1992-1998
    
@@ -21,8 +20,6 @@
 
 #include "includes.h"
 
-extern int DEBUGLEVEL;
-
 struct timeval smb_last_time;
 
 static char *InBuffer = NULL;
@@ -44,11 +41,9 @@ int max_recv = BUFFER_SIZE;
 extern int last_message;
 extern int global_oplock_break;
 extern userdom_struct current_user_info;
-extern char *last_inbuf;
-extern char *InBuffer;
-extern char *OutBuffer;
 extern int smb_read_error;
-extern VOLATILE SIG_ATOMIC_T reload_after_sighup;
+SIG_ATOMIC_T reload_after_sighup = 0;
+SIG_ATOMIC_T got_sig_term = 0;
 extern BOOL global_machine_password_needs_changing;
 extern fstring global_myworkgroup;
 extern pstring global_myname;
@@ -87,7 +82,7 @@ static BOOL push_message(ubi_slList *list_head, char *buf, int msg_len)
   if(msg->msg_buf == NULL)
   {
     DEBUG(0,("push_message: malloc fail (2)\n"));
-    free((char *)msg);
+    SAFE_FREE(msg);
     return False;
   }
 
@@ -110,25 +105,32 @@ BOOL push_oplock_pending_smb_message(char *buf, int msg_len)
 }
 
 /****************************************************************************
-do all async processing in here. This includes UDB oplock messages, kernel
-oplock messages, change notify events etc.
+ Do all async processing in here. This includes UDB oplock messages, kernel
+ oplock messages, change notify events etc.
 ****************************************************************************/
-static void async_processing(fd_set *fds, char *buffer, int buffer_len)
+
+static void async_processing(char *buffer, int buffer_len)
 {
+       DEBUG(10,("async_processing: Doing async processing.\n"));
+
        /* check for oplock messages (both UDP and kernel) */
-       if (receive_local_message(fds, buffer, buffer_len, 0)) {
+       if (receive_local_message(buffer, buffer_len, 1)) {
                process_local_message(buffer, buffer_len);
        }
 
+       if (got_sig_term) {
+               exit_server("Caught TERM signal");
+       }
+
        /* check for async change notify events */
        process_pending_change_notify_queue(0);
 
        /* check for sighup processing */
        if (reload_after_sighup) {
-               unbecome_user();
+               change_to_root_user();
                DEBUG(1,("Reloading services after SIGHUP\n"));
                reload_services(False);
-               reload_after_sighup = False;
+               reload_after_sighup = 0;
        }
 }
 
@@ -150,7 +152,7 @@ static void async_processing(fd_set *fds, char *buffer, int buffer_len)
   Returns False on timeout or error.
   Else returns True.
 
-The timeout is in milli seconds
+The timeout is in milliseconds
 ****************************************************************************/
 
 static BOOL receive_message_or_smb(char *buffer, int buffer_len, int timeout)
@@ -180,8 +182,8 @@ static BOOL receive_message_or_smb(char *buffer, int buffer_len, int timeout)
                memcpy(buffer, msg->msg_buf, MIN(buffer_len, msg->msg_len));
   
                /* Free the message we just copied. */
-               free((char *)msg->msg_buf);
-               free((char *)msg);
+               SAFE_FREE(msg->msg_buf);
+               SAFE_FREE(msg);
                
                DEBUG(5,("receive_message_or_smb: returning queued smb message.\n"));
                return True;
@@ -193,20 +195,46 @@ static BOOL receive_message_or_smb(char *buffer, int buffer_len, int timeout)
         */
 
        FD_ZERO(&fds);
+
+       /*
+        * Ensure we process oplock break messages by preference.
+        * We have to do this before the select, after the select
+        * and if the select returns EINTR. This is due to the fact
+        * that the selects called from async_processing can eat an EINTR
+        * caused by a signal (we can't take the break message there).
+        * This is hideously complex - *MUST* be simplified for 3.0 ! JRA.
+        */
+
+       if (oplock_message_waiting(&fds)) {
+               DEBUG(10,("receive_message_or_smb: oplock_message is waiting.\n"));
+               async_processing(buffer, buffer_len);
+               /*
+                * After async processing we must go and do the select again, as
+                * the state of the flag in fds for the server file descriptor is
+                * indeterminate - we may have done I/O on it in the oplock processing. JRA.
+                */
+               goto again;
+       }
+       
        FD_SET(smbd_server_fd(),&fds);
        maxfd = setup_oplock_select_set(&fds);
 
        to.tv_sec = timeout / 1000;
        to.tv_usec = (timeout % 1000) * 1000;
 
-       selrtn = sys_select(MAX(maxfd,smbd_server_fd())+1,&fds,timeout>0?&to:NULL);
+       selrtn = sys_select(MAX(maxfd,smbd_server_fd())+1,&fds,NULL,NULL,timeout>0?&to:NULL);
 
        /* if we get EINTR then maybe we have received an oplock
           signal - treat this as select returning 1. This is ugly, but
           is the best we can do until the oplock code knows more about
           signals */
        if (selrtn == -1 && errno == EINTR) {
-               async_processing(&fds, buffer, buffer_len);
+               async_processing(buffer, buffer_len);
+               /*
+                * After async processing we must go and do the select again, as
+                * the state of the flag in fds for the server file descriptor is
+                * indeterminate - we may have done I/O on it in the oplock processing. JRA.
+                */
                goto again;
        }
 
@@ -223,9 +251,20 @@ static BOOL receive_message_or_smb(char *buffer, int buffer_len, int timeout)
                return False;
        }
 
-       if (!FD_ISSET(smbd_server_fd(),&fds) || selrtn > 1) {
-               async_processing(&fds, buffer, buffer_len);
-               if (!FD_ISSET(smbd_server_fd(),&fds)) goto again;
+       /*
+        * Ensure we process oplock break messages by preference.
+        * This is IMPORTANT ! Otherwise we can starve other processes
+        * sending us an oplock break message. JRA.
+        */
+
+       if (oplock_message_waiting(&fds)) {
+               async_processing(buffer, buffer_len);
+               /*
+                * After async processing we must go and do the select again, as
+                * the state of the flag in fds for the server file descriptor is
+                * indeterminate - we may have done I/O on it in the oplock processing. JRA.
+                */
+               goto again;
        }
        
        return receive_smb(smbd_server_fd(), buffer, 0);
@@ -243,7 +282,7 @@ BOOL receive_next_smb(char *inbuf, int bufsize, int timeout)
        do {
                ret = receive_message_or_smb(inbuf,bufsize,timeout);
                
-               got_keepalive = (ret && (CVAL(inbuf,0) == 0x85));
+               got_keepalive = (ret && (CVAL(inbuf,0) == SMBkeepalive));
        } while (ret && got_keepalive);
 
        return ret;
@@ -259,7 +298,6 @@ BOOL receive_next_smb(char *inbuf, int bufsize, int timeout)
 void respond_to_all_remaining_local_messages(void)
 {
   char buffer[1024];
-  fd_set fds;
 
   /*
    * Assert we have no exclusive open oplocks.
@@ -271,24 +309,13 @@ void respond_to_all_remaining_local_messages(void)
     return;
   }
 
-  /*
-   * Setup the select read fd set.
-   */
-
-  FD_ZERO(&fds);
-  if(!setup_oplock_select_set(&fds))
-    return;
-
   /*
    * Keep doing receive_local_message with a 1 ms timeout until
    * we have no more messages.
    */
-  while(receive_local_message(&fds, buffer, sizeof(buffer), 1)) {
+  while(receive_local_message(buffer, sizeof(buffer), 1)) {
          /* Deal with oplock break requests from other smbd's. */
          process_local_message(buffer, sizeof(buffer));
-
-         FD_ZERO(&fds);
-         (void)setup_oplock_select_set(&fds);
   }
 
   return;
@@ -314,9 +341,9 @@ force write permissions on print services.
    functions. Any message that has a NULL function is unimplemented -
    please feel free to contribute implementations!
 */
-struct smb_message_struct
+const static struct smb_message_struct
 {
-  char *name;
+  const char *name;
   int (*fn)(connection_struct *conn, char *, char *, int, int);
   int flags;
 }
@@ -350,16 +377,16 @@ struct smb_message_struct
 /* 0x19 */ { NULL, NULL, 0 },
 /* 0x1a */ { "SMBreadbraw",reply_readbraw,AS_USER},
 /* 0x1b */ { "SMBreadBmpx",reply_readbmpx,AS_USER},
-/* 0x1c */ { "SMBreadBs",NULL,AS_USER},
+/* 0x1c */ { "SMBreadBs",NULL,},
 /* 0x1d */ { "SMBwritebraw",reply_writebraw,AS_USER},
 /* 0x1e */ { "SMBwriteBmpx",reply_writebmpx,AS_USER},
 /* 0x1f */ { "SMBwriteBs",reply_writebs,AS_USER},
-/* 0x20 */ { "SMBwritec",NULL,AS_USER},
+/* 0x20 */ { "SMBwritec",NULL,0},
 /* 0x21 */ { NULL, NULL, 0 },
 /* 0x22 */ { "SMBsetattrE",reply_setattrE,AS_USER | NEED_WRITE },
 /* 0x23 */ { "SMBgetattrE",reply_getattrE,AS_USER },
 /* 0x24 */ { "SMBlockingX",reply_lockingX,AS_USER },
-/* 0x25 */ { "SMBtrans",reply_trans,AS_USER | CAN_IPC | QUEUE_IN_OPLOCK},
+/* 0x25 */ { "SMBtrans",reply_trans,AS_USER | CAN_IPC },
 /* 0x26 */ { "SMBtranss",NULL,AS_USER | CAN_IPC},
 /* 0x27 */ { "SMBioctl",reply_ioctl,0},
 /* 0x28 */ { "SMBioctls",NULL,AS_USER},
@@ -372,7 +399,7 @@ struct smb_message_struct
 /* 0x2f */ { "SMBwriteX",reply_write_and_X,AS_USER | CAN_IPC },
 /* 0x30 */ { NULL, NULL, 0 },
 /* 0x31 */ { NULL, NULL, 0 },
-/* 0x32 */ { "SMBtrans2", reply_trans2, AS_USER | QUEUE_IN_OPLOCK | CAN_IPC },
+/* 0x32 */ { "SMBtrans2", reply_trans2, AS_USER | CAN_IPC },
 /* 0x33 */ { "SMBtranss2", reply_transs2, AS_USER},
 /* 0x34 */ { "SMBfindclose", reply_findclose,AS_USER},
 /* 0x35 */ { "SMBfindnclose", reply_findnclose, AS_USER},
@@ -582,9 +609,10 @@ struct smb_message_struct
 };
 
 /*******************************************************************
-dump a prs to a file
- ********************************************************************/
-static void smb_dump(char *name, int type, char *data, ssize_t len)
+ Dump a packet to a file.
+********************************************************************/
+
+static void smb_dump(const char *name, int type, char *data, ssize_t len)
 {
        int fd, i;
        pstring fname;
@@ -598,7 +626,9 @@ static void smb_dump(char *name, int type, char *data, ssize_t len)
                if (fd != -1 || errno != EEXIST) break;
        }
        if (fd != -1) {
-               write(fd, data, len);
+               ssize_t ret = write(fd, data, len);
+               if (ret != len)
+                       DEBUG(0,("smb_dump: problem: write returned %d\n", (int)ret ));
                close(fd);
                DEBUG(0,("created %s len %d\n", fname, len));
        }
@@ -606,175 +636,174 @@ static void smb_dump(char *name, int type, char *data, ssize_t len)
 
 
 /****************************************************************************
-do a switch on the message type, and return the response size
+ Do a switch on the message type, and return the response size
 ****************************************************************************/
+
 static int switch_message(int type,char *inbuf,char *outbuf,int size,int bufsize)
 {
-  static pid_t pid= (pid_t)-1;
-  int outsize = 0;
-  extern int global_smbpid;
+       static pid_t pid= (pid_t)-1;
+       int outsize = 0;
+       extern uint16 global_smbpid;
 
-  type &= 0xff;
+       type &= 0xff;
 
-  if (pid == (pid_t)-1)
-    pid = sys_getpid();
+       if (pid == (pid_t)-1)
+               pid = sys_getpid();
 
-  errno = 0;
-  last_message = type;
+       errno = 0;
+       last_message = type;
 
-  /* make sure this is an SMB packet */
-  if (strncmp(smb_base(inbuf),"\377SMB",4) != 0)
-  {
-    DEBUG(2,("Non-SMB packet of length %d\n",smb_len(inbuf)));
-    return(-1);
-  }
-
-  /* yuck! this is an interim measure before we get rid of our
-     current inbuf/outbuf system */
-  global_smbpid = SVAL(inbuf,smb_pid);
+       /* Make sure this is an SMB packet. smb_size contains NetBIOS header so subtract 4 from it. */
+       if ((strncmp(smb_base(inbuf),"\377SMB",4) != 0) || (size < (smb_size - 4))) {
+               DEBUG(2,("Non-SMB packet of length %d. Terminating server\n",smb_len(inbuf)));
+               exit_server("Non-SMB packet");
+               return(-1);
+       }
 
-  if (smb_messages[type].fn == NULL)
-  {
-    DEBUG(0,("Unknown message type %d!\n",type));
-    smb_dump("Unknown", 1, inbuf, size);
-    outsize = reply_unknown(inbuf,outbuf);
-  }
-  else
-  {
-    int flags = smb_messages[type].flags;
-    static uint16 last_session_tag = UID_FIELD_INVALID;
-    /* In share mode security we must ignore the vuid. */
-    uint16 session_tag = (lp_security() == SEC_SHARE) ? UID_FIELD_INVALID : SVAL(inbuf,smb_uid);
-    connection_struct *conn = conn_find(SVAL(inbuf,smb_tid));
+       /* yuck! this is an interim measure before we get rid of our
+               current inbuf/outbuf system */
+       global_smbpid = SVAL(inbuf,smb_pid);
+
+       if (smb_messages[type].fn == NULL) {
+               DEBUG(0,("Unknown message type %d!\n",type));
+               smb_dump("Unknown", 1, inbuf, size);
+               outsize = reply_unknown(inbuf,outbuf);
+       } else {
+               int flags = smb_messages[type].flags;
+               static uint16 last_session_tag = UID_FIELD_INVALID;
+               /* In share mode security we must ignore the vuid. */
+               uint16 session_tag = (lp_security() == SEC_SHARE) ? UID_FIELD_INVALID : SVAL(inbuf,smb_uid);
+               connection_struct *conn = conn_find(SVAL(inbuf,smb_tid));
+
+               DEBUG(3,("switch message %s (pid %d)\n",smb_fn_name(type),(int)pid));
+
+               smb_dump(smb_fn_name(type), 1, inbuf, size);
+               if(global_oplock_break) {
+                       if(flags & QUEUE_IN_OPLOCK) {
+                               /* 
+                                * Queue this message as we are the process of an oplock break.
+                                */
+
+                               DEBUG( 2, ( "switch_message: queueing message due to being in " ) );
+                               DEBUGADD( 2, ( "oplock break state.\n" ) );
+
+                               push_oplock_pending_smb_message( inbuf, size );
+                               return -1;
+                       }
+               }
 
-    DEBUG(3,("switch message %s (pid %d)\n",smb_fn_name(type),(int)pid));
+               /* Ensure this value is replaced in the incoming packet. */
+               SSVAL(inbuf,smb_uid,session_tag);
 
-    smb_dump(smb_fn_name(type), 1, inbuf, size);
-    if(global_oplock_break)
-    {
-      if(flags & QUEUE_IN_OPLOCK)
-      {
-        /* 
-         * Queue this message as we are the process of an oplock break.
-         */
-
-        DEBUG( 2, ( "switch_message: queueing message due to being in " ) );
-        DEBUGADD( 2, ( "oplock break state.\n" ) );
-
-        push_oplock_pending_smb_message( inbuf, size );
-        return -1;
-      }          
-    }
+               /*
+                * Ensure the correct username is in current_user_info.
+                * This is a really ugly bugfix for problems with
+                * multiple session_setup_and_X's being done and
+                * allowing %U and %G substitutions to work correctly.
+                * There is a reason this code is done here, don't
+                * move it unless you know what you're doing... :-).
+                * JRA.
+                */
 
-    /* Ensure this value is replaced in the incoming packet. */
-    SSVAL(inbuf,smb_uid,session_tag);
+               if (session_tag != last_session_tag) {
+                       user_struct *vuser = NULL;
 
-    /*
-     * Ensure the correct username is in current_user_info.
-     * This is a really ugly bugfix for problems with
-     * multiple session_setup_and_X's being done and
-     * allowing %U and %G substitutions to work correctly.
-     * There is a reason this code is done here, don't
-     * move it unless you know what you're doing... :-).
-     * JRA.
-     */
+                       last_session_tag = session_tag;
+                       if(session_tag != UID_FIELD_INVALID)
+                               vuser = get_valid_user_struct(session_tag);           
+                       if(vuser != NULL)
+                               current_user_info = vuser->user;
+               }
 
-    if (session_tag != last_session_tag) {
-      user_struct *vuser = NULL;
+               /* does this protocol need to be run as root? */
+               if (!(flags & AS_USER))
+                       change_to_root_user();
 
-      last_session_tag = session_tag;
-      if(session_tag != UID_FIELD_INVALID)
-        vuser = get_valid_user_struct(session_tag);           
-      if(vuser != NULL)
-        current_user_info = vuser->user;
-    }
+               /* does this protocol need a valid tree connection? */
+               if ((flags & AS_USER) && !conn)
+                       return ERROR_DOS(ERRSRV, ERRinvnid);
 
-    /* does this protocol need to be run as root? */
-    if (!(flags & AS_USER))
-      unbecome_user();
 
-    /* does this protocol need to be run as the connected user? */
-    if ((flags & AS_USER) && !become_user(conn,session_tag)) {
-      if (flags & AS_GUEST) 
-        flags &= ~AS_USER;
-      else
-        return(ERROR(ERRSRV,ERRaccess));
-    }
+               /* does this protocol need to be run as the connected user? */
+               if ((flags & AS_USER) && !change_to_user(conn,session_tag)) {
+                       if (flags & AS_GUEST) 
+                               flags &= ~AS_USER;
+                       else
+                               return(ERROR_DOS(ERRSRV,ERRaccess));
+               }
 
-    /* this code is to work around a bug is MS client 3 without
-       introducing a security hole - it needs to be able to do
-       print queue checks as guest if it isn't logged in properly */
-    if (flags & AS_USER)
-      flags &= ~AS_GUEST;
+               /* this code is to work around a bug is MS client 3 without
+                       introducing a security hole - it needs to be able to do
+                       print queue checks as guest if it isn't logged in properly */
+               if (flags & AS_USER)
+                       flags &= ~AS_GUEST;
 
-    /* does it need write permission? */
-    if ((flags & NEED_WRITE) && !CAN_WRITE(conn))
-      return(ERROR(ERRSRV,ERRaccess));
+               /* does it need write permission? */
+               if ((flags & NEED_WRITE) && !CAN_WRITE(conn))
+                       return(ERROR_DOS(ERRSRV,ERRaccess));
 
-    /* ipc services are limited */
-    if (IS_IPC(conn) && (flags & AS_USER) && !(flags & CAN_IPC)) {
-      return(ERROR(ERRSRV,ERRaccess));     
-    }
+               /* ipc services are limited */
+               if (IS_IPC(conn) && (flags & AS_USER) && !(flags & CAN_IPC))
+                       return(ERROR_DOS(ERRSRV,ERRaccess));        
 
-    /* load service specific parameters */
-    if (conn && !become_service(conn,(flags & AS_USER)?True:False)) {
-      return(ERROR(ERRSRV,ERRaccess));
-    }
+               /* load service specific parameters */
+               if (conn && !set_current_service(conn,(flags & AS_USER)?True:False))
+                       return(ERROR_DOS(ERRSRV,ERRaccess));
 
-    /* does this protocol need to be run as guest? */
-    if ((flags & AS_GUEST) && 
-                (!become_guest() || 
-               !check_access(smbd_server_fd(), lp_hostsallow(-1), lp_hostsdeny(-1)))) {
-      return(ERROR(ERRSRV,ERRaccess));
-    }
+               /* does this protocol need to be run as guest? */
+               if ((flags & AS_GUEST) && (!change_to_guest() || 
+                               !check_access(smbd_server_fd(), lp_hostsallow(-1), lp_hostsdeny(-1))))
+                       return(ERROR_DOS(ERRSRV,ERRaccess));
 
-    last_inbuf = inbuf;
+               last_inbuf = inbuf;
 
-    outsize = smb_messages[type].fn(conn, inbuf,outbuf,size,bufsize);
-  }
+               outsize = smb_messages[type].fn(conn, inbuf,outbuf,size,bufsize);
+       }
 
-  smb_dump(smb_fn_name(type), 0, outbuf, outsize);
+       smb_dump(smb_fn_name(type), 0, outbuf, outsize);
 
-  return(outsize);
+       return(outsize);
 }
 
 
 /****************************************************************************
-  construct a reply to the incoming packet
+ Construct a reply to the incoming packet.
 ****************************************************************************/
+
 static int construct_reply(char *inbuf,char *outbuf,int size,int bufsize)
 {
-  int type = CVAL(inbuf,smb_com);
-  int outsize = 0;
-  int msg_type = CVAL(inbuf,0);
+       int type = CVAL(inbuf,smb_com);
+       int outsize = 0;
+       int msg_type = CVAL(inbuf,0);
 
-  GetTimeOfDay(&smb_last_time);
+       GetTimeOfDay(&smb_last_time);
 
-  chain_size = 0;
-  file_chain_reset();
-  reset_chain_p();
+       chain_size = 0;
+       file_chain_reset();
+       reset_chain_p();
 
-  if (msg_type != 0)
-    return(reply_special(inbuf,outbuf));  
+       if (msg_type != 0)
+               return(reply_special(inbuf,outbuf));  
 
-  construct_reply_common(inbuf, outbuf);
+       construct_reply_common(inbuf, outbuf);
 
-  outsize = switch_message(type,inbuf,outbuf,size,bufsize);
+       outsize = switch_message(type,inbuf,outbuf,size,bufsize);
 
-  outsize += chain_size;
+       outsize += chain_size;
 
-  if(outsize > 4)
-    smb_setlen(outbuf,outsize - 4);
-  return(outsize);
+       if(outsize > 4)
+               smb_setlen(outbuf,outsize - 4);
+       return(outsize);
 }
 
 /****************************************************************************
 Keep track of the number of running smbd's. This functionality is used to
 'hard' limit Samba overhead on resource constrained systems. 
+ Keep track of the number of running smbd's. This functionality is used to
+ 'hard' limit Samba overhead on resource constrained systems. 
 ****************************************************************************/
+
 static BOOL smbd_process_limit(void)
 {
-       int  total_smbds;
+       int32  total_smbds;
        
        if (lp_max_smbd_processes()) {
 
@@ -784,7 +813,13 @@ static BOOL smbd_process_limit(void)
 
                total_smbds = 1; /* In case we need to create the entry. */
 
-               if (tdb_change_int_atomic(conn_tdb_ctx(), "INFO/total_smbds", &total_smbds, 1) == -1)
+               if (!conn_tdb_ctx()) {
+                       DEBUG(0,("smbd_process_limit: max smbd processes parameter set with status parameter not \
+set. Ignoring max smbd restriction.\n"));
+                       return False;
+               }
+
+               if (tdb_change_int32_atomic(conn_tdb_ctx(), "INFO/total_smbds", &total_smbds, 1) == -1)
                        return True;
 
                return total_smbds > lp_max_smbd_processes();
@@ -794,15 +829,11 @@ static BOOL smbd_process_limit(void)
 }
 
 /****************************************************************************
-  process an smb from the client - split out from the process() code so
+  process an smb from the client - split out from the smbd_process() code so
   it can be used by the oplock break code.
 ****************************************************************************/
 void process_smb(char *inbuf, char *outbuf)
 {
-#ifdef WITH_SSL
-  extern BOOL sslEnabled;     /* don't use function for performance reasons */
-  static int sslConnected = 0;
-#endif /* WITH_SSL */
   static int trans_num;
   int msg_type = CVAL(inbuf,0);
   int32 len = smb_len(inbuf);
@@ -822,7 +853,7 @@ void process_smb(char *inbuf, char *outbuf)
                  static unsigned char buf[5] = {0x83, 0, 0, 1, 0x81};
                  DEBUG( 1, ( "Connection denied from %s\n",
                              client_addr() ) );
-                 send_smb(smbd_server_fd(),(char *)buf);
+                 (void)send_smb(smbd_server_fd(),(char *)buf);
                  exit_server("connection denied");
          }
   }
@@ -830,21 +861,9 @@ void process_smb(char *inbuf, char *outbuf)
   DEBUG( 6, ( "got message type 0x%x of len 0x%x\n", msg_type, len ) );
   DEBUG( 3, ( "Transaction %d of length %d\n", trans_num, nread ) );
 
-#ifdef WITH_SSL
-    if(sslEnabled && !sslConnected){
-        sslConnected = sslutil_negotiate_ssl(smbd_server_fd(), msg_type);
-        if(sslConnected < 0){   /* an error occured */
-            exit_server("SSL negotiation failed");
-        }else if(sslConnected){
-            trans_num++;
-            return;
-        }
-    }
-#endif  /* WITH_SSL */
-
   if (msg_type == 0)
     show_msg(inbuf);
-  else if(msg_type == 0x85)
+  else if(msg_type == SMBkeepalive)
     return; /* Keepalive packet. */
 
   nread = construct_reply(inbuf,outbuf,nread,max_send);
@@ -860,7 +879,8 @@ void process_smb(char *inbuf, char *outbuf)
                  nread, smb_len(outbuf)));
     }
     else
-      send_smb(smbd_server_fd(),outbuf);
+      if (!send_smb(smbd_server_fd(),outbuf))
+        exit_server("process_smb: send_smb failed.");
   }
   trans_num++;
 }
@@ -870,7 +890,7 @@ void process_smb(char *inbuf, char *outbuf)
 /****************************************************************************
 return a string containing the function name of a SMB command
 ****************************************************************************/
-char *smb_fn_name(int type)
+const char *smb_fn_name(int type)
 {
        static char *unknown_name = "SMBunknown";
 
@@ -887,25 +907,25 @@ char *smb_fn_name(int type)
 
 void construct_reply_common(char *inbuf,char *outbuf)
 {
-  memset(outbuf,'\0',smb_size);
-
-  set_message(outbuf,0,0,True);
-  CVAL(outbuf,smb_com) = CVAL(inbuf,smb_com);
-
-  memcpy(outbuf+4,inbuf+4,4);
-  CVAL(outbuf,smb_rcls) = SMB_SUCCESS;
-  CVAL(outbuf,smb_reh) = 0;
-  SCVAL(outbuf,smb_flg, FLAG_REPLY | (CVAL(inbuf,smb_flg) & FLAG_CASELESS_PATHNAMES)); /* bit 7 set
-                                 means a reply */
-  SSVAL(outbuf,smb_flg2,
-       (SVAL(inbuf,smb_flg2)&FLAGS2_UNICODE_STRINGS) | FLAGS2_LONG_PATH_COMPONENTS);
-       /* say we support long filenames */
-
-  SSVAL(outbuf,smb_err,SMB_SUCCESS);
-  SSVAL(outbuf,smb_tid,SVAL(inbuf,smb_tid));
-  SSVAL(outbuf,smb_pid,SVAL(inbuf,smb_pid));
-  SSVAL(outbuf,smb_uid,SVAL(inbuf,smb_uid));
-  SSVAL(outbuf,smb_mid,SVAL(inbuf,smb_mid));
+       memset(outbuf,'\0',smb_size);
+
+       set_message(outbuf,0,0,True);
+       SCVAL(outbuf,smb_com,CVAL(inbuf,smb_com));
+       
+       memcpy(outbuf+4,inbuf+4,4);
+       SCVAL(outbuf,smb_rcls,SMB_SUCCESS);
+       SCVAL(outbuf,smb_reh,0);
+       SCVAL(outbuf,smb_flg, FLAG_REPLY | (CVAL(inbuf,smb_flg) & FLAG_CASELESS_PATHNAMES)); 
+       SSVAL(outbuf,smb_flg2,
+             (SVAL(inbuf,smb_flg2) & FLAGS2_UNICODE_STRINGS) |
+             FLAGS2_LONG_PATH_COMPONENTS |
+             FLAGS2_32_BIT_ERROR_CODES | FLAGS2_EXTENDED_SECURITY);
+
+       SSVAL(outbuf,smb_err,SMB_SUCCESS);
+       SSVAL(outbuf,smb_tid,SVAL(inbuf,smb_tid));
+       SSVAL(outbuf,smb_pid,SVAL(inbuf,smb_pid));
+       SSVAL(outbuf,smb_uid,SVAL(inbuf,smb_uid));
+       SSVAL(outbuf,smb_mid,SVAL(inbuf,smb_mid));
 }
 
 /****************************************************************************
@@ -926,7 +946,7 @@ int chain_reply(char *inbuf,char *outbuf,int size,int bufsize)
 
   /* maybe its not chained */
   if (smb_com2 == 0xFF) {
-    CVAL(outbuf,smb_vwv0) = 0xFF;
+    SCVAL(outbuf,smb_vwv0,0xFF);
     return outsize;
   }
 
@@ -946,7 +966,7 @@ int chain_reply(char *inbuf,char *outbuf,int size,int bufsize)
 
   /* we need to tell the client where the next part of the reply will be */
   SSVAL(outbuf,smb_vwv1,smb_offset(outbuf+outsize,outbuf));
-  CVAL(outbuf,smb_vwv0) = smb_com2;
+  SCVAL(outbuf,smb_vwv0,smb_com2);
 
   /* remember how much the caller added to the chain, only counting stuff
      after the parameter words */
@@ -968,7 +988,7 @@ int chain_reply(char *inbuf,char *outbuf,int size,int bufsize)
   memmove(inbuf2,inbuf,smb_wct);
 
   /* create the in buffer */
-  CVAL(inbuf2,smb_com) = smb_com2;
+  SCVAL(inbuf2,smb_com,smb_com2);
 
   /* create the out buffer */
   construct_reply_common(inbuf2, outbuf2);
@@ -983,7 +1003,7 @@ int chain_reply(char *inbuf,char *outbuf,int size,int bufsize)
   /* copy the new reply and request headers over the old ones, but
      preserve the smb_com field */
   memmove(orig_outbuf,outbuf2,smb_wct);
-  CVAL(orig_outbuf,smb_com) = smb_com1;
+  SCVAL(orig_outbuf,smb_com,smb_com1);
 
   /* restore the saved data, being careful not to overwrite any
    data from the reply header */
@@ -1006,16 +1026,15 @@ static int setup_select_timeout(void)
        int select_timeout;
        int t;
 
-       /*
-        * Increase the select timeout back to SMBD_SELECT_TIMEOUT if we
-        * have removed any blocking locks. JRA.
-        */
-
-       select_timeout = blocking_locks_pending() ? SMBD_SELECT_TIMEOUT_WITH_PENDING_LOCKS*1000 :
-               SMBD_SELECT_TIMEOUT*1000;
+       select_timeout = blocking_locks_timeout(SMBD_SELECT_TIMEOUT);
+       select_timeout *= 1000;
 
        t = change_notify_timeout();
-       if (t != -1) select_timeout = MIN(select_timeout, t*1000);
+       if (t != -1)
+               select_timeout = MIN(select_timeout, t*1000);
+
+       if (print_notify_messages_pending())
+               select_timeout = MIN(select_timeout, 1000);
 
        return select_timeout;
 }
@@ -1073,7 +1092,7 @@ static BOOL timeout_processing(int deadtime, int *select_timeout, time_t *last_t
     last_idle_closed_check = t;
 
   /* become root again if waiting */
-  unbecome_user();
+  change_to_root_user();
 
   /* check if we need to reload services */
   check_reload(t);
@@ -1089,16 +1108,23 @@ static BOOL timeout_processing(int deadtime, int *select_timeout, time_t *last_t
 
   if (keepalive && (t - last_keepalive_sent_time)>keepalive) 
   {
-    struct cli_state *cli = server_client();
-    if (!send_keepalive(smbd_server_fd())) {
-      DEBUG( 2, ( "Keepalive failed - exiting.\n" ) );
-      return False;
-    }      
-    /* also send a keepalive to the password server if its still
-       connected */
-    if (cli && cli->initialised)
-      send_keepalive(cli->fd);
-    last_keepalive_sent_time = t;
+         extern struct auth_context *negprot_global_auth_context;
+         if (!send_keepalive(smbd_server_fd())) {
+                 DEBUG( 2, ( "Keepalive failed - exiting.\n" ) );
+                 return False;
+         }
+         
+         /* send a keepalive for a password server or the like.
+            This is attached to the auth_info created in the
+            negprot */
+         if (negprot_global_auth_context 
+             && negprot_global_auth_context->challenge_set_method 
+             && negprot_global_auth_context->challenge_set_method->send_keepalive) {
+                 negprot_global_auth_context->challenge_set_method->send_keepalive
+                         (&negprot_global_auth_context->challenge_set_method->private_data);
+         }
+
+         last_keepalive_sent_time = t;
   }
 
   /* check for connection timeouts */
@@ -1109,7 +1135,10 @@ static BOOL timeout_processing(int deadtime, int *select_timeout, time_t *last_t
     return False;
   }
 
-  if(global_machine_password_needs_changing)
+  if(global_machine_password_needs_changing && 
+     /* for ADS we need to do a regular ADS password change, not a domain
+        password change */
+     lp_security() == SEC_DOMAIN)
   {
     unsigned char trust_passwd_hash[16];
     time_t lct;
@@ -1125,9 +1154,16 @@ static BOOL timeout_processing(int deadtime, int *select_timeout, time_t *last_t
      * First, open the machine password file with an exclusive lock.
      */
 
+    if (secrets_lock_trust_account_password(global_myworkgroup, True) == False) {
+      DEBUG(0,("process: unable to lock the machine account password for \
+machine %s in domain %s.\n", global_myname, global_myworkgroup ));
+      return True;
+    }
+
     if(!secrets_fetch_trust_account_password(global_myworkgroup, trust_passwd_hash, &lct)) {
       DEBUG(0,("process: unable to read the machine account password for \
 machine %s in domain %s.\n", global_myname, global_myworkgroup ));
+      secrets_lock_trust_account_password(global_myworkgroup, False);
       return True;
     }
 
@@ -1137,6 +1173,7 @@ machine %s in domain %s.\n", global_myname, global_myworkgroup ));
 
     if(t < lct + lp_machine_password_timeout()) {
       global_machine_password_needs_changing = False;
+      secrets_lock_trust_account_password(global_myworkgroup, False);
       return True;
     }
 
@@ -1144,6 +1181,7 @@ machine %s in domain %s.\n", global_myname, global_myworkgroup ));
 
     change_trust_account_password( global_myworkgroup, remote_machine_list);
     global_machine_password_needs_changing = False;
+    secrets_lock_trust_account_password(global_myworkgroup, False);
   }
 
   /*
@@ -1160,9 +1198,14 @@ machine %s in domain %s.\n", global_myname, global_myworkgroup ));
 
   /*
    * Now we are root, check if the log files need pruning.
+   * Force a log file check.
    */
-  if(need_to_check_log_size())
-      check_log_size();
+  force_check_log_size();
+  check_log_size();
+
+  /* Send any queued printer notify message to interested smbd's. */
+
+  print_notify_send_messages();
 
   /*
    * Modify the select timeout depending upon
@@ -1191,9 +1234,6 @@ void smbd_process(void)
 
        max_recv = MIN(lp_maxxmit(),BUFFER_SIZE);
 
-       /* re-initialise the timezone */
-       TimeInit();
-
        while (True) {
                int deadtime = lp_deadtime()*60;
                int select_timeout = setup_select_timeout();
@@ -1244,7 +1284,7 @@ void smbd_process(void)
                
                if ((num_smbs % 200) == 0) {
                        time_t new_check_time = time(NULL);
-                       if(last_timeout_processing_time - new_check_time >= (select_timeout/1000)) {
+                       if(new_check_time - last_timeout_processing_time >= (select_timeout/1000)) {
                                if(!timeout_processing( deadtime, &select_timeout, &last_timeout_processing_time))
                                        return;
                                num_smbs = 0; /* Reset smb counter. */