Как мне напечатать строку из буфера ядра?Я использую copy_from_user () правильно? - PullRequest
Новая
       33

Как мне напечатать строку из буфера ядра?Я использую copy_from_user () правильно?

0 голосов
/ 22 марта 2012

Привет, я пишу метод write () для моего драйвера персонажа, и мне было интересно, почему, когда я копирую данные из пользователя в буфер ядра, мой буфер содержит случайный мусор. Ниже приводится метод, который я использую. 

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 \0 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 "мое предложение"> / dev / my_dev для записи на мое устройство. Мне все равно, если эхо работает. Я просто хочу, чтобы printk () отображал «мое предложение», что означает, что copy_from работает. Также я попытался использовать echo «мое предложение \ 0»> / dev / my_dev, чтобы придерживаться правила вставки нулевого символа в конце строки.

Спасибо за любую помощь.

1 Ответ

3 голосов
/ 22 марта 2012

Первый аргумент 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 к адресу и обратите внимание, что я предполагаю, что ваш буфер имеет тип данных, который1009 * оценивает до 1.

...