r23221: Fix a bad merge, avoid a memory leak and tdb lock problem
[sfrench/samba-autobuild/.git] / source / smbd / connection.c
index da904e0a04a33c44cf0d24b96791392243147269..211f0e6acba8208764d342aca4277232779960bc 100644 (file)
@@ -1,6 +1,5 @@
 /* 
-   Unix SMB/Netbios implementation.
-   Version 1.9.
+   Unix SMB/CIFS implementation.
    connection claim routines
    Copyright (C) Andrew Tridgell 1998
    
 
 #include "includes.h"
 
-extern fstring remote_machine;
-static TDB_CONTEXT *tdb;
-
-/****************************************************************************
- Return the connection tdb context (used for message send all).
-****************************************************************************/
-
-TDB_CONTEXT *conn_tdb_ctx(void)
-{
-       return tdb;
-}
-
 /****************************************************************************
  Delete a connection record.
 ****************************************************************************/
 
-BOOL yield_connection(connection_struct *conn,char *name)
+BOOL yield_connection(connection_struct *conn, const char *name)
 {
-       struct connections_key key;
-       TDB_DATA kbuf;
-
-       if (!tdb) return False;
+       struct db_record *rec;
+       NTSTATUS status;
 
        DEBUG(3,("Yielding connection to %s\n",name));
 
-       ZERO_STRUCT(key);
-       key.pid = sys_getpid();
-       key.cnum = conn?conn->cnum:-1;
-       fstrcpy(key.name, name);
-
-       kbuf.dptr = (char *)&key;
-       kbuf.dsize = sizeof(key);
+       if (!(rec = connections_fetch_entry(NULL, conn, name))) {
+               DEBUG(0, ("connections_fetch_entry failed\n"));
+               return False;
+       }
 
-       if (tdb_delete(tdb, kbuf) != 0) {
-               int dbg_lvl = (!conn && (tdb_error(tdb) == TDB_ERR_NOEXIST)) ? 3 : 0;
-               DEBUG(dbg_lvl,("yield_connection: tdb_delete for name %s failed with error %s.\n",
-                       name, tdb_errorstr(tdb) ));
-               return (False);
+       status = rec->delete_rec(rec);
+       if (!NT_STATUS_IS_OK(status)) {
+               DEBUG( NT_STATUS_EQUAL(status, NT_STATUS_NOT_FOUND) ? 3 : 0,
+                      ("deleting connection record returned %s\n",
+                       nt_errstr(status)));
        }
 
-       return(True);
+       TALLOC_FREE(rec);
+       return NT_STATUS_IS_OK(status);
 }
 
 struct count_stat {
        pid_t mypid;
        int curr_connections;
-       char *name;
+       const char *name;
        BOOL Clear;
 };
 
@@ -75,31 +58,45 @@ struct count_stat {
  Count the entries belonging to a service in the connection db.
 ****************************************************************************/
 
-static int count_fn( TDB_CONTEXT *the_tdb, TDB_DATA kbuf, TDB_DATA dbuf, void *udp)
+static int count_fn(struct db_record *rec,
+                   const struct connections_key *ckey,
+                   const struct connections_data *crec,
+                   void *udp)
 {
-       struct connections_data crec;
        struct count_stat *cs = (struct count_stat *)udp;
  
-       if (dbuf.dsize != sizeof(crec))
-               return 0;
-
-       memcpy(&crec, dbuf.dptr, sizeof(crec));
-    if (crec.cnum == -1)
+       if (crec->cnum == -1) {
                return 0;
+       }
 
        /* If the pid was not found delete the entry from connections.tdb */
 
-       if (cs->Clear && !process_exists(crec.pid) && (errno == ESRCH)) {
-               DEBUG(2,("pid %u doesn't exist - deleting connections %d [%s]\n",
-                       (unsigned int)crec.pid, crec.cnum, crec.name));
-               if (tdb_delete(the_tdb, kbuf) != 0)
-                       DEBUG(0,("count_fn: tdb_delete failed with error %s\n", tdb_errorstr(tdb) ));
+       if (cs->Clear && !process_exists(crec->pid) && (errno == ESRCH)) {
+               NTSTATUS status;
+               DEBUG(2,("pid %s doesn't exist - deleting connections %d [%s]\n",
+                        procid_str_static(&crec->pid), crec->cnum,
+                        crec->servicename));
+
+               status = rec->delete_rec(rec);
+               if (!NT_STATUS_IS_OK(status)) {
+                       DEBUG(0,("count_fn: tdb_delete failed with error %s\n",
+                                nt_errstr(status)));
+               }
                return 0;
        }
-
-       if (strequal(crec.name, cs->name))
+       if (cs->name) {
+               /* We are counting all the connections to a given share. */
+               if (strequal(crec->servicename, cs->name)) {
+                       cs->curr_connections++;
+               }
+       } else {
+               /* We are counting all the connections. Static registrations
+                * like the lpq backgroud process and the smbd daemon process
+                * have a cnum of -1, so won't be counted here.
+                */
                cs->curr_connections++;
+       }
 
        return 0;
 }
@@ -108,83 +105,243 @@ static int count_fn( TDB_CONTEXT *the_tdb, TDB_DATA kbuf, TDB_DATA dbuf, void *u
  Claim an entry in the connections database.
 ****************************************************************************/
 
-BOOL claim_connection(connection_struct *conn,char *name,int max_connections,BOOL Clear)
+int count_current_connections( const char *sharename, BOOL clear  )
 {
-       struct connections_key key;
-       struct connections_data crec;
-       TDB_DATA kbuf, dbuf;
+       struct count_stat cs;
 
-       if (!tdb) {
-               tdb = tdb_open_log(lock_path("connections.tdb"), 0, TDB_CLEAR_IF_FIRST|TDB_DEFAULT, 
-                              O_RDWR | O_CREAT, 0644);
-       }
-       if (!tdb)
-               return False;
+       cs.mypid = sys_getpid();
+       cs.curr_connections = 0;
+       cs.name = sharename;
+       cs.Clear = clear;
 
        /*
-        * Enforce the max connections parameter.
+        * This has a race condition, but locking the chain before hand is worse
+        * as it leads to deadlock.
         */
 
-       if (max_connections > 0) {
-               struct count_stat cs;
+       if (connections_forall(count_fn, &cs) == -1) {
+               DEBUG(0,("count_current_connections: traverse of "
+                        "connections.tdb failed\n"));
+               DEBUGADD(0, ("count_current_connections: connection count of %d might not be accurate",
+                           cs.curr_connections));
+       }
 
-               cs.mypid = sys_getpid();
-               cs.curr_connections = 0;
-               cs.name = lp_servicename(SNUM(conn));
-               cs.Clear = Clear;
+       /* If the traverse failed part-way through, we at least return
+        * as many connections as we had already counted. If it failed
+        * right at the start, we will return 0, which is about all we
+        * can do anywway.
+        */
 
-               /*
-                * This has a race condition, but locking the chain before hand is worse
-                * as it leads to deadlock.
-                */
+       return cs.curr_connections;
+}
 
-               if (tdb_traverse(tdb, count_fn, &cs) == -1) {
-                       DEBUG(0,("claim_connection: traverse of connections.tdb failed with error %s.\n",
-                               tdb_errorstr(tdb) ));
-                       return False;
-               }
+/****************************************************************************
+ Count the number of connections open across all shares.
+****************************************************************************/
 
-               if (cs.curr_connections >= max_connections) {
-                       DEBUG(1,("claim_connection: Max connections (%d) exceeded for %s\n",
-                               max_connections, name ));
-                       return False;
-               }
-       }
+int count_all_current_connections(void)
+{
+       return count_current_connections(NULL, True /* clear stale entries */);
+}
+
+/****************************************************************************
+ Claim an entry in the connections database.
+****************************************************************************/
 
-       DEBUG(5,("claiming %s %d\n",name,max_connections));
+BOOL claim_connection(connection_struct *conn, const char *name,
+                     uint32 msg_flags)
+{
+       struct db_record *rec;
+       struct connections_data crec;
+       TDB_DATA dbuf;
+       NTSTATUS status;
 
-       ZERO_STRUCT(key);
-       key.pid = sys_getpid();
-       key.cnum = conn?conn->cnum:-1;
-       fstrcpy(key.name, name);
+       DEBUG(5,("claiming [%s]\n", name));
 
-       kbuf.dptr = (char *)&key;
-       kbuf.dsize = sizeof(key);
+       if (!(rec = connections_fetch_entry(NULL, conn, name))) {
+               DEBUG(0, ("connections_fetch_entry failed\n"));
+               return False;
+       }
 
        /* fill in the crec */
        ZERO_STRUCT(crec);
        crec.magic = 0x280267;
-       crec.pid = sys_getpid();
+       crec.pid = procid_self();
        crec.cnum = conn?conn->cnum:-1;
        if (conn) {
                crec.uid = conn->uid;
                crec.gid = conn->gid;
-               StrnCpy(crec.name,
-                       lp_servicename(SNUM(conn)),sizeof(crec.name)-1);
+               strlcpy(crec.servicename, lp_servicename(SNUM(conn)),
+                       sizeof(crec.servicename));
        }
        crec.start = time(NULL);
+       crec.bcast_msg_flags = msg_flags;
        
-       StrnCpy(crec.machine,remote_machine,sizeof(crec.machine)-1);
-       StrnCpy(crec.addr,conn?conn->client_address:client_addr(),sizeof(crec.addr)-1);
+       strlcpy(crec.machine,get_remote_machine_name(),sizeof(crec.machine));
+       strlcpy(crec.addr,conn?conn->client_address:client_addr(),
+               sizeof(crec.addr));
 
-       dbuf.dptr = (char *)&crec;
+       dbuf.dptr = (uint8 *)&crec;
        dbuf.dsize = sizeof(crec);
 
-       if (tdb_store(tdb, kbuf, dbuf, TDB_REPLACE) != 0) {
+       status = rec->store(rec, dbuf, TDB_REPLACE);
+
+       TALLOC_FREE(rec);
+
+       if (!NT_STATUS_IS_OK(status)) {
                DEBUG(0,("claim_connection: tdb_store failed with error %s.\n",
-                       tdb_errorstr(tdb) ));
+                        nt_errstr(status)));
+               return False;
+       }
+
+       return True;
+}
+
+BOOL register_message_flags(BOOL doreg, uint32 msg_flags)
+{
+       struct db_record *rec;
+       struct connections_data *pcrec;
+       NTSTATUS status;
+
+       DEBUG(10,("register_message_flags: %s flags 0x%x\n",
+               doreg ? "adding" : "removing",
+               (unsigned int)msg_flags ));
+
+       if (!(rec = connections_fetch_entry(NULL, NULL, NULL))) {
+               DEBUG(0, ("connections_fetch_entry failed\n"));
+               return False;
+       }
+
+       if (rec->value.dsize != sizeof(struct connections_data)) {
+               DEBUG(0,("register_message_flags: Got wrong record size\n"));
+               TALLOC_FREE(rec);
+               return False;
+       }
+
+       pcrec = (struct connections_data *)rec->value.dptr;
+       if (doreg)
+               pcrec->bcast_msg_flags |= msg_flags;
+       else
+               pcrec->bcast_msg_flags &= ~msg_flags;
+
+       status = rec->store(rec, rec->value, TDB_REPLACE);
+
+       TALLOC_FREE(rec);
+
+       if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(0,("register_message_flags: tdb_store failed: %s.\n",
+                        nt_errstr(status)));
                return False;
        }
 
+       DEBUG(10,("register_message_flags: new flags 0x%x\n",
+               (unsigned int)pcrec->bcast_msg_flags ));
+
        return True;
 }
+
+/*********************************************************************
+*********************************************************************/
+
+static TDB_DATA* make_pipe_rec_key( struct pipe_open_rec *prec )
+{
+       TDB_DATA *kbuf = NULL;
+       fstring key_string;
+       
+       if ( !prec )
+               return NULL;
+       
+       if ( (kbuf = TALLOC_P(prec, TDB_DATA)) == NULL ) {
+               return NULL;
+       }
+       
+       snprintf( key_string, sizeof(key_string), "%s/%d/%d",
+               prec->name, procid_to_pid(&prec->pid), prec->pnum );
+               
+       *kbuf = string_term_tdb_data(talloc_strdup(prec, key_string));
+       if (kbuf->dptr == NULL )
+               return NULL;
+
+       return kbuf;
+}
+
+/*********************************************************************
+*********************************************************************/
+
+static void fill_pipe_open_rec( struct pipe_open_rec *prec, smb_np_struct *p )
+{
+       prec->pid = pid_to_procid(sys_getpid());
+       prec->pnum = p->pnum;
+       prec->uid = geteuid();
+       fstrcpy( prec->name, p->name );
+
+       return;
+}
+
+/*********************************************************************
+*********************************************************************/
+
+BOOL store_pipe_opendb( smb_np_struct *p )
+{
+       struct db_record *dbrec;
+       struct pipe_open_rec *prec;
+       TDB_DATA *key;
+       TDB_DATA data;
+       BOOL ret = False;
+       
+       if ( (prec = TALLOC_P( NULL, struct pipe_open_rec)) == NULL ) {
+               DEBUG(0,("store_pipe_opendb: talloc failed!\n"));
+               return False;
+       }
+       
+       fill_pipe_open_rec( prec, p );
+       if ( (key = make_pipe_rec_key( prec )) == NULL ) {
+               goto done;
+       }
+       
+       data.dptr = (uint8 *)prec;
+       data.dsize = sizeof(struct pipe_open_rec);
+
+       if (!(dbrec = connections_fetch_record(prec, *key))) {
+               DEBUG(0, ("connections_fetch_record failed\n"));
+               goto done;
+       }
+
+       ret = NT_STATUS_IS_OK(dbrec->store(dbrec, data, TDB_REPLACE));
+       
+done:
+       TALLOC_FREE( prec );    
+       return ret;
+}
+
+/*********************************************************************
+*********************************************************************/
+
+BOOL delete_pipe_opendb( smb_np_struct *p )
+{
+       struct db_record *dbrec;
+       struct pipe_open_rec *prec;
+       TDB_DATA *key;
+       BOOL ret = False;
+       
+       if ( (prec = TALLOC_P( NULL, struct pipe_open_rec)) == NULL ) {
+               DEBUG(0,("store_pipe_opendb: talloc failed!\n"));
+               return False;
+       }
+       
+       fill_pipe_open_rec( prec, p );
+       if ( (key = make_pipe_rec_key( prec )) == NULL ) {
+               goto done;
+       }
+       
+       if (!(dbrec = connections_fetch_record(prec, *key))) {
+               DEBUG(0, ("connections_fetch_record failed\n"));
+               goto done;
+       }
+
+       ret = NT_STATUS_IS_OK(dbrec->delete_rec(dbrec));
+       
+done:
+       TALLOC_FREE( prec );
+       return ret;
+}