← Back to branch summary

~ubuntu-branches/ubuntu/wily/apparmor/wily

« back to all changes in this revision

Viewing changes to kernel-patches/for-mainline/file-handle-ops.diff

  • Committer: Bazaar Package Importer
  • Author(s): Kees Cook
  • Date: 2011-04-27 10:38:07 UTC
  • mfrom: (5.1.118 natty)
  • Revision ID: james.westby@ubuntu.com-20110427103807-ym3rhwys6o84ith0
Tags: 2.6.1-2
debian/copyright: clarify for some full organization names.

Show diffs side-by-side

added added

removed removed

Lines of Context:
kernel-patches/for-mainline/file-handle-ops.diff
1
 
Index: b/include/linux/fs.h
2
 
===================================================================
3
 
--- a/include/linux/fs.h
4
 
+++ b/include/linux/fs.h
5
 
@@ -349,6 +349,9 @@ struct iattr {
6
 
         * Not an attribute, but an auxilary info for filesystems wanting to
7
 
         * implement an ftruncate() like method.  NOTE: filesystem should
8
 
         * check for (ia_valid & ATTR_FILE), and not for (ia_file != NULL).
9
 
+        *
10
 
+        * The LSM hooks also use this to distinguish operations on a file
11
 
+        * descriptors from operations on pathnames.
12
 
         */
13
 
        struct file     *ia_file;
14
 
 };
15
 
Index: b/fs/open.c
16
 
===================================================================
17
 
--- a/fs/open.c
18
 
+++ b/fs/open.c
19
 
@@ -520,6 +520,8 @@ asmlinkage long sys_fchmod(unsigned int 
20
 
                mode = inode->i_mode;
21
 
        newattrs.ia_mode = (mode & S_IALLUGO) | (inode->i_mode & ~S_IALLUGO);
22
 
        newattrs.ia_valid = ATTR_MODE | ATTR_CTIME;
23
 
+       newattrs.ia_valid |= ATTR_FILE;
24
 
+       newattrs.ia_file = file;
25
 
        err = notify_change(dentry, file->f_path.mnt, &newattrs);
26
 
        mutex_unlock(&inode->i_mutex);
27
 
 
28
 
@@ -570,7 +572,7 @@ asmlinkage long sys_chmod(const char __u
29
 
 }
30
 
 
31
 
 static int chown_common(struct dentry * dentry, struct vfsmount *mnt,
32
 
-                       uid_t user, gid_t group)
33
 
+                       uid_t user, gid_t group, struct file *file)
34
 
 {
35
 
        struct inode * inode;
36
 
        int error;
37
 
@@ -598,6 +600,10 @@ static int chown_common(struct dentry * 
38
 
        }
39
 
        if (!S_ISDIR(inode->i_mode))
40
 
                newattrs.ia_valid |= ATTR_KILL_SUID|ATTR_KILL_SGID;
41
 
+       if (file) {
42
 
+               newattrs.ia_file = file;
43
 
+               newattrs.ia_valid |= ATTR_FILE;
44
 
+       }
45
 
        mutex_lock(&inode->i_mutex);
46
 
        error = notify_change(dentry, mnt, &newattrs);
47
 
        mutex_unlock(&inode->i_mutex);
48
 
@@ -613,7 +619,7 @@ asmlinkage long sys_chown(const char __u
49
 
        error = user_path_walk(filename, &nd);
50
 
        if (error)
51
 
                goto out;
52
 
-       error = chown_common(nd.dentry, nd.mnt, user, group);
53
 
+       error = chown_common(nd.dentry, nd.mnt, user, group, NULL);
54
 
        path_release(&nd);
55
 
 out:
56
 
        return error;
57
 
@@ -633,7 +639,7 @@ asmlinkage long sys_fchownat(int dfd, co
58
 
        error = __user_walk_fd(dfd, filename, follow, &nd);
59
 
        if (error)
60
 
                goto out;
61
 
-       error = chown_common(nd.dentry, nd.mnt, user, group);
62
 
+       error = chown_common(nd.dentry, nd.mnt, user, group, NULL);
63
 
        path_release(&nd);
64
 
 out:
65
 
        return error;
66
 
@@ -647,7 +653,7 @@ asmlinkage long sys_lchown(const char __
67
 
        error = user_path_walk_link(filename, &nd);
68
 
        if (error)
69
 
                goto out;
70
 
-       error = chown_common(nd.dentry, nd.mnt, user, group);
71
 
+       error = chown_common(nd.dentry, nd.mnt, user, group, NULL);
72
 
        path_release(&nd);
73
 
 out:
74
 
        return error;
75
 
@@ -666,7 +672,7 @@ asmlinkage long sys_fchown(unsigned int 
76
 
 
77
 
        dentry = file->f_path.dentry;
78
 
        audit_inode(NULL, dentry->d_inode);
79
 
-       error = chown_common(dentry, file->f_path.mnt, user, group);
80
 
+       error = chown_common(dentry, file->f_path.mnt, user, group, file);
81
 
        fput(file);
82
 
 out:
83
 
        return error;
84
 
Index: b/security/apparmor/lsm.c
85
 
===================================================================
86
 
--- a/security/apparmor/lsm.c
87
 
+++ b/security/apparmor/lsm.c
88
 
@@ -82,7 +82,7 @@ static int apparmor_ptrace(struct task_s
89
 
         * under the rules that the kernel normally permits if the two
90
 
         * processes are running under the same profile, but then we
91
 
         * would probably have to reject profile changes for processes
92
 
-        * that are being ptraces as well as for processes ptracing
93
 
+        * that are being ptraced as well as for processes ptracing
94
 
         * others.
95
 
         */
96
 
 
97
 
Index: b/security/apparmor/main.c
98
 
===================================================================
99
 
--- a/security/apparmor/main.c
100
 
+++ b/security/apparmor/main.c
101
 
@@ -190,7 +190,7 @@ static inline void aa_put_name_buffer(ch
102
 
 
103
 
 static int aa_perm_dentry(struct aa_profile *profile, struct dentry *dentry,
104
 
                          struct vfsmount *mnt, struct aa_audit *sa, int mask,
105
 
-                         int is_dir)
106
 
+                         int is_dir, int accessed_through_fd)
107
 
 {
108
 
        char *buffer = NULL;
109
 
        int denied_mask, error;
110
 
@@ -198,8 +198,11 @@ static int aa_perm_dentry(struct aa_prof
111
 
        sa->name = aa_get_name(dentry, mnt, &buffer, is_dir);
112
 
 
113
 
        if (IS_ERR(sa->name)) {
114
 
-               /* deleted files are given a pass on permission checks */
115
 
-               if (PTR_ERR(sa->name) == -ENOENT)
116
 
+               /*
117
 
+                * deleted files are given a pass on permission checks when
118
 
+                * accessed through a file descriptor.
119
 
+                */
120
 
+               if (PTR_ERR(sa->name) == -ENOENT && accessed_through_fd)
121
 
                        denied_mask = 0;
122
 
                else
123
 
                        denied_mask = PTR_ERR(sa->name);
124
 
@@ -538,7 +541,8 @@ int aa_attr(struct aa_profile *profile, 
125
 
        sa.gfp_mask = GFP_KERNEL;
126
 
 
127
 
        error = aa_perm_dentry(profile, dentry, mnt, &sa, MAY_WRITE,
128
 
-                              S_ISDIR(dentry->d_inode->i_mode));
129
 
+                              S_ISDIR(dentry->d_inode->i_mode),
130
 
+                              iattr->ia_valid & ATTR_FILE);
131
 
 
132
 
        return error;
133
 
 }
134
 
@@ -566,7 +570,8 @@ int aa_perm_xattr(struct aa_profile *pro
135
 
        sa.gfp_mask = GFP_KERNEL;
136
 
 
137
 
        error = aa_perm_dentry(profile, dentry, mnt, &sa, mask,
138
 
-                              S_ISDIR(dentry->d_inode->i_mode));
139
 
+                              S_ISDIR(dentry->d_inode->i_mode),
140
 
+                              0 /* FIXME */);
141
 
 
142
 
        return error;
143
 
 }
144
 
@@ -608,7 +613,8 @@ int aa_perm(struct aa_profile *profile, 
145
 
        sa.flags = 0;
146
 
        sa.gfp_mask = GFP_KERNEL;
147
 
        error = aa_perm_dentry(profile, dentry, mnt, &sa, mask,
148
 
-                              inode && S_ISDIR(inode->i_mode));
149
 
+                              inode && S_ISDIR(inode->i_mode),
150
 
+                              0 /* FIXME */);
151
 
 
152
 
 out:
153
 
        return error;
154
 
@@ -636,7 +642,8 @@ int aa_perm_dir(struct aa_profile *profi
155
 
        sa.flags = 0;
156
 
        sa.gfp_mask = GFP_KERNEL;
157
 
 
158
 
-       return aa_perm_dentry(profile, dentry, mnt, &sa, mask, 1);
159
 
+       return aa_perm_dentry(profile, dentry, mnt, &sa, mask, 1,
160
 
+                             0 /* FIXME */);
161
 
 }
162
 
 
163
 
 /**