Commit 9c4a45b1 authored by Christian Lamparter's avatar Christian Lamparter Committed by Greg Kroah-Hartman

b43legacy: fix debugfs crash

This patch fixes a crash that happens because b43legacy's
debugfs code expects file->f_op to be a pointer to its own
b43legacy_debugfs_fops struct. This is no longer the case
since commit 9fd4dcec
("debugfs: prevent access to possibly dead file_operations at file open")
Reviewed-by: default avatarNicolai Stange <nicstange@gmail.com>
Signed-off-by: default avatarChristian Lamparter <chunkeey@gmail.com>
Cc: stable <stable@vger.kernel.org> # 4.7+
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 51b275a6
...@@ -221,7 +221,8 @@ static ssize_t b43legacy_debugfs_read(struct file *file, char __user *userbuf, ...@@ -221,7 +221,8 @@ static ssize_t b43legacy_debugfs_read(struct file *file, char __user *userbuf,
goto out_unlock; goto out_unlock;
} }
dfops = container_of(file->f_op, struct b43legacy_debugfs_fops, fops); dfops = container_of(debugfs_real_fops(file),
struct b43legacy_debugfs_fops, fops);
if (!dfops->read) { if (!dfops->read) {
err = -ENOSYS; err = -ENOSYS;
goto out_unlock; goto out_unlock;
...@@ -287,7 +288,8 @@ static ssize_t b43legacy_debugfs_write(struct file *file, ...@@ -287,7 +288,8 @@ static ssize_t b43legacy_debugfs_write(struct file *file,
goto out_unlock; goto out_unlock;
} }
dfops = container_of(file->f_op, struct b43legacy_debugfs_fops, fops); dfops = container_of(debugfs_real_fops(file),
struct b43legacy_debugfs_fops, fops);
if (!dfops->write) { if (!dfops->write) {
err = -ENOSYS; err = -ENOSYS;
goto out_unlock; goto out_unlock;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment