catch things we know are wrong, instead of requiring SELECT

which allows CALL to be used

diff --git a/src/modules/rlm_sql_map/rlm_sql_map.c b/src/modules/rlm_sql_map/rlm_sql_map.c
index 5443cf3c4f0ab21a9d0f6cc7c816adb0b1a46ed2..e720c8e7aa0158a69399f44bd0b180907d54f9e5 100644 (file)
--- a/src/modules/rlm_sql_map/rlm_sql_map.c
+++ b/src/modules/rlm_sql_map/rlm_sql_map.c
@@ -317,8 +317,10 @@ static int mod_bootstrap(CONF_SECTION *conf, void *instance)
 
        while (isspace((int) *p)) p++;
 
-       if (strncasecmp(p, "select", 6) != 0) {
-               cf_log_err_cs(conf, "'query' MUST be 'SELECT ...', not 'INSERT' or 'UPDATE'");
+       if ((strncasecmp(p, "insert", 6) == 0) ||
+           (strncasecmp(p, "update", 6) == 0) ||
+           (strncasecmp(p, "delete", 6) == 0)) {
+               cf_log_err_cs(conf, "'query' MUST be 'SELECT ...', not 'INSERT', 'UPDATE', or 'DELETE'");
                return -1;
        }