编写我自己的系统调用来计算字符串指针中出现了多少个 o答案

【问题标题】：Writing my own system call to count how many o's appear in string pointer编写我自己的系统调用来计算字符串指针中出现了多少个 o
【发布时间】：2019-08-17 00:43:41
【问题描述】：

所以我的任务是在 linux 中编写我自己的系统调用。此系统调用将采用指向字符数组的指针并将所有 o 替换为 0。系统调用将返回执行的替换次数。如果字符串大小大于 128 字节，则返回 -1。所以我已经实现了一个不同的系统调用，按照这个链接中的步骤完全可以正常工作。 https://medium.com/anubhav-shrimal/adding-a-hello-world-system-call-to-linux-kernel-dad32875872 。我已经仔细检查了一切，一切似乎都很好，所以我相信我编写的系统调用或测试它时有问题。假设是这种情况，我的系统调用有什么问题吗？我认为这可能与字符指针或类似的东西有关。

这是我的系统调用。

#include <linux/kernel.h>

asmlinkage int sys_my_syscall2(char * string){
    if(sizeof(string) > 128){
        return -1;
    }
    int x, count = 0;
    for(x = 0; x < sizeof(string); x++){
        if(string[x] == 'o'){
            string[x] = '0';
            count++;
        }
    }
    return count;
}

这是我的测试文件。

#include <stdio.h>
#include <linux/kernel.h>
#include <sys/syscall.h>
#include <unistd.h>
int main() {
    int syscall2 = syscall(333, "Hello World");
    printf("Syscall 333 printed %d", syscall2); 
    return 0;
}

333 是我的系统调用的编号。运行测试文件后，它会暂停很长时间，并且似乎冻结了。即使在尝试控制-C 之后，程序似乎仍然可以运行。这些文件有问题吗？

【问题讨论】：

sizeof(string) 并没有按照你的想法去做。
您正在将 const 字符串传递给将修改您的内存的系统调用。您是否尝试过使用可写内存缓冲区？
您有三个问题：第一个是指针的sizeof 给出了指针的大小，而不是它指向的内容。第二个问题是您要迭代的数据（字符串）在用户空间中，需要复制到内核空间（尽管我不确定）。第三个问题是文字字符串是只读字符数组，不能修改。

标签： c linux system-calls

【解决方案1】：

正确的做法是：

将指向字符串的指针传递给系统调用
执行strlen_user(string) 以找出字符串的大小
提供一个足够大的缓冲区来保存字符串（例如kmalloc()）
使用copy_from_user()将字符串从用户空间复制到内核缓冲区（检查调用是否成功！）
对内核缓冲区执行替换操作
使用copy_to_user()将字符串复制回用户空间
如果您使用kmalloc() 分配了一个缓冲区，请释放缓冲区。

您不应该直接访问用户空间，因为如果您的指针指向的内存不可用（即换出）或受保护以供读/写，内核可能会发生错误。例如，您的程序就是这种情况；尝试直接写入此内存（没有 copy_to_user）将导致内核页面错误。 注意：直接访问可以工作，但它不安全！

此外，在对字符串进行操作时，您必须使用strlen(string)（或strlen(string)+1 来说明终止的零字节）。

【讨论】：