Return rather than use the same ptr twice
authorAlan T. DeKok <aland@freeradius.org>
Mon, 12 Oct 2009 10:14:09 +0000 (12:14 +0200)
committerAlan T. DeKok <aland@freeradius.org>
Mon, 12 Oct 2009 10:14:31 +0000 (12:14 +0200)
src/modules/rlm_sql/rlm_sql.c

index d4e3738..89146e1 100644 (file)
@@ -1323,7 +1323,7 @@ static int rlm_sql_accounting(void *instance, REQUEST * request) {
                                                        radius_xlat(logstr, sizeof(logstr), "stop packet with zero session length. [user '%{User-Name}', nas '%{NAS-IP-Address}']", request, NULL);
                                                        radlog_request(L_ERR, 0, request, "%s", logstr);
                                                        sql_release_socket(inst, sqlsocket);
-                                                       ret = RLM_MODULE_NOOP;
+                                                       return RLM_MODULE_NOOP;
                                                }
 #endif