Re: [patch 8/8] vfs: create file_remove_suid() helper

From: Christoph Hellwig
Date: Sat May 31 2008 - 04:29:45 EST


On Thu, May 29, 2008 at 01:32:53PM +0200, Miklos Szeredi wrote:
> From: Miklos Szeredi <mszeredi@xxxxxxx>
>
> All calls to remove_suid() are made with a file pointer. This is not
> accidental: similarly to file_update_time(), this function is called
> when the file is written.
>
> So simplify callers by passing the file pointer instead of the dentry.

It's not simplifying anything at all. You just move the pointer
derefence down one step the callchain without any need.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/