如何禁用 proc 文件的读取或写入功能？

Question

我正在创建一个 proc 文件 (/proc/key)，用户可以将其 decryption_key 写入该文件，然后该密钥将用于解密存储在内核模块中的缓冲区的内容。此外，我还有另一个 proc 条目（/proc/decrypted），用于读取存储解密文本的缓冲区的内容。

问题是我不希望用户能够向 (/proc/decrypted) 文件写入任何内容，也不希望他从 (/proc/key) 读取任何内容。如何实现？

我已经将file_operations结构中的相应函数指向NULL，但是很明显，一旦用户尝试它们，这将导致分段错误。

如何防止从 procfs 读取或写入？我应该只创建没有主体的函数并在需要时将 file_operations 结构指向它们吗？

static ssize_t key_proc_write(struct file *filp, const char __user *buf, size_t count, loff_t *ppos)
{
    char temp[128];
    memset(temp, 0, 128);
    int c; 
    c = copy_from_user(temp, buf, count);
 return count;
}


static const struct file_operations Proc_key_fops = {
 .owner = THIS_MODULE,
 .open = hello_proc_open,
 .read = NULL,
 .write = key_proc_write,
 .llseek = seq_lseek,
 .release = single_release,
};  

Answer 1

如果你想禁止读取，你可以省略显式设置struct file_operation 的.read 字段。如果结构体定义为static，因此初始化为0，所有没有显式覆盖的字段都将默认为NULL，内核将根本不做任何事情并返回错误（我相信-EINVAL）每当用户代码尝试在您打开的文件上调用 read 时。

或者，如果您想返回自定义错误，您可以定义一个只返回错误的虚拟函数（例如return -EFAULT;）。

你认为我将密钥“写入”缓冲区的方式是正确的吗？

这是错误的，原因有很多。

首先，您的copy_from_user() 盲目信任用户count，因此这会导致temp 变量上的内核缓冲区溢出，这非常糟糕。您需要先检查和/或限制大小。您也没有检查copy_from_user() 的返回值，这是您应该检查的（而且它不是int，而是unsigned long）。

static ssize_t key_proc_write(struct file *filp, const char __user *buf, size_t count, loff_t *ppos)
{
    char temp[128];
    memset(temp, 0, 128);

    if (count > 128)
        count = 128; // or alternatively return -EINVAL or another error

    if (copy_from_user(temp, buf, count))
        return -EFAULT;

    return count;
}

现在代码更有意义了，但是变量temp只在函数内部定义，因此在函数返回后会丢失，您将无法在其他文件操作函数中使用它。如果你想这样做，你可以使用filp->private_data，这是struct file的一个字段，正是为此目的。

您应该在open 上创建并初始化一个缓冲区，然后在您的release 函数中释放它，如下所示：

static int hello_proc_open(struct inode *ino, struct file *filp) 
{
    void *buf = kmalloc(128, GFP_KERNEL);
    if (!buf)
        return -ENOMEM;

    filp->private_data = buf;

    // ... whatever else you need to do

    return 0;
}

static int hello_proc_release(struct inode *ino, struct file *filp) 
{
    kfree(filp->private_data);

    // ... whatever else you need to do

    return 0;
}

然后，在您的write 中，您可以在将缓冲区转换为正确的类型后直接使用它：

static ssize_t key_proc_write(struct file *filp, const char __user *buf, size_t count, loff_t *ppos)
{
    char *temp = filp->private_data;
    memset(temp, 0, 128);

    if (count > 128)
        count = 128; // or alternatively return -EINVAL or another error

    if (copy_from_user(temp, buf, count))
        return -EFAULT;

    return count;
}

最后，我看到你在使用：

 .llseek = seq_lseek,
 .release = single_release,

不要这样做。您不需要使用预定义的操作。不要混合您创建的自定义函数和其他规范函数，例如用于序列文件的函数。序列文件需要执行一些初始化和拆卸，或者您使用 all seq_ 系列文件操作，或者您自己手动执行，或者您不使用这些函数。在您的情况下，最后一个选项适用。

您可以将.release 设置为我上面显示的hello_proc_release()，而不设置.llseek（默认为NULL）。