stupid coverity

doesn't realize that returning in a failure case with the mutex
unlocked, means that the mutex is unlocked

CID #1206498

diff --git a/src/modules/rlm_sql/sql.c b/src/modules/rlm_sql/sql.c
index 6e21c3bdc11e2fa03c970a84c36f479376a13c46..dc4eea7640ee5ac9741b8fa337f603bcbcc62ad3 100644 (file)
--- a/src/modules/rlm_sql/sql.c
+++ b/src/modules/rlm_sql/sql.c
@@ -545,6 +545,7 @@ void rlm_sql_query_log(rlm_sql_t const *inst, REQUEST *request, sql_acct_section
                ERROR("Couldn't open logfile '%s': %s", expanded, fr_syserror(errno));
 
                talloc_free(expanded);
+               /* coverity[missing_unlock] */
                return;
        }