From dc9b7056674910f469ba97ad67944a08b388ab6a Mon Sep 17 00:00:00 2001 From: Günther Deschner Date: Wed, 3 Jun 2009 14:31:20 +0200 Subject: nss_wrapper: add coherency tests for get{gr,pw}{nam,id}. Guenther --- lib/nss_wrapper/testsuite.c | 86 +++++++++++++++++++++++++++++++++++++++------ 1 file changed, 75 insertions(+), 11 deletions(-) (limited to 'lib/nss_wrapper') diff --git a/lib/nss_wrapper/testsuite.c b/lib/nss_wrapper/testsuite.c index a98c141e3e..8a5fae0e67 100644 --- a/lib/nss_wrapper/testsuite.c +++ b/lib/nss_wrapper/testsuite.c @@ -53,7 +53,8 @@ static void print_passwd(struct passwd *pwd) static bool test_nwrap_getpwnam(struct torture_context *tctx, - const char *name) + const char *name, + struct passwd *pwd_p) { struct passwd *pwd; @@ -64,11 +65,16 @@ static bool test_nwrap_getpwnam(struct torture_context *tctx, print_passwd(pwd); } + if (pwd_p) { + copy_passwd(tctx, pwd, pwd_p); + } + return pwd ? true : false; } static bool test_nwrap_getpwuid(struct torture_context *tctx, - uid_t uid) + uid_t uid, + struct passwd *pwd_p) { struct passwd *pwd; @@ -79,6 +85,10 @@ static bool test_nwrap_getpwuid(struct torture_context *tctx, print_passwd(pwd); } + if (pwd_p) { + copy_passwd(tctx, pwd, pwd_p); + } + return pwd ? true : false; } @@ -122,7 +132,8 @@ static void print_group(struct group *grp) } static bool test_nwrap_getgrnam(struct torture_context *tctx, - const char *name) + const char *name, + struct group *grp_p) { struct group *grp; @@ -133,11 +144,16 @@ static bool test_nwrap_getgrnam(struct torture_context *tctx, print_group(grp); } + if (grp_p) { + copy_group(tctx, grp, grp_p); + } + return grp ? true : false; } static bool test_nwrap_getgrgid(struct torture_context *tctx, - gid_t gid) + gid_t gid, + struct group *grp_p) { struct group *grp; @@ -148,6 +164,10 @@ static bool test_nwrap_getgrgid(struct torture_context *tctx, print_group(grp); } + if (grp_p) { + copy_group(tctx, grp, grp_p); + } + return grp ? true : false; } @@ -187,20 +207,38 @@ static bool test_nwrap_enum_passwd(struct torture_context *tctx, return true; } +static bool torture_assert_passwd_equal(struct torture_context *tctx, + const struct passwd *p1, + const struct passwd *p2, + const char *comment) +{ + torture_assert_str_equal(tctx, p1->pw_name, p2->pw_name, comment); + torture_assert_str_equal(tctx, p1->pw_passwd, p2->pw_passwd, comment); + torture_assert_int_equal(tctx, p1->pw_uid, p2->pw_uid, comment); + torture_assert_int_equal(tctx, p1->pw_gid, p2->pw_gid, comment); + torture_assert_str_equal(tctx, p1->pw_gecos, p2->pw_gecos, comment); + torture_assert_str_equal(tctx, p1->pw_dir, p2->pw_dir, comment); + torture_assert_str_equal(tctx, p1->pw_shell, p2->pw_shell, comment); + + return true; +} + static bool test_nwrap_passwd(struct torture_context *tctx) { int i; - struct passwd *pwd; + struct passwd *pwd, pwd1, pwd2; size_t num_pwd; torture_assert(tctx, test_nwrap_enum_passwd(tctx, &pwd, &num_pwd), "failed to enumerate passwd"); for (i=0; i < num_pwd; i++) { - torture_assert(tctx, test_nwrap_getpwnam(tctx, pwd[i].pw_name), + torture_assert(tctx, test_nwrap_getpwnam(tctx, pwd[i].pw_name, &pwd1), "failed to call getpwnam for enumerated user"); - torture_assert(tctx, test_nwrap_getpwuid(tctx, pwd[i].pw_uid), + torture_assert(tctx, test_nwrap_getpwuid(tctx, pwd[i].pw_uid, &pwd2), "failed to call getpwuid for enumerated user"); + torture_assert_passwd_equal(tctx, &pwd1, &pwd2, + "getpwnam and getpwuid gave different results"); } return true; @@ -239,6 +277,30 @@ static bool test_nwrap_enum_group(struct torture_context *tctx, *num_grp_p = num_grp; } + return true; +} + +static bool torture_assert_group_equal(struct torture_context *tctx, + const struct group *g1, + const struct group *g2, + const char *comment) +{ + int i; + torture_assert_str_equal(tctx, g1->gr_name, g2->gr_name, comment); + torture_assert_str_equal(tctx, g1->gr_passwd, g2->gr_passwd, comment); + torture_assert_int_equal(tctx, g1->gr_gid, g2->gr_gid, comment); + if (g1->gr_mem && !g2->gr_mem) { + return false; + } + if (!g1->gr_mem && g2->gr_mem) { + return false; + } + if (!g1->gr_mem && !g2->gr_mem) { + return true; + } + for (i=0; g1->gr_mem[i] && g2->gr_mem[i]; i++) { + torture_assert_str_equal(tctx, g1->gr_mem[i], g2->gr_mem[i], comment); + } return true; } @@ -246,17 +308,19 @@ static bool test_nwrap_enum_group(struct torture_context *tctx, static bool test_nwrap_group(struct torture_context *tctx) { int i; - struct group *grp; + struct group *grp, grp1, grp2; size_t num_grp; torture_assert(tctx, test_nwrap_enum_group(tctx, &grp, &num_grp), "failed to enumerate group"); for (i=0; i < num_grp; i++) { - torture_assert(tctx, test_nwrap_getgrnam(tctx, grp[i].gr_name), + torture_assert(tctx, test_nwrap_getgrnam(tctx, grp[i].gr_name, &grp1), "failed to call getgrnam for enumerated user"); - torture_assert(tctx, test_nwrap_getgrgid(tctx, grp[i].gr_gid), + torture_assert(tctx, test_nwrap_getgrgid(tctx, grp[i].gr_gid, &grp2), "failed to call getgrgid for enumerated user"); + torture_assert_group_equal(tctx, &grp1, &grp2, + "getgrnam and getgrgid gave different results"); } return true; @@ -312,7 +376,7 @@ static bool test_nwrap_membership_user(struct torture_context *tctx, "failed to test getgrouplist"); for (g=0; g < num_user_groups; g++) { - torture_assert(tctx, test_nwrap_getgrgid(tctx, user_groups[g]), + torture_assert(tctx, test_nwrap_getgrgid(tctx, user_groups[g], NULL), "failed to find the group the user is a member of"); } -- cgit