trylock may return positive or negative on failure. <sigh>

	Patch from Liyan Tan

diff --git a/src/modules/rlm_sql/sql.c b/src/modules/rlm_sql/sql.c
index 174dc35..180ce7c 100644 (file)
--- a/src/modules/rlm_sql/sql.c
+++ b/src/modules/rlm_sql/sql.c
@@ -197,7 +197,7 @@ SQLSOCK * sql_get_socket(SQL_INST * inst)
                 *
                 *      If it isn't used, then grab it ourselves.
                 */
-               if (pthread_mutex_trylock(&cur->mutex) < 0) {
+               if (pthread_mutex_trylock(&cur->mutex) != 0) {
                        goto next;
                } /* else we now have the lock */
 #endif