Removed knowledge of sqlsocket->id

The drivers have no business using it

diff --git a/src/modules/rlm_sql/drivers/rlm_sql_firebird/sql_firebird.c b/src/modules/rlm_sql/drivers/rlm_sql_firebird/sql_firebird.c
index 11b6ba662fdcdff894b726c444ca8943d1d539c4..d7319555e070060e6af4df929f064733dea4f7ea 100644 (file)
--- a/src/modules/rlm_sql/drivers/rlm_sql_firebird/sql_firebird.c
+++ b/src/modules/rlm_sql/drivers/rlm_sql_firebird/sql_firebird.c
@@ -90,8 +90,6 @@ static int sql_destroy_socket(SQLSOCK *sqlsocket, SQL_CONFIG *config)
 static int sql_query(SQLSOCK *sqlsocket, SQL_CONFIG *config, char *querystr) {
     rlm_sql_firebird_sock *firebird_sock = sqlsocket->conn;
     int deadlock=0;
- if (config->sqltrace)
-        radlog(L_DBG, "sock_id %i: query:  %s", sqlsocket->id,querystr);
 
 #ifdef _PTHREAD_H
  pthread_mutex_lock(&firebird_sock->mut);
@@ -102,14 +100,13 @@ TryAgain:
 //Try again query when deadlock, beacuse in any case it will be retried.
 // but may be lost for short sessions
    if ((firebird_sock->sql_code==DEADLOCK_SQL_CODE) && !deadlock) {
-      radlog(L_DBG,"sock_id %i: deadlock. Retry query %s\n",sqlsocket->id,querystr);
+      radlog(L_DBG,"sock_id deadlock. Retry query %s\n",querystr);
 //For non READ_COMMITED transactions put rollback here
 // fb_rollback(sock);
       deadlock=1;
       goto TryAgain;
    }
-   radlog(L_ERR, "sock_id %i: rlm_sql_firebird,sql_query error:sql_code=%li, error='%s', query=%s\n",
-     sqlsocket->id,
+   radlog(L_ERR, "sock_id rlm_sql_firebird,sql_query error:sql_code=%li, error='%s', query=%s\n",
      firebird_sock->sql_code,
      firebird_sock->lasterror,
      querystr);
@@ -185,7 +182,6 @@ static int sql_num_fields(SQLSOCK *sqlsocket, SQL_CONFIG *config) {
  *************************************************************************/
 static int sql_num_rows(SQLSOCK *sqlsocket, SQL_CONFIG *config) {
     int res=sql_affected_rows(sqlsocket, config);
-    radlog(L_DBG,"sock_id %i: sql_num_rows: %i\n",sqlsocket->id,res);
     return res;
 }
 
@@ -295,7 +291,6 @@ static int sql_affected_rows(SQLSOCK *sqlsocket, SQL_CONFIG *config) {
  int affected_rows=fb_affected_rows(sqlsocket->conn);
  if (affected_rows<0)
    radlog(L_ERR, "sql_affected_rows, rlm_sql_firebird. error:%s\n", sql_error(sqlsocket,config));
- radlog(L_DBG,"sock_id %i: affected_rows: %i\n",sqlsocket->id,affected_rows);
  return affected_rows;
 }
 

diff --git a/src/modules/rlm_sql/drivers/rlm_sql_freetds/sql_freetds.c b/src/modules/rlm_sql/drivers/rlm_sql_freetds/sql_freetds.c
index bb044cd2c83ab0bd158ab37b40e8e875290b61ef..a56576a66f4b3a017431f103ae864dad11ba4854 100644 (file)
--- a/src/modules/rlm_sql/drivers/rlm_sql_freetds/sql_freetds.c
+++ b/src/modules/rlm_sql/drivers/rlm_sql_freetds/sql_freetds.c
@@ -92,7 +92,7 @@ static int sql_init_socket(SQLSOCK *sqlsocket, SQL_CONFIG *config)
        freetds_sock = sqlsocket->conn;
        memset(freetds_sock, 0, sizeof(*freetds_sock));
        
-       radlog(L_INFO, "rlm_sql_freetds (%s): Starting connect to FreeTDS/MSSQL server for #%d", config->xlat_name, sqlsocket->id);
+       radlog(L_INFO, "rlm_sql_freetds (%s): Starting connect to FreeTDS/MSSQL server", config->xlat_name);
        
        if (!(login = dblogin())) {
                radlog(L_ERR, "rlm_sql_freetds (%s): Unable to allocate login record", config->xlat_name);

diff --git a/src/modules/rlm_sql/drivers/rlm_sql_mysql/sql_mysql.c b/src/modules/rlm_sql/drivers/rlm_sql_mysql/sql_mysql.c
index 85e3e76042f69e82b0027d9f52ab0ac0e32b6c72..48d437a759beb129e648e83eb9565609d2643b6d 100644 (file)
--- a/src/modules/rlm_sql/drivers/rlm_sql_mysql/sql_mysql.c
+++ b/src/modules/rlm_sql/drivers/rlm_sql_mysql/sql_mysql.c
@@ -76,8 +76,7 @@ static int sql_init_socket(SQLSOCK *sqlsocket, SQL_CONFIG *config)
        mysql_sock = sqlsocket->conn;
        memset(mysql_sock, 0, sizeof(*mysql_sock));
 
-       radlog(L_INFO, "rlm_sql_mysql: Starting connect to MySQL server for #%d",
-                       sqlsocket->id);
+       radlog(L_INFO, "rlm_sql_mysql: Starting connect to MySQL server");
 
        mysql_init(&(mysql_sock->conn));
        mysql_options(&(mysql_sock->conn), MYSQL_READ_DEFAULT_GROUP, "freeradius");

diff --git a/src/modules/rlm_sql/drivers/rlm_sql_sqlite/sql_sqlite.c b/src/modules/rlm_sql/drivers/rlm_sql_sqlite/sql_sqlite.c
index d7bd2b992ff95abcf11745d5f021b19552b6c44d..8d7e467cf7dd596ace8e5987783195235e5b372f 100644 (file)
--- a/src/modules/rlm_sql/drivers/rlm_sql_sqlite/sql_sqlite.c
+++ b/src/modules/rlm_sql/drivers/rlm_sql_sqlite/sql_sqlite.c
@@ -67,8 +67,8 @@ static int sql_init_socket(SQLSOCK *sqlsocket, SQL_CONFIG *config)
                         radius_dir);
                filename = buffer;
        }
-       radlog(L_INFO, "rlm_sql_sqlite: Opening sqlite database %s for #%d",
-              filename, sqlsocket->id);
+       radlog(L_INFO, "rlm_sql_sqlite: Opening sqlite database %s",
+              filename);
        
        status = sqlite3_open(filename, &sqlite_sock->pDb);
        radlog(L_INFO, "rlm_sql_sqlite: sqlite3_open() = %d\n", status);