libadmin: Don't pass garbage to pts_GroupCreate
[openafs.git] / src / libadmin / test / pts.c
index 74f46b8..109dda9 100644 (file)
@@ -14,6 +14,7 @@
 #include <afsconfig.h>
 #include <afs/param.h>
 
+#include <roken.h>
 
 #include "pts.h"
 
@@ -90,7 +91,7 @@ GetUserAccessFromString(const char *in_str, const char *error_msg)
 int
 DoPtsGroupMemberAdd(struct cmd_syndesc *as, void *arock)
 {
-    typedef enum { USER, GROUP } DoPtsGroupMemberAdd_parm_t;
+    enum { USER, GROUP };
     afs_status_t st = 0;
     const char *user = as->parms[USER].items->data;
     const char *group = as->parms[GROUP].items->data;
@@ -105,7 +106,7 @@ DoPtsGroupMemberAdd(struct cmd_syndesc *as, void *arock)
 int
 DoPtsGroupOwnerChange(struct cmd_syndesc *as, void *arock)
 {
-    typedef enum { OWNER, GROUP } DoPtsGroupOwnerChange_parm_t;
+    enum { OWNER, GROUP };
     afs_status_t st = 0;
     const char *owner = as->parms[OWNER].items->data;
     const char *group = as->parms[GROUP].items->data;
@@ -120,11 +121,11 @@ DoPtsGroupOwnerChange(struct cmd_syndesc *as, void *arock)
 int
 DoPtsGroupCreate(struct cmd_syndesc *as, void *arock)
 {
-    typedef enum { OWNER, GROUP } DoPtsGroupCreate_parm_t;
+    enum { OWNER, GROUP };
     afs_status_t st = 0;
-    const char *owner = as->parms[OWNER].items->data;
-    const char *group = as->parms[GROUP].items->data;
-    int new_group_id;
+    char *owner = as->parms[OWNER].items->data;
+    char *group = as->parms[GROUP].items->data;
+    int new_group_id = 0;
 
     if (!pts_GroupCreate(cellHandle, group, owner, &new_group_id, &st)) {
        ERR_ST_EXT("pts_GroupMemberAdd", st);
@@ -170,7 +171,7 @@ Print_pts_GroupEntry_p(pts_GroupEntry_p entry, const char *prefix)
 int
 DoPtsGroupGet(struct cmd_syndesc *as, void *arock)
 {
-    typedef enum { GROUP } DoPtsGroupGet_parm_t;
+    enum { GROUP };
     afs_status_t st = 0;
     const char *group = as->parms[GROUP].items->data;
     pts_GroupEntry_t entry;
@@ -187,7 +188,7 @@ DoPtsGroupGet(struct cmd_syndesc *as, void *arock)
 int
 DoPtsGroupDelete(struct cmd_syndesc *as, void *arock)
 {
-    typedef enum { GROUP } DoPtsGroupDelete_parm_t;
+    enum { GROUP };
     afs_status_t st = 0;
     const char *group = as->parms[GROUP].items->data;
 
@@ -214,7 +215,7 @@ DoPtsGroupMaxGet(struct cmd_syndesc *as, void *arock)
 int
 DoPtsGroupMaxSet(struct cmd_syndesc *as, void *arock)
 {
-    typedef enum { MAX } DoPtsGroupMaxSet_parm_t;
+    enum { MAX };
     afs_status_t st = 0;
     const char *max = as->parms[MAX].items->data;
     int max_group_id;
@@ -231,7 +232,7 @@ DoPtsGroupMaxSet(struct cmd_syndesc *as, void *arock)
 int
 DoPtsGroupMemberList(struct cmd_syndesc *as, void *arock)
 {
-    typedef enum { GROUP } DoPtsGroupMemberList_parm_t;
+    enum { GROUP };
     afs_status_t st = 0;
     const char *group = as->parms[GROUP].items->data;
     void *iter;
@@ -260,7 +261,7 @@ DoPtsGroupMemberList(struct cmd_syndesc *as, void *arock)
 int
 DoPtsGroupMemberRemove(struct cmd_syndesc *as, void *arock)
 {
-    typedef enum { USER, GROUP } DoPtsGroupMemberRemove_parm_t;
+    enum { USER, GROUP };
     afs_status_t st = 0;
     const char *user = as->parms[USER].items->data;
     const char *group = as->parms[GROUP].items->data;
@@ -275,10 +276,10 @@ DoPtsGroupMemberRemove(struct cmd_syndesc *as, void *arock)
 int
 DoPtsGroupRename(struct cmd_syndesc *as, void *arock)
 {
-    typedef enum { GROUP, NEWNAME } DoPtsGroupRename_parm_t;
+    enum { GROUP, NEWNAME };
     afs_status_t st = 0;
     const char *group = as->parms[GROUP].items->data;
-    const char *new_name = as->parms[NEWNAME].items->data;
+    char *new_name = as->parms[NEWNAME].items->data;
 
     if (!pts_GroupRename(cellHandle, group, new_name, &st)) {
        ERR_ST_EXT("pts_GroupRename", st);
@@ -290,9 +291,9 @@ DoPtsGroupRename(struct cmd_syndesc *as, void *arock)
 int
 DoPtsGroupModify(struct cmd_syndesc *as, void *arock)
 {
-    typedef enum { GROUP, LISTSTATUS, LISTGROUPSOWNED, LISTMEMBERSHIP,
+    enum { GROUP, LISTSTATUS, LISTGROUPSOWNED, LISTMEMBERSHIP,
        LISTADD, LISTDELTE
-    } DoPtsGroupModify_parm_t;
+    };
     afs_status_t st = 0;
     const char *group = as->parms[GROUP].items->data;
     pts_GroupUpdateEntry_t entry;
@@ -323,9 +324,9 @@ DoPtsGroupModify(struct cmd_syndesc *as, void *arock)
 int
 DoPtsUserCreate(struct cmd_syndesc *as, void *arock)
 {
-    typedef enum { USER } DoPtsUserCreate_parm_t;
+    enum { USER };
     afs_status_t st = 0;
-    const char *user = as->parms[USER].items->data;
+    char *user = as->parms[USER].items->data;
     int new_user_id;
 
     if (!pts_UserCreate(cellHandle, user, &new_user_id, &st)) {
@@ -340,7 +341,7 @@ DoPtsUserCreate(struct cmd_syndesc *as, void *arock)
 int
 DoPtsUserDelete(struct cmd_syndesc *as, void *arock)
 {
-    typedef enum { USER } DoPtsUserDelete_parm_t;
+    enum { USER };
     afs_status_t st = 0;
     const char *user = as->parms[USER].items->data;
 
@@ -383,7 +384,7 @@ Print_pts_UserEntry_p(pts_UserEntry_p entry, const char *prefix)
 int
 DoPtsUserGet(struct cmd_syndesc *as, void *arock)
 {
-    typedef enum { USER } DoPtsUserGet_parm_t;
+    enum { USER };
     afs_status_t st = 0;
     const char *user = as->parms[USER].items->data;
     pts_UserEntry_t entry;
@@ -400,10 +401,10 @@ DoPtsUserGet(struct cmd_syndesc *as, void *arock)
 int
 DoPtsUserRename(struct cmd_syndesc *as, void *arock)
 {
-    typedef enum { USER, NEWNAME } DoPtsUserRename_parm_t;
+    enum { USER, NEWNAME };
     afs_status_t st = 0;
     const char *user = as->parms[USER].items->data;
-    const char *new_name = as->parms[NEWNAME].items->data;
+    char *new_name = as->parms[NEWNAME].items->data;
 
     if (!pts_UserRename(cellHandle, user, new_name, &st)) {
        ERR_ST_EXT("pts_UserRename", st);
@@ -415,13 +416,12 @@ DoPtsUserRename(struct cmd_syndesc *as, void *arock)
 int
 DoPtsUserModify(struct cmd_syndesc *as, void *arock)
 {
-    typedef enum { USER, GROUPQUOTA, LISTSTATUS, LISTGROUPSOWNED,
+    enum { USER, GROUPQUOTA, LISTSTATUS, LISTGROUPSOWNED,
        LISTMEMBERSHIP
-    } DoPtsUserModify_parm_t;
+    };
     afs_status_t st = 0;
     const char *user = as->parms[USER].items->data;
     pts_UserUpdateEntry_t entry;
-    int have_quota = 0;
     int have_list_status_perm = 0;
     int have_list_groups_owned_perm = 0;
     int have_list_membership_perm = 0;
@@ -432,7 +432,6 @@ DoPtsUserModify(struct cmd_syndesc *as, void *arock)
        entry.groupCreationQuota =
            GetIntFromString(as->parms[GROUPQUOTA].items->data, "bad quota");
        entry.flag = PTS_USER_UPDATE_GROUP_CREATE_QUOTA;
-       have_quota = 1;
     }
 
     if (as->parms[LISTSTATUS].items) {
@@ -494,7 +493,7 @@ DoPtsUserMaxGet(struct cmd_syndesc *as, void *arock)
 int
 DoPtsUserMaxSet(struct cmd_syndesc *as, void *arock)
 {
-    typedef enum { MAX } DoPtsUserMaxSet_parm_t;
+    enum { MAX };
     afs_status_t st = 0;
     const char *max = as->parms[MAX].items->data;
     int max_user_id;
@@ -511,7 +510,7 @@ DoPtsUserMaxSet(struct cmd_syndesc *as, void *arock)
 int
 DoPtsUserMemberList(struct cmd_syndesc *as, void *arock)
 {
-    typedef enum { USER } DoPtsUserMemberList_parm_t;
+    enum { USER };
     afs_status_t st = 0;
     const char *user = as->parms[USER].items->data;
     void *iter;
@@ -541,7 +540,7 @@ DoPtsUserMemberList(struct cmd_syndesc *as, void *arock)
 int
 DoPtsOwnedGroupList(struct cmd_syndesc *as, void *arock)
 {
-    typedef enum { USER } DoPtsOwnedGroupList_parm_t;
+    enum { USER };
     afs_status_t st = 0;
     const char *user = as->parms[USER].items->data;
     void *iter;