From patchwork Mon Mar 19 20:01:13 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sandeep Patil X-Patchwork-Id: 887862 X-Patchwork-Delegate: petr.vorel@gmail.com Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (mailfrom) smtp.mailfrom=lists.linux.it (client-ip=2001:1418:10:5::2; helo=picard.linux.it; envelope-from=ltp-bounces+incoming=patchwork.ozlabs.org@lists.linux.it; receiver=) Authentication-Results: ozlabs.org; dmarc=fail (p=reject dis=none) header.from=google.com Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=google.com header.i=@google.com header.b="Wb9xyoPx"; dkim-atps=neutral Received: from picard.linux.it (picard.linux.it [IPv6:2001:1418:10:5::2]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 404n560Dvnz9s5H for ; Tue, 20 Mar 2018 07:01:27 +1100 (AEDT) Received: from picard.linux.it (localhost [IPv6:::1]) by picard.linux.it (Postfix) with ESMTP id 257EF3E7703 for ; Mon, 19 Mar 2018 21:01:23 +0100 (CET) X-Original-To: ltp@lists.linux.it Delivered-To: ltp@picard.linux.it Received: from in-7.smtp.seeweb.it (in-7.smtp.seeweb.it [217.194.8.7]) by picard.linux.it (Postfix) with ESMTP id 5FE623E76E3 for ; Mon, 19 Mar 2018 21:01:21 +0100 (CET) Received: from mail-pl0-x244.google.com (mail-pl0-x244.google.com [IPv6:2607:f8b0:400e:c01::244]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by in-7.smtp.seeweb.it (Postfix) with ESMTPS id 5BA5B200AEC for ; Mon, 19 Mar 2018 21:01:20 +0100 (CET) Received: by mail-pl0-x244.google.com with SMTP id u11-v6so7250614plq.1 for ; Mon, 19 Mar 2018 13:01:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=from:to:cc:subject:date:message-id; bh=SZki5QqH3Fe6iSFIlYdRkAEBC7uLRwIyZ/7z3ZHC8yI=; b=Wb9xyoPxP/+ikNlLWmxObGfvim3i1Igyr/OTIQKrP2PpWcxC0tF9llkZmiaooK+upl G3860ZWg26Dyd7D3TAsUpqrjM2GdOhm5KxHU9r6c6fB1rcbTL1B12bGgIRkqT2iJq6sb wowyfuy9nBu05jxLsgHwhDE3LVmq/z+8laBBG7IBpjg8V4LS7pmGvmtuEIj4rYDCKFuE ISAPeeVckAKihcDNJ66HgO3VL68hHydtwM/ny939FHtcF/EOvTGz0JiNc/T7PIqGzuaL XuvAqNH3BcIVD7iqIp1rOqRoaxywFktLTNvowip5crCRVXS2eM9AT0wVWCzdaEqwRXMs 85Sg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=SZki5QqH3Fe6iSFIlYdRkAEBC7uLRwIyZ/7z3ZHC8yI=; b=ZvhOmFikfMXpfKyoWERbI1HbMXw2XK+VsVZHeR1zMWK0WqDBb57AQTTyJV2EhRSiab oR8UhLHiazvXBLcZCLIhqL6emI7wy6KSIO9hrSA9y0JXZAUfE9U6VGOeTcoj2nu8U3L2 UC8+i3QkHOh6UeeB5GWWZegI4yeo4uOhsl5e5EPjTjR7fJ0O/qI0d7h/7KR22nHEUNJx f65b54uxxUDoX55DT5191gWgLmjFqZIPe57pnht/mmB3xPBYPIKloKVY2g5EJBPVW23R OJFunIMrVR1guHvDE72KvPDOHrKhBhxnEDnbENSRZDUY2tkU6nwu+gooOc4vsf3DEZQI NnEA== X-Gm-Message-State: AElRT7HAT/odGBKD8ORn+Iv4KOYWCXvK0e0Xd1dtzf+DEqCPiilumLM5 hnY6s16xOu6iXp2onG3fpwYmYIadcuY= X-Google-Smtp-Source: AG47ELv+qzQkpnhSwhcBjiq2qp83341UiBQ85M45p3dE0NewQkWdeV94VYzNKqZYIgof8adZY0znnA== X-Received: by 2002:a17:902:34f:: with SMTP id 73-v6mr13729302pld.55.1521489677726; Mon, 19 Mar 2018 13:01:17 -0700 (PDT) Received: from sspatil-desktop.mtv.corp.google.com ([2620:0:1000:1600:6e29:33f4:7c9a:4918]) by smtp.gmail.com with ESMTPSA id w10sm985722pgr.57.2018.03.19.13.01.15 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 19 Mar 2018 13:01:16 -0700 (PDT) From: Sandeep Patil To: ltp@lists.linux.it Date: Mon, 19 Mar 2018 13:01:13 -0700 Message-Id: <20180319200113.106692-1-sspatil@google.com> X-Mailer: git-send-email 2.16.2.804.g6dcf76e118-goog X-Virus-Scanned: clamav-milter 0.99.2 at in-7.smtp.seeweb.it X-Virus-Status: Clean X-Spam-Status: No, score=0.1 required=7.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU, SPF_PASS, T_RP_MATCHES_RCVD autolearn=disabled version=3.4.0 X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on in-7.smtp.seeweb.it Cc: chrubis@suze.cz, kernel-team@android.com Subject: [LTP] [PATCH] setregid: use common user and group names. X-BeenThere: ltp@lists.linux.it X-Mailman-Version: 2.1.18 Precedence: list List-Id: Linux Test Project List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: ltp-bounces+incoming=patchwork.ozlabs.org@lists.linux.it Sender: "ltp" Android systems do not have groups names 'users' 'sys' etc. Replace them with 'nobody' and 'daemon' for the purposes of setregid0[34] tests and make the nomenclature across both of these tests more consistent. After the change, both setregid03 and setregid04 can run on Android systems. Signed-off-by: Sandeep Patil --- .../kernel/syscalls/setregid/setregid03.c | 51 ++++++++++--------- .../kernel/syscalls/setregid/setregid04.c | 18 +++---- 2 files changed, 35 insertions(+), 34 deletions(-) diff --git a/testcases/kernel/syscalls/setregid/setregid03.c b/testcases/kernel/syscalls/setregid/setregid03.c index a51719e80..97102e0ee 100644 --- a/testcases/kernel/syscalls/setregid/setregid03.c +++ b/testcases/kernel/syscalls/setregid/setregid03.c @@ -41,7 +41,7 @@ static gid_t neg_one = -1; /* flag to tell parent if child passed or failed. */ static int flag; -struct group users, sys, root, bin; +struct group nobody_gr, daemon_gr, root_gr, bin_gr; struct passwd nobody; /* * The following structure contains all test data. Each structure in the array @@ -57,27 +57,28 @@ struct test_data_t { char *test_msg; } test_data[] = { { - &sys.gr_gid, &bin.gr_gid, &pass, &sys, &bin, - "After setregid(sys, bin),"}, { - &neg_one, &sys.gr_gid, &pass, &sys, &sys, "After setregid(-1, sys)"}, - { - &neg_one, &bin.gr_gid, &pass, &sys, &bin, "After setregid(-1, bin),"}, - { - &bin.gr_gid, &neg_one, &pass, &bin, &bin, "After setregid(bin, -1),"}, - { - &neg_one, &neg_one, &pass, &bin, &bin, "After setregid(-1, -1),"}, { - &neg_one, &bin.gr_gid, &pass, &bin, &bin, "After setregid(-1, bin),"}, - { - &bin.gr_gid, &neg_one, &pass, &bin, &bin, "After setregid(bin, -1),"}, - { - &bin.gr_gid, &bin.gr_gid, &pass, &bin, &bin, + &daemon_gr.gr_gid, &bin_gr.gr_gid, &pass, &daemon_gr, &bin_gr, + "After setregid(daemon, bin),"}, { + &neg_one, &daemon_gr.gr_gid, &pass, &daemon_gr, &daemon_gr, + "After setregid(-1, daemon)"}, { + &neg_one, &bin_gr.gr_gid, &pass, &daemon_gr, &bin_gr, + "After setregid(-1, bin),"}, { + &bin_gr.gr_gid, &neg_one, &pass, &bin_gr, &bin_gr, + "After setregid(bin, -1),"}, { + &neg_one, &neg_one, &pass, &bin_gr, &bin_gr, + "After setregid(-1, -1),"}, { + &neg_one, &bin_gr.gr_gid, &pass, &bin_gr, &bin_gr, + "After setregid(-1, bin),"}, { + &bin_gr.gr_gid, &neg_one, &pass, &bin_gr, &bin_gr, + "After setregid(bin, -1),"}, { + &bin_gr.gr_gid, &bin_gr.gr_gid, &pass, &bin_gr, &bin_gr, "After setregid(bin, bin),"}, { - &sys.gr_gid, &neg_one, &fail, &bin, &bin, "After setregid(sys, -1)"}, - { - &neg_one, &sys.gr_gid, &fail, &bin, &bin, "After setregid(-1, sys)"}, - { - &sys.gr_gid, &sys.gr_gid, &fail, &bin, &bin, - "After setregid(sys, sys)"},}; + &daemon_gr.gr_gid, &neg_one, &fail, &bin_gr, &bin_gr, + "After setregid(daemon, -1)"}, { + &neg_one, &daemon_gr.gr_gid, &fail, &bin_gr, &bin_gr, + "After setregid(-1, daemon)"}, { + &daemon_gr.gr_gid, &daemon_gr.gr_gid, &fail, &bin_gr, &bin_gr, + "After setregid(daemon, daemon)"},}; int TST_TOTAL = sizeof(test_data) / sizeof(test_data[0]); @@ -102,7 +103,7 @@ int main(int ac, char **av) tst_count = 0; /* set the appropriate ownership values */ - if (SETREGID(NULL, sys.gr_gid, bin.gr_gid) == -1) + if (SETREGID(NULL, daemon_gr.gr_gid, bin_gr.gr_gid) == -1) tst_brkm(TBROK, NULL, "Initial setregid failed"); SAFE_SETEUID(NULL, nobody.pw_uid); @@ -191,11 +192,11 @@ static void setup(void) tst_brkm(TBROK, NULL, "%s must be a valid group", #group); \ } \ GID16_CHECK(junk->gr_gid, setregid, NULL); \ - group = *(junk); \ + group ## _gr = *(junk); \ } while (0) - GET_GID(users); - GET_GID(sys); + GET_GID(nobody); + GET_GID(daemon); GET_GID(bin); TEST_PAUSE; diff --git a/testcases/kernel/syscalls/setregid/setregid04.c b/testcases/kernel/syscalls/setregid/setregid04.c index 0e0aae782..73f8bcb03 100644 --- a/testcases/kernel/syscalls/setregid/setregid04.c +++ b/testcases/kernel/syscalls/setregid/setregid04.c @@ -35,7 +35,7 @@ TCID_DEFINE(setregid04); static gid_t neg_one = -1; -static struct group users_gr, daemon_gr, root_gr, bin_gr; +static struct group nobody_gr, daemon_gr, root_gr, bin_gr; /* * The following structure contains all test data. Each structure in the array @@ -52,8 +52,8 @@ struct test_data_t { { &root_gr.gr_gid, &root_gr.gr_gid, &root_gr, &root_gr, "After setregid(root, root),"}, { - &users_gr.gr_gid, &neg_one, &users_gr, &root_gr, - "After setregid(users, -1)"}, { + &nobody_gr.gr_gid, &neg_one, &nobody_gr, &root_gr, + "After setregid(nobody, -1)"}, { &root_gr.gr_gid, &neg_one, &root_gr, &root_gr, "After setregid(root,-1),"}, { &neg_one, &neg_one, &root_gr, &root_gr, @@ -62,12 +62,12 @@ struct test_data_t { "After setregid(-1, root)"}, { &root_gr.gr_gid, &neg_one, &root_gr, &root_gr, "After setregid(root, -1),"}, { - &daemon_gr.gr_gid, &users_gr.gr_gid, &daemon_gr, &users_gr, - "After setregid(daemon, users)"}, { - &neg_one, &neg_one, &daemon_gr, &users_gr, + &daemon_gr.gr_gid, &nobody_gr.gr_gid, &daemon_gr, &nobody_gr, + "After setregid(daemon, nobody)"}, { + &neg_one, &neg_one, &daemon_gr, &nobody_gr, "After setregid(-1, -1)"}, { - &neg_one, &users_gr.gr_gid, &daemon_gr, &users_gr, - "After setregid(-1, users)"} + &neg_one, &nobody_gr.gr_gid, &daemon_gr, &nobody_gr, + "After setregid(-1, nobody)"} }; int TST_TOTAL = sizeof(test_data) / sizeof(test_data[0]); @@ -123,7 +123,7 @@ static void setup(void) tst_sig(FORK, DEF_HANDLER, cleanup); SAFE_GETGROUP(root); - SAFE_GETGROUP(users); + SAFE_GETGROUP(nobody); SAFE_GETGROUP(daemon); SAFE_GETGROUP(bin);