git: 1a05d9519de6 - main - libcasper/cap_grp tests: Reset the group database handle

Mark Johnston markj at FreeBSD.org
Thu Jan 21 19:30:56 UTC 2021


The branch main has been updated by markj:

URL: https://cgit.FreeBSD.org/src/commit/?id=1a05d9519de62cc0f7719a995fded0926d8b2747

commit 1a05d9519de62cc0f7719a995fded0926d8b2747
Author:     Mark Johnston <markj at FreeBSD.org>
AuthorDate: 2021-01-21 19:30:19 +0000
Commit:     Mark Johnston <markj at FreeBSD.org>
CommitDate: 2021-01-21 19:30:19 +0000

    libcasper/cap_grp tests: Reset the group database handle
    
    Some tests verify that the capgrp capability does not permit calls to
    setgrent(3), but all tests need to ensure that they reset the
    capability's group database handle, otherwise the local process and
    casper process will be out of sync.
    
    The cap_pwd tests already handle this.
    
    Sponsored by:   The FreeBSD Foundation
    MFC after:      1 month
---
 lib/libcasper/services/cap_grp/tests/grp_test.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/lib/libcasper/services/cap_grp/tests/grp_test.c b/lib/libcasper/services/cap_grp/tests/grp_test.c
index e2bb5e22eb1a..1e84e293e214 100644
--- a/lib/libcasper/services/cap_grp/tests/grp_test.c
+++ b/lib/libcasper/services/cap_grp/tests/grp_test.c
@@ -321,6 +321,8 @@ test_cmds(cap_channel_t *origcapgrp)
 	capgrp = cap_clone(origcapgrp);
 	CHECK(capgrp != NULL);
 
+	cap_setgrent(capgrp);
+
 	cmds[0] = "getgrent";
 	cmds[1] = "getgrent_r";
 	cmds[2] = "getgrnam";
@@ -361,6 +363,8 @@ test_cmds(cap_channel_t *origcapgrp)
 	capgrp = cap_clone(origcapgrp);
 	CHECK(capgrp != NULL);
 
+	cap_setgrent(capgrp);
+
 	cmds[0] = "getgrent";
 	cmds[1] = "getgrent_r";
 	cmds[2] = "getgrnam";


More information about the dev-commits-src-all mailing list