Query may be an empty string, too.

We should really fix that in the init function

diff --git a/src/modules/rlm_sql/sql.c b/src/modules/rlm_sql/sql.c
index 34ab8dd..fae7171 100644 (file)
--- a/src/modules/rlm_sql/sql.c
+++ b/src/modules/rlm_sql/sql.c
@@ -86,7 +86,7 @@ static void *mod_conn_create(void *instance)
                return NULL;
        }
 
-       if (inst->config->open_query) {
+       if (inst->config->open_query && *inst->config->open_query) {
                if (rlm_sql_select_query(&handle, inst, inst->config->open_query)) {
                        goto fail;
                }