Merge pull request #326 from edk0/filter-part

[solanum.git] / authd / authd.c

diff --git a/authd/authd.c b/authd/authd.c
index b122db2ee71f642f0c942b809cd1ddecea442a57..974b6a020d672b9b6827bf18dbf25da839c72ca0 100644 (file)
--- a/authd/authd.c
+++ b/authd/authd.c
@@ -25,6 +25,7 @@
 
 #define MAXPARA 10
 
+static void error_cb(rb_helper *helper) __attribute__((noreturn));
 static void handle_reload(int parc, char *parv[]);
 static void handle_stat(int parc, char *parv[]);
 static void handle_options(int parc, char *parv[]);
@@ -53,7 +54,7 @@ static void
 handle_stat(int parc, char *parv[])
 {
        authd_stat_handler handler;
-       long lrid;
+       unsigned long long rid;
 
        if(parc < 3)
        {
@@ -61,16 +62,16 @@ handle_stat(int parc, char *parv[])
                return;
        }
 
-       if((lrid = strtol(parv[1], NULL, 16)) > UINT32_MAX)
+       if((rid = strtoull(parv[1], NULL, 16)) > UINT32_MAX)
        {
-               warn_opers(L_CRIT, "BUG: handle_stat got a rid that was too large: %lx", lrid);
+               warn_opers(L_CRIT, "BUG: handle_stat got a rid that was too large: %s", parv[1]);
                return;
        }
 
        if (!(handler = authd_stat_handlers[(unsigned char)parv[2][0]]))
                return;
 
-       handler((uint32_t)lrid, parv[2][0]);
+       handler((uint32_t)rid, parv[2][0]);
 }
 
 static void
@@ -78,9 +79,9 @@ handle_options(int parc, char *parv[])
 {
        struct auth_opts_handler *handler;
 
-       if(parc < 3)
+       if(parc < 2)
        {
-               warn_opers(L_CRIT, "BUG: handle_options received too few parameters (at least 3 expected, got %d)", parc);
+               warn_opers(L_CRIT, "BUG: handle_options received too few parameters (at least 2 expected, got %d)", parc);
                return;
        }
 
@@ -187,12 +188,6 @@ setup_signals(void)
 #endif
 }
 
-static void
-do_exit(void)
-{
-       destroy_providers();
-}
-
 int
 main(int argc, char *argv[])
 {
@@ -208,15 +203,18 @@ main(int argc, char *argv[])
        rb_set_time();
        setup_signals();
 
-       authd_option_handlers = rb_dictionary_create("authd options handlers", strcasecmp);
+       authd_option_handlers = rb_dictionary_create("authd options handlers", rb_strcasecmp);
 
        init_resolver();
        init_providers();
        rb_init_prng(NULL, RB_PRNG_DEFAULT);
 
-       atexit(do_exit);
-
        rb_helper_loop(authd_helper, 0);
 
+       /*
+        * XXX this function will never be called from here -- is it necessary?
+        */
+       destroy_providers();
+
        return 0;
 }