sql.c

While coding the rlm_sql_sqlite module I found that even though it fails to "connect" to the database it doesn't stop Freeradius from launching. While I understand that once it's started it may need to reconnect, would it not be a good behavior to stop execution under sql_init_socketpool() ? if (!success) { radlog(L_DBG, "rlm_sql (%s): Failed to connect to any SQL server.", inst->config->xlat_name); } Why not return -1 here within this block?