From: "Darrick J. Wong" Subject: Re: [PATCH] ioctl_getfsmap.2: document the GETFSMAP ioctl Date: Wed, 10 May 2017 13:14:37 -0700 Message-ID: <20170510201437.GA9854@birch.djwong.org> References: <20170507155855.GD5970@birch.djwong.org> <20170508184112.GJ5973@birch.djwong.org> <20170508204738.GL5973@birch.djwong.org> <20170509015324.GM5973@birch.djwong.org> <20170509211746.GA87747@gmail.com> <20170510163818.7bleiykxgnx3pkds@thunk.org> <87mvakpl5m.fsf@xmission.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: "Theodore Ts'o" , Eric Biggers , Jann Horn , Michael Kerrisk-manpages , linux-xfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-fsdevel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-ext4-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Linux API , linux-man-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-btrfs To: "Eric W. Biederman" Return-path: Content-Disposition: inline In-Reply-To: <87mvakpl5m.fsf-aS9lmoZGLiVWk0Htik3J/w@public.gmane.org> Sender: linux-api-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-ext4.vger.kernel.org [cc btrfs, since afaict that's where most of the dedupe tool authors hang out] On Wed, May 10, 2017 at 02:27:33PM -0500, Eric W. Biederman wrote: > Theodore Ts'o writes: > > > On Tue, May 09, 2017 at 02:17:46PM -0700, Eric Biggers wrote: > >> 1.) Privacy implications. Say the filesystem is being shared between multiple > >> users, and one user unpacks foo.tar.gz into their home directory, which > >> they've set to mode 700 to hide from other users. Because of this new > >> ioctl, all users will be able to see every (inode number, size in blocks) > >> pair that was added to the filesystem, as well as the exact layout of the > >> physical block allocations which might hint at how the files were created. > >> If there is a known "fingerprint" for the unpacked foo.tar.gz in this > >> regard, its presence on the filesystem will be revealed to all users. And > >> if any filesystems happen to prefer allocating blocks near the containing > >> directory, the directory the files are in would likely be revealed too. Frankly, why are container users even allowed to make unrestricted ioctl calls? I thought we had a bunch of security infrastructure to constrain what userspace can do to a system, so why don't ioctls fall under these same protections? If your containers are really that adversarial, you ought to be blacklisting as much as you can. > > Unix/Linux has historically not been terribly concerned about trying > > to protect this kind of privacy between users. So for example, in > > order to do this, you would have to call GETFSMAP continously to track > > this sort of thing. Someone who wanted to do this could probably get > > this information (and much, much more) by continuously running "ps" to > > see what processes are running. > > > > (I will note. wryly, that in the bad old days, when dozens of users > > were sharing a one MIPS Vax/780, it was considered a *good* thing > > that social pressure could be applied when it was found that someone > > was running a CPU or memory hogger on a time sharing system. The > > privacy right of someone running "xtrek" to be able to hide this from > > other users on the system was never considered important at all. :-) Not to mention someone running GETFSMAP in a loop will be pretty obvious both from the high kernel cpu usage and the huge number of metadata operations. > > Fortunately, the days of timesharing seem to well behind us. For > > those people who think that containers are as secure as VM's (hah, > > hah, hah), it might be that best way to handle this is to have a mount > > option that requires root access to this functionality. For those > > people who really care about this, they can disable access. Or use separate filesystems for each container so that exploitable bugs that shut down the filesystem can't be used to kill the other containers. You could use a torrent of metadata-heavy operations (fallocate a huge file, punch every block, truncate file, repeat) to DoS the other containers. > What would be the reason for not putting this behind > capable(CAP_SYS_ADMIN)? > > What possible legitimate function could this functionality serve to > users who don't own your filesystem? As I've said before, it's to enable dedupe tools to decide, given a set of files with shareable blocks, roughly how many other times each of those shareable blocks are shared so that they can make better decisions about which file keeps its shareable blocks, and which file gets remapped. Dedupe is not a privileged operation, nor are any of the tools. > I have seen several people speak up how this is a concern I don't see > anyone saying here is a legitimate use for a non-system administrator. /I/ said that a few emails ago. --D > This doesn't seem like something where abuses of time-sharing systems > can be observed. > > Eric