From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nick Piggin Subject: Re: [PATCH] LSM: Remove unused time_attrs argument. Date: Wed, 2 Jun 2010 14:41:50 +1000 Message-ID: <20100602044150.GJ9453@laptop> References: <20100601155055.GZ9453@laptop> <20100601172425.GC9453@laptop> <201006020328.o523SOJ7073247@www262.sakura.ne.jp> <20100602035012.GH9453@laptop> <201006020424.o524OhPk085993@www262.sakura.ne.jp> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-security-module@vger.kernel.org, torvalds@linux-foundation.org, linux-fsdevel@vger.kernel.org To: Tetsuo Handa Return-path: Content-Disposition: inline In-Reply-To: <201006020424.o524OhPk085993@www262.sakura.ne.jp> Sender: linux-security-module-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On Wed, Jun 02, 2010 at 01:24:43PM +0900, Tetsuo Handa wrote: > Nick Piggin wrote: > > On Wed, Jun 02, 2010 at 12:28:24PM +0900, Tetsuo Handa wrote: > > > When commit be6d3e56a6b9b3a4ee44a0685e39e595073c6f0d "introduce new LSM hooks > > > where vfsmount is available." was proposed, regarding security_path_truncate(), > > > only "struct file *" argument (which AppArmor wanted to use) was removed. > > > But length and time_attrs arguments are not used by TOMOYO nor AppArmor. > > > > Well if length is not used either, can we remove that too seeing as > > we're changing the API anyway. > Yes. Here is the updated patch. Great, thanks. Ack. > ---------- > [PATCH] LSM: Remove unused arguments from security_path_truncate().