diff options
| author | Eric Paris <eparis@redhat.com> | 2012-01-03 18:25:15 +0100 |
|---|---|---|
| committer | Eric Paris <eparis@redhat.com> | 2012-01-06 00:52:53 +0100 |
| commit | b7e724d303b684655e4ca3dabd5a6840ad19012d (patch) | |
| tree | 5474d8d49d61ade4c5e306a0485a835587237bf4 /drivers/pci | |
| parent | capabilities: remove the task from capable LSM hook entirely (diff) | |
| download | linux-b7e724d303b684655e4ca3dabd5a6840ad19012d.tar.xz linux-b7e724d303b684655e4ca3dabd5a6840ad19012d.zip | |
capabilities: reverse arguments to security_capable
security_capable takes ns, cred, cap. But the LSM capable() hook takes
cred, ns, cap. The capability helper functions also take cred, ns, cap.
Rather than flip argument order just to flip it back, leave them alone.
Heck, this should be a little faster since argument will be in the right
place!
Signed-off-by: Eric Paris <eparis@redhat.com>
Diffstat (limited to 'drivers/pci')
| -rw-r--r-- | drivers/pci/pci-sysfs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/pci/pci-sysfs.c b/drivers/pci/pci-sysfs.c index 7bcf12adced7..a4457ab61342 100644 --- a/drivers/pci/pci-sysfs.c +++ b/drivers/pci/pci-sysfs.c @@ -431,7 +431,7 @@ pci_read_config(struct file *filp, struct kobject *kobj, u8 *data = (u8*) buf; /* Several chips lock up trying to read undefined config space */ - if (security_capable(&init_user_ns, filp->f_cred, CAP_SYS_ADMIN) == 0) { + if (security_capable(filp->f_cred, &init_user_ns, CAP_SYS_ADMIN) == 0) { size = dev->cfg_size; } else if (dev->hdr_type == PCI_HEADER_TYPE_CARDBUS) { size = 128; |