c-如何从内核缓冲区打印字符串?我用的是copy_from_user()吗

Question

嗨，我正在为我的角色驱动程序编写一个write（）方法，我想知道为什么当我将用户的数据复制到内核缓冲区时，我的缓冲区包含随机jibberish。贝娄是我正在使用的方法。

ssize_t dev_write(struct file *filp, const char __user *buff, size_t count, loff_t *offp){// This function looks good
    struct my_char_structure *my_dev = filp->private_data;
    char *offset; // This points to my buffer
    size_t np, left_to_print = count; // These are the bytes left to write

//my_dev->set.write is a double pointer to a sentence where
//set.write = array[sentence number][character in sentence]
//set.write if set.write points to non-allocated memory, the sentence number is NULL
//My device only holds 100 sentences in all
    if(my_dev->set.write == NULL){// Write is a double pointer that points
        printk(KERN_ALERT "Device has no more room");
        return -ENOMEM; // Look up
    }

//you can ignore the commented out stuff but I want to check to see if I'm referencing
//a sentence that has been filled out. other wise the sentence is null and may be 
//written to. I will add this to my code later for traversing sentences.
    /*if(*(my_dev->set.write) != NULL){
        my_dev->set.write ++;
        dev_write(filp,buff,count,offp);
        exit 0;
    }*/
    if (down_interruptible(&my_dev->sem))
        return -ERESTARTSYS;
// Here I'm referencing a memory segment that acts as a pointer to a sentence which
//I write to. *(my_dev->set.write) is a pointer to a char buff which I will place chars
// which is essentially my string
    *(my_dev->set.write) = kmalloc(count,GFP_KERNEL);
    offset = *(my_dev->set.write); // my offset points to that buffer as well
//A sentence can only be the size of count, which is passed by the user
//thats why I allocate count bytes for memory.

        if((np = copy_from_user(offset,buff,left_to_print)) < 0)
            goto erro_out;
        left_to_print -= (count-np);
        offset += (count-np);
        offp += (count-np);
//For debbuging purposes I have printk(). My offset points to my buffer, however
//prints jibberish. In user space I insert  at the end of the string.
            printk(KERN_ALERT " 4 :: write && left is ... %ld. The string %s", left_to_print, offset); // %s prints jibberish
    return left_to_print; // change to count
    erro_out:// if there was an error I free the sentence so that it may be written to.
        kfree(*(my_dev->set.write));
        printk(KERN_ALERT "Print error %ld",np);
        return -EFAULT; // look this up
}

如果我的方法有点冗长，我的方法总结如下。这对于第一次写入设备来说应该很好。

ssize_t dev_write(struct file *filp, const char __user *buff, size_t count, loff_t *offp){// This function looks good
    struct my_char_structure *my_dev = filp->private_data;
    static char *offset;
    if (down_interruptible(&my_dev->sem))
        return -ERESTARTSYS;
    *(my_dev->set.write) = kmalloc(count,GFP_KERNEL);
    offset = *(my_dev->set.write);
        if(copy_from_user(offset,buff,left_to_print) < 0)
            goto erro_out;
// This is my question, why does %s print jibberish
            printk(KERN_ALERT " 4 :: write && left is ... %ld. The string %s", left_to_print, offset); 
    return 0; // change to count
    erro_out:
        kfree(*(my_dev->set.write));
        printk(KERN_ALERT "Print error %ld",np);
        return -EFAULT; // look this up     
}

我使用echo"my sentence">/dev/my_dev写入设备。我不在乎echo是否有效。我只想让printk（）显示"我的句子"，这意味着copy_from正在工作。此外，我还尝试使用echo"my senten\0">/dev/my_dev来遵守在字符串末尾粘贴null字符的规则。

谢谢你的帮助。

Answer 1

copy_from_user的第一个参数需要一个目标地址。通过传递这样的偏移：

    if((np = copy_from_user(offset,buff,left_to_print)) < 0)

你给了它错误的写入地址。你应该通过你的缓冲区，比如：

    if((np = copy_from_user(_my_buffer_ + offset,buff,left_to_print)) < 0)

请确保您分配了足够的空间，以便可以将offset添加到地址中，并注意，我假设您的缓冲区是sizeof计算为1的数据类型。