xfs
[Top] [All Lists]

TAKE 971186 - [PATCH 1/4] remove permission check from xfs_change_file_s

To: sgi.bugs.xfs@xxxxxxxxxxxx, xfs@xxxxxxxxxxx
Subject: TAKE 971186 - [PATCH 1/4] remove permission check from xfs_change_file_space
From: lachlan@xxxxxxx (Lachlan McIlroy)
Date: Fri, 18 Jan 2008 16:08:00 +1100 (EST)
Sender: xfs-bounce@xxxxxxxxxxx
[PATCH 1/4] remove permission check from xfs_change_file_space

Both callers of xfs_change_file_space alreaedy do the
file->f_mode & FMODE_WRITE check to ensure we have a file descriptor
that has been opened for write mode, so there is no need to re-check
that with xfs_iaccess.  Especially as the later might wrongly deny it
for corner cases like file descriptor passing through unix domain
sockets.


Signed-off-by: Christoph Hellwig <hch@xxxxxx>

Date:  Fri Jan 18 16:07:07 AEDT 2008
Workarea:  redback.melbourne.sgi.com:/home/lachlan/isms/2.6.x-hch
Inspected by:  Christoph Hellwig <hch@xxxxxx>
Author:  lachlan

The following file(s) were checked into:
  longdrop.melbourne.sgi.com:/isms/linux/2.6.x-xfs-melb


Modid:  xfs-linux-melb:xfs-kern:30365a
fs/xfs/xfs_vnodeops.c - 1.728 - changed
http://oss.sgi.com/cgi-bin/cvsweb.cgi/xfs-linux/xfs_vnodeops.c.diff?r1=text&tr1=1.728&r2=text&tr2=1.727&f=h
        - remove permission check from xfs_change_file_space




<Prev in Thread] Current Thread [Next in Thread>
  • TAKE 971186 - [PATCH 1/4] remove permission check from xfs_change_file_space, Lachlan McIlroy <=