From patchwork Tue Apr 26 19:36:17 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Seth Forshee X-Patchwork-Id: 8944491 Return-Path: X-Original-To: patchwork-selinux@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 7C7A19F441 for ; Tue, 26 Apr 2016 20:12:36 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id A7EB02014A for ; Tue, 26 Apr 2016 20:12:35 +0000 (UTC) Received: from emsm-gh1-uea11.nsa.gov (emsm-gh1-uea11.nsa.gov [8.44.101.9]) by mail.kernel.org (Postfix) with ESMTP id 7C8ED20138 for ; Tue, 26 Apr 2016 20:12:34 +0000 (UTC) X-IronPort-AV: E=Sophos;i="5.24,538,1454976000"; d="scan'208";a="15644008" IronPort-PHdr: =?us-ascii?q?9a23=3ACF+nHxM2e8F01SEcxOkl6mtUPXoX/o7sNwtQ0KIM?= =?us-ascii?q?zox0LPTyrarrMEGX3/hxlliBBdydsKIUzbWH+Pm7ByQp2tWojjMrSNR0TRgLiM?= =?us-ascii?q?EbzUQLIfWuLgnFFsPsdDEwB89YVVVorDmROElRH9viNRWJ+iXhpQAbFhi3Dwdp?= =?us-ascii?q?POO9QteU1JTnkb/jsMSIPU1hv3mUX/BbFF2OtwLft80b08NJC50a7V/3mEZOYP?= =?us-ascii?q?lc3mhyJFiezF7W78a0+4N/oWwL46pyv+YJa6jxfrw5QLpEF3xmdjltvIy4/SXE?= =?us-ascii?q?GEGi/HoXGlpQ2jBJDgTI9hTzWN255ibwt+dx1TOfFd3zTKsvWDOkqaxsTUmswA?= =?us-ascii?q?QKMSJx13/akNdslq9WvFr1pBF538jTfoyPL+Zkc67BVdwfTGtFGM1WUnoFSqq4?= =?us-ascii?q?ZpADBusbdcVVtYb6oRNapBS9LROhCOPm1nlDgXqgmeUf0ucsWTrA3RAyEshG5G?= =?us-ascii?q?/SrcnnOb06S+25zKDUiz7EaqUSkTv4rYTFbB0lieuBUbJ5bYzazkxrXzjFh1KQ?= =?us-ascii?q?tIWtGzLdgv8Nt2mz7Od6U++rzWk9pFc16hSm2MYgwrbOgY4VzU3f+G0tzJkwOo?= =?us-ascii?q?eQSUh1e9eoHZJc8SaAON0yCuk4Qmhu8AA90KcHotbvdy8P0Ik7ygTEZfuda4WB?= =?us-ascii?q?6R/LW+OYIDM+j3VgLvb3rhK/4UGmxuDmHve1ylEC+jREiN7kpH0L1gKV7smBV+?= =?us-ascii?q?s7+V2unyuMgUSbyOhCaWM9j6OTf5woyb0YjYcIvALIGSqg3A3UgaGYPmEt6/Sv?= =?us-ascii?q?7ayzYLHvt4ObMadsjQb1MqUjgMGzGqI+PxRYGyDR26L0/qf+9kCxbrRHivEt2O?= =?us-ascii?q?GNup7bKMAXvYa3AgJR3p0u8FC4FTjwg/oCmnxSF19ZeFqpiJLyO1zDaKTjAOu/?= =?us-ascii?q?xVqriixryv3uNbTnBZPMJXHH1rzmeOAuuAZn1AMvwIUHtNpvAbYbLae2AxepuQ?= =?us-ascii?q?=3D=3D?= X-IPAS-Result: =?us-ascii?q?A2EJBQBzyh9X/wHyM5BeHAGCcCuBULt3H4F2hUVMAQEBAQE?= =?us-ascii?q?BAgJiJ4ItfVs9AQEBAwECDxUTBgEBDCALAQIDCQEBFykICAMBLQMBBQELEQYBB?= =?us-ascii?q?wsFGAQBiAgBpVWBMT4xik+FKAEEjEYBAQgBAQEBFgYKhA2CCohaEQEDhXEBjVh?= =?us-ascii?q?0iUiBVYxEiSolhUBFjS0wgQ5iggUbgWlOAYdwCBeBHgEBAQ?= Received: from unknown (HELO tarius.tycho.ncsc.mil) ([144.51.242.1]) by emsm-gh1-uea11.nsa.gov with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 26 Apr 2016 20:12:33 +0000 Received: from prometheus.infosec.tycho.ncsc.mil (prometheus [192.168.25.40]) by tarius.tycho.ncsc.mil (8.14.4/8.14.4) with ESMTP id u3QKCW3t021619; Tue, 26 Apr 2016 16:12:32 -0400 Received: from tarius.tycho.ncsc.mil (tarius.infosec.tycho.ncsc.mil [144.51.242.1]) by prometheus.infosec.tycho.ncsc.mil (8.15.2/8.15.2) with ESMTP id u3QJarjF172291 for ; Tue, 26 Apr 2016 15:36:53 -0400 Received: from goalie.tycho.ncsc.mil (goalie [144.51.242.250]) by tarius.tycho.ncsc.mil (8.14.4/8.14.4) with ESMTP id u3QJakxn011587 for ; Tue, 26 Apr 2016 15:36:53 -0400 X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: A0A+BQD9wR9X/yUp0ApeHYJwK4FQt3CEEYYPAoFATAEBAQEBAWYnhEIBAQEDEhUZAQE3AQ9RNAEFARwGARIiiAgBpVeBMT4xik+FKAEEjEoBAQEBAQEEAgEXBgqEDYIKiG+FcY1ZdIlIgVWMRIkqhWVFjS0wgQ5igXgNG4FpTgGHcIE9AQEB X-IPAS-Result: A0A+BQD9wR9X/yUp0ApeHYJwK4FQt3CEEYYPAoFATAEBAQEBAWYnhEIBAQEDEhUZAQE3AQ9RNAEFARwGARIiiAgBpVeBMT4xik+FKAEEjEoBAQEBAQEEAgEXBgqEDYIKiG+FcY1ZdIlIgVWMRIkqhWVFjS0wgQ5igXgNG4FpTgGHcIE9AQEB X-IronPort-AV: E=Sophos;i="5.24,537,1454994000"; d="scan'208";a="5410374" Received: from emsm-gh1-uea11.corp.nsa.gov (HELO emsm-gh1-uea11.nsa.gov) ([10.208.41.37]) by goalie.tycho.ncsc.mil with ESMTP; 26 Apr 2016 15:36:53 -0400 IronPort-PHdr: =?us-ascii?q?9a23=3AebawPBZaephkhAQkvICazMn/LSx+4OfEezUN459i?= =?us-ascii?q?sYplN5qZocSzbnLW6fgltlLVR4KTs6sC0LqG9f+xEjVbud6oizMrTt9lb1c9k8?= =?us-ascii?q?IYnggtUoauKHbQC7rUVRE8B9lIT1R//nu2YgB/Ecf6YEDO8DXptWZBUiv2OQc9?= =?us-ascii?q?HOnpAIma153xjLDivcKCKFwT3HKUWvBbElaflU3prM4YgI9veO4a6yDihT92Qd?= =?us-ascii?q?lQ3n5iPlmJnhzxtY+a9Z9n9DlM6bp6r5YTGY2zRakzTKRZATI6KCh1oZSz7ViQ?= =?us-ascii?q?BTeIszExSGQd2iUOSyLE4R33RJL4tGGy4ud32SSWMNfzZaAxWC+57qBtDhTvjX?= =?us-ascii?q?FDfx009n2frdJ9lrlHuh+su1QrwIPSfMeSKvdkZb/HcN4GbWVEV8dVESdGB9X4?= =?us-ascii?q?J6ILBvENNOdC56rwvVQJq1PqDAavLPnixj9Bmjn92qhsl6wEEALLlDcmHs4cvW?= =?us-ascii?q?6c+M7zNbsAXPqd1qTEzTzfKfhR3GG5oITNNxwsv/yIdah9fcrY1Q8kEAaBxm6Q?= =?us-ascii?q?oIvlIjrd8uVF53Ob6+tIVu+1jWMj7QZrrW7rjusllIjFzr0SzVzE8jhizc5hKc?= =?us-ascii?q?K9VxRTYNiqCpJcuiiWcYxsTZVmC0tyuS88gpsBo4S6Z2Bew5Mi3wPDZuacfY+S?= =?us-ascii?q?+h/qX+u5LjJxhXYjc7W60VL62Eyt2uT6WsSumGhLtC0NxsfNrHclxRHV69bBTv?= =?us-ascii?q?p741fn3iyAkR3QvKUMA0ky3YnSMZdplrw5nZk7qlnYE2n9n0CgyOe0e0sqssyh?= =?us-ascii?q?+vjgZP2ypJuVJpNzjCnmNaMlms2yG+k0KU4IUnTNvaz29vur11fkS7gCoPw7lq?= =?us-ascii?q?XC+tiOL8oaq6q1GCdb04sn5w2yFHGhyttOzlcdK1cQQx+bj8DHPEvVIP31RaOk?= =?us-ascii?q?jkis1jls3evGM7DJCZzLKX7Cm77lO7167hgPm0II0dlD6scMWfk6K/XpVxq07Y?= =?us-ascii?q?SAAw=3D=3D?= X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: =?us-ascii?q?A0HzAAD9wR9Xj6vVVdFeHYJwgXu3cIQJC?= =?us-ascii?q?BeBdoQCAoFATAEBAQEBAQICDwEBAQEHCwsJIS+CLX1bPQEBAQMSFRkBATcBD1E?= =?us-ascii?q?0AQUBHAYBEiKICAGlV4ExPjGKT4UoAQSMSgEBAQEBAQQCARcGCoQNggqIb4JjC?= =?us-ascii?q?0CCQ41ZdIlIgVWMRIkqhWVFjS0wgQ6CWg0RCoFpTgGHcIE9AQEB?= X-IPAS-Result: =?us-ascii?q?A0HzAAD9wR9Xj6vVVdFeHYJwgXu3cIQJCBeBdoQCAoFATAE?= =?us-ascii?q?BAQEBAQICDwEBAQEHCwsJIS+CLX1bPQEBAQMSFRkBATcBD1E0AQUBHAYBEiKIC?= =?us-ascii?q?AGlV4ExPjGKT4UoAQSMSgEBAQEBAQQCARcGCoQNggqIb4JjC0CCQ41ZdIlIgVW?= =?us-ascii?q?MRIkqhWVFjS0wgQ6CWg0RCoFpTgGHcIE9AQEB?= X-IronPort-AV: E=Sophos;i="5.24,537,1454976000"; d="scan'208";a="15642546" Received: from mail-ig0-f171.google.com ([209.85.213.171]) by emsm-gh1-uea11.nsa.gov with ESMTP/TLS/AES128-GCM-SHA256; 26 Apr 2016 19:36:52 +0000 Received: by mail-ig0-f171.google.com with SMTP id u5so9534415igk.1 for ; Tue, 26 Apr 2016 12:36:52 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=canonical-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=mXBhIoAqhrV6mpB/EUHfq7FSjviAQtSxLFyR7VtwsDY=; b=aBraqV+MxFa7AngXmhSzym6TQiilDEqIgOeq/nYQMyATiYI3gYrXsEWae/lFe4X2AA V1CxX4yynW0W7ER1BlQzuq0hzwcHTJrl/MXXQz3u/b9NUF8wssEMKouRfyoBtYSiYkDp B3nQ34jQbzvpHDp3G8c0q3uHiuuQzMw4LH82xYPpspFL0Gh4tRuuBFhgeOBgjcegLuob jNkc5yAuASgKtxmLKFRQ77pPII0NqW9BYkTJ1UMN3I3LLqTU0061eYKb9uIqVHUWOWq1 od9jdsMY4NtyeDHT6jpQI5ymVis7WxO7pyTADSinxT2Qh+CHq9JmDeLgGqaEFS0Wp8wE OmPQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=mXBhIoAqhrV6mpB/EUHfq7FSjviAQtSxLFyR7VtwsDY=; b=l0qoCIABrHY2GXWrKqkbLY7hiBGiXUbGKMJqhBvc/c5DgzJCAg2/4u9ELQvZPrN/aG j2XsDU33lbbMSQUKzka0gGfrDUjGqwnCam5iG6txrmOdU1pMbAjYHwd06sDGY+pBNIx4 jhkOX2xIYLYX7xs3shiP9/qsEwljvMYKjR1J1xeqK9slwOIiqcpzSwnnc24BqFcXDxBd ZPYS02U0kqkV7YKEkWcV22KLw7cYMlHbdLj7qbzYmP3MG2tOW9mWerqPxtpu88GwyvsV p3uxwF5RoDebTd9CTfVEerTyAgz1MQA7VTuUObrIaG9o158uT/7HESL/lvJSv+7RuQ0v ocNg== X-Gm-Message-State: AOPr4FXNabLd2nBJ5km/iSv89zRjYYfxyKVSYWvOt+pNs5OOxHLvEC9K0APMyNvT29W+oJoM X-Received: by 10.50.168.100 with SMTP id zv4mr21728113igb.4.1461699411715; Tue, 26 Apr 2016 12:36:51 -0700 (PDT) Received: from localhost ([2605:a601:aab:f920:39a1:5bcf:aa:5b00]) by smtp.gmail.com with ESMTPSA id q9sm2368333igr.7.2016.04.26.12.36.51 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 26 Apr 2016 12:36:51 -0700 (PDT) From: Seth Forshee To: "Eric W. Biederman" , Kent Overstreet , Alasdair Kergon , Mike Snitzer , dm-devel@redhat.com, Shaohua Li , David Woodhouse , Brian Norris , Alexander Viro , Jan Kara , Jeff Layton , "J. Bruce Fields" Subject: [PATCH v4 04/21] block_dev: Support checking inode permissions in lookup_bdev() Date: Tue, 26 Apr 2016 14:36:17 -0500 Message-Id: <1461699396-33000-5-git-send-email-seth.forshee@canonical.com> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1461699396-33000-1-git-send-email-seth.forshee@canonical.com> References: <1461699396-33000-1-git-send-email-seth.forshee@canonical.com> X-Mailman-Approved-At: Tue, 26 Apr 2016 16:11:01 -0400 X-BeenThere: selinux@tycho.nsa.gov X-Mailman-Version: 2.1.20 Precedence: list List-Id: "Security-Enhanced Linux \(SELinux\) mailing list" List-Post: List-Help: Cc: linux-bcache@vger.kernel.org, Serge Hallyn , Seth Forshee , Miklos Szeredi , Richard Weinberger , linux-security-module@vger.kernel.org, linux-kernel@vger.kernel.org, linux-raid@vger.kernel.org, fuse-devel@lists.sourceforge.net, Austin S Hemmelgarn , linux-mtd@lists.infradead.org, selinux@tycho.nsa.gov, linux-fsdevel@vger.kernel.org, cgroups@vger.kernel.org, Pavel Tikhomirov MIME-Version: 1.0 Errors-To: selinux-bounces@tycho.nsa.gov Sender: "Selinux" X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED, RP_MATCHES_RCVD, T_DKIM_INVALID, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP When looking up a block device by path no permission check is done to verify that the user has access to the block device inode at the specified path. In some cases it may be necessary to check permissions towards the inode, such as allowing unprivileged users to mount block devices in user namespaces. Add an argument to lookup_bdev() to optionally perform this permission check. A value of 0 skips the permission check and behaves the same as before. A non-zero value specifies the mask of access rights required towards the inode at the specified path. The check is always skipped if the user has CAP_SYS_ADMIN. All callers of lookup_bdev() currently pass a mask of 0, so this patch results in no functional change. Subsequent patches will add permission checks where appropriate. Signed-off-by: Seth Forshee Acked-by: Serge Hallyn --- drivers/md/bcache/super.c | 2 +- drivers/md/dm-table.c | 2 +- drivers/mtd/mtdsuper.c | 2 +- fs/block_dev.c | 13 ++++++++++--- fs/quota/quota.c | 2 +- include/linux/fs.h | 2 +- 6 files changed, 15 insertions(+), 8 deletions(-) diff --git a/drivers/md/bcache/super.c b/drivers/md/bcache/super.c index a296425a7270..e169739a0253 100644 --- a/drivers/md/bcache/super.c +++ b/drivers/md/bcache/super.c @@ -1950,7 +1950,7 @@ static ssize_t register_bcache(struct kobject *k, struct kobj_attribute *attr, sb); if (IS_ERR(bdev)) { if (bdev == ERR_PTR(-EBUSY)) { - bdev = lookup_bdev(strim(path)); + bdev = lookup_bdev(strim(path), 0); mutex_lock(&bch_register_lock); if (!IS_ERR(bdev) && bch_is_open(bdev)) err = "device already registered"; diff --git a/drivers/md/dm-table.c b/drivers/md/dm-table.c index f9e8f0bef332..13f568d527b5 100644 --- a/drivers/md/dm-table.c +++ b/drivers/md/dm-table.c @@ -372,7 +372,7 @@ dev_t dm_get_dev_t(const char *path) dev_t uninitialized_var(dev); struct block_device *bdev; - bdev = lookup_bdev(path); + bdev = lookup_bdev(path, 0); if (IS_ERR(bdev)) dev = name_to_dev_t(path); else { diff --git a/drivers/mtd/mtdsuper.c b/drivers/mtd/mtdsuper.c index 20c02a3b7417..b5b60e1af31c 100644 --- a/drivers/mtd/mtdsuper.c +++ b/drivers/mtd/mtdsuper.c @@ -176,7 +176,7 @@ struct dentry *mount_mtd(struct file_system_type *fs_type, int flags, /* try the old way - the hack where we allowed users to mount * /dev/mtdblock$(n) but didn't actually _use_ the blockdev */ - bdev = lookup_bdev(dev_name); + bdev = lookup_bdev(dev_name, 0); if (IS_ERR(bdev)) { ret = PTR_ERR(bdev); pr_debug("MTDSB: lookup_bdev() returned %d\n", ret); diff --git a/fs/block_dev.c b/fs/block_dev.c index 3e84d62d0a25..e9b937845bdb 100644 --- a/fs/block_dev.c +++ b/fs/block_dev.c @@ -1431,7 +1431,7 @@ struct block_device *blkdev_get_by_path(const char *path, fmode_t mode, struct block_device *bdev; int err; - bdev = lookup_bdev(path); + bdev = lookup_bdev(path, 0); if (IS_ERR(bdev)) return bdev; @@ -1821,12 +1821,14 @@ EXPORT_SYMBOL(ioctl_by_bdev); /** * lookup_bdev - lookup a struct block_device by name * @pathname: special file representing the block device + * @mask: rights to check for (%MAY_READ, %MAY_WRITE, %MAY_EXEC) * * Get a reference to the blockdevice at @pathname in the current * namespace if possible and return it. Return ERR_PTR(error) - * otherwise. + * otherwise. If @mask is non-zero, check for access rights to the + * inode at @pathname. */ -struct block_device *lookup_bdev(const char *pathname) +struct block_device *lookup_bdev(const char *pathname, int mask) { struct block_device *bdev; struct inode *inode; @@ -1841,6 +1843,11 @@ struct block_device *lookup_bdev(const char *pathname) return ERR_PTR(error); inode = d_backing_inode(path.dentry); + if (mask != 0 && !capable(CAP_SYS_ADMIN)) { + error = __inode_permission(inode, mask); + if (error) + goto fail; + } error = -ENOTBLK; if (!S_ISBLK(inode->i_mode)) goto fail; diff --git a/fs/quota/quota.c b/fs/quota/quota.c index 0f10ee9892ce..59223384b22e 100644 --- a/fs/quota/quota.c +++ b/fs/quota/quota.c @@ -799,7 +799,7 @@ static struct super_block *quotactl_block(const char __user *special, int cmd) if (IS_ERR(tmp)) return ERR_CAST(tmp); - bdev = lookup_bdev(tmp->name); + bdev = lookup_bdev(tmp->name, 0); putname(tmp); if (IS_ERR(bdev)) return ERR_CAST(bdev); diff --git a/include/linux/fs.h b/include/linux/fs.h index 66a639ec1bc4..173b8adc6131 100644 --- a/include/linux/fs.h +++ b/include/linux/fs.h @@ -2438,7 +2438,7 @@ static inline void unregister_chrdev(unsigned int major, const char *name) #define BLKDEV_MAJOR_HASH_SIZE 255 extern const char *__bdevname(dev_t, char *buffer); extern const char *bdevname(struct block_device *bdev, char *buffer); -extern struct block_device *lookup_bdev(const char *); +extern struct block_device *lookup_bdev(const char *, int mask); extern void blkdev_show(struct seq_file *,off_t); #else