Give functions a return code, even if the last thing they do is call

	exit()

diff --git a/src/main/modcall.c b/src/main/modcall.c
index e07f142..e562377 100644 (file)
--- a/src/main/modcall.c
+++ b/src/main/modcall.c
@@ -122,6 +122,7 @@ static int str2rcode(const char *s, const char *filename, int lineno)
                        filename, lineno, s);
                exit(1);
        }
+       return -1;
 }
 
 static const char *rcode2str[] = {
@@ -148,6 +149,7 @@ static int str2action(const char *s, const char *filename, int lineno)
                                filename, lineno, s);
                exit(1);
        }
+       return -1;
 }
 
 static const char *action2str(int action)

diff --git a/src/main/session.c b/src/main/session.c
index 1926dfe..7c19793 100644 (file)
--- a/src/main/session.c
+++ b/src/main/session.c
@@ -216,4 +216,5 @@ int rad_check_ts(uint32_t nasaddr, int portnum, const char *user,
         *      Exit - 2 means "some error occured".
         */
        exit(2);
+       return -1;
 }

diff --git a/src/modules/rlm_sql/sql.c b/src/modules/rlm_sql/sql.c
index a4a38a2..0cc5bf8 100644 (file)
--- a/src/modules/rlm_sql/sql.c
+++ b/src/modules/rlm_sql/sql.c
@@ -389,7 +389,7 @@ static int sql_check_ts(SQL_ROW row) {
         *      Exit - 2 means "some error occured".
         */
        exit(2);
-
+       return -1;
 }