Invert the sense of the return from addmember() or addclub()
[mspang/pyceo.git] / src / addmember.c
index 382e19f..9f35078 100644 (file)
@@ -45,7 +45,7 @@ static void usage() {
     exit(2);
 }
 
-void addmember() {
+int addmember() {
     int krb_ok, user_ok, group_ok, home_ok, quota_ok;
     int id;
     char homedir[1024];
@@ -59,7 +59,7 @@ void addmember() {
         deny("user %s already exists", userid);
 
     if (ceo_read_password(password, sizeof(password), use_stdin))
-        return;
+        return 1;
 
     ceo_krb5_init();
     ceo_ldap_init();
@@ -144,7 +144,7 @@ void addmember() {
     ceo_ldap_cleanup();
     ceo_krb5_cleanup();
 
-    return !krb_ok && !user_ok && !group_ok && !home_ok && quota_ok;
+    return krb_ok || user_ok || group_ok || home_ok || quota_ok;
 }
 
 int main(int argc, char *argv[]) {
@@ -190,7 +190,5 @@ int main(int argc, char *argv[]) {
     if (argc - optind)
         program = argv[optind++];
 
-    addmember();
-
-    return 0;
+    return addmember();
 }