Skip to content

Commit d102d89

Browse files
committed
apparmor: pass gfp param into aa_policy_init()
Signed-off-by: John Johansen <[email protected]>
1 parent bbe4a7c commit d102d89

File tree

4 files changed

+7
-7
lines changed

4 files changed

+7
-7
lines changed

security/apparmor/include/lib.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -194,7 +194,7 @@ static inline struct aa_policy *__policy_strn_find(struct list_head *head,
194194
}
195195

196196
bool aa_policy_init(struct aa_policy *policy, const char *prefix,
197-
const char *name);
197+
const char *name, gfp_t gfp);
198198
void aa_policy_destroy(struct aa_policy *policy);
199199

200200
#endif /* AA_LIB_H */

security/apparmor/lib.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -171,20 +171,20 @@ void *__aa_kvmalloc(size_t size, gfp_t flags)
171171
* Returns: true if policy init successful
172172
*/
173173
bool aa_policy_init(struct aa_policy *policy, const char *prefix,
174-
const char *name)
174+
const char *name, gfp_t gfp)
175175
{
176176
/* freed by policy_free */
177177
if (prefix) {
178178
policy->hname = kmalloc(strlen(prefix) + strlen(name) + 3,
179-
GFP_KERNEL);
179+
gfp);
180180
if (policy->hname)
181181
sprintf((char *)policy->hname, "%s//%s", prefix, name);
182182
} else
183-
policy->hname = kstrdup(name, GFP_KERNEL);
183+
policy->hname = kstrdup(name, gfp);
184184
if (!policy->hname)
185185
return 0;
186186
/* base.name is a substring of fqname */
187-
policy->name = (char *)basename(policy->hname);
187+
policy->name = basename(policy->hname);
188188
INIT_LIST_HEAD(&policy->list);
189189
INIT_LIST_HEAD(&policy->profiles);
190190

security/apparmor/policy.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -272,7 +272,7 @@ struct aa_profile *aa_alloc_profile(const char *hname)
272272
goto fail;
273273
kref_init(&profile->proxy->count);
274274

275-
if (!aa_policy_init(&profile->base, NULL, hname))
275+
if (!aa_policy_init(&profile->base, NULL, hname, GFP_KERNEL))
276276
goto fail;
277277
kref_init(&profile->count);
278278

security/apparmor/policy_ns.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ static struct aa_ns *alloc_ns(const char *prefix, const char *name)
9595
AA_DEBUG("%s(%p)\n", __func__, ns);
9696
if (!ns)
9797
return NULL;
98-
if (!aa_policy_init(&ns->base, prefix, name))
98+
if (!aa_policy_init(&ns->base, prefix, name, GFP_KERNEL))
9999
goto fail_ns;
100100

101101
INIT_LIST_HEAD(&ns->sub_ns);

0 commit comments

Comments
 (0)