sql.c

[Gabriel Blanchard]

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?