Почему копирование структуры из пользовательского пространства не выполняется в команде ioctl? - PullRequest
Новая
       53

Почему копирование структуры из пользовательского пространства не выполняется в команде ioctl?

0 голосов
/ 08 января 2011

Я разрабатываю драйвер устройства и мне нужно использовать IOCTL. К сожалению, я не могу скопировать структуру из пространства пользователя. Вот код (упрощенный, обработка ошибок удалена):

Структура 

struct secvault_createoptions {
    int secvaultId;
    long dataSize;
    char key[SECVAULT_KEYSIZE];
};

Применение 

void createSecvault(int secvaultId)
{
    struct secvault_createoptions creationOptions;
    /* fill with data */
    sendIoctlCommand(SECVAULT_IOCTL_CREATE, &creationOptions);
}

void sendIoctlCommand(int command, void *arg)
{
    FILE *stream;
    int fd, err;

    stream = fopen(SECVAULT_DEV_CONTROL, "r");
    fd = fileno(stream);
    ioctl(fd, command, arg);
    fclose(stream);
}

Модуль ядра 

int control_device_ioctl(struct inode *node, struct file *filp, unsigned int cmd, unsigned long arg)
{
    struct secvault_createoptions creationOptions;
    int returnCode;

    switch (cmd)
    {
        case SECVAULT_IOCTL_CREATE:
            if (copy_from_user(&creationOptions, (void*)arg, sizeof(struct secvault_createoptions)) != sizeof(struct secvault_createoptions))
            {
                /* Always this branch gets executed */
                printk(KERN_ALERT "Copying secure vault creation options from user space failed.\n");
                returnCode = -EFAULT;
                break;
            }
            printk(KERN_ALERT "2 IOCTL create request on control device received: secvaultId = %d, dataSize = %ld.\n",
                creationOptions.secvaultId, creationOptions.dataSize);

            returnCode = createDataDevice(&creationOptions);
            break;
    }
    return returnCode;
}

С наилучшими пожеланиями,
Оливер Ханаппи

Ответы [ 3 ]

4 голосов
/ 08 января 2011

Ваш copy_from_user звонок неправильный. Он не возвращает количество скопированных байтов, но количество байтов, которые не были скопированы. То, что вы хотите, это 

if (copy_from_user(...) != 0)
        return -EFAULT;

(Вы можете пропустить назначение на ret в своем фрагменте.)

3 голосов
/ 08 января 2011

copy_from_user() возвращает количество байтов, которые не могут быть скопированы.Так что вам следует ожидать 0 успеха, а не sizeof(struct secvault_createoptions).

0 голосов
/ 02 сентября 2014

Вы должны изменить оператор как, 

  if (copy_from_user(&creationOptions, (void*)arg, sizeof(struct secvault_createoptions)) != 0)
            {
                /* Always this branch gets executed */
                printk(KERN_ALERT "Copying secure vault creation options from user space failed.\n");
                returnCode = -EFAULT;
                break;
            }

Поскольку copy_from_user всегда возвращает 0 после успешного завершения.

Пожалуйста, обратитесь к этому для более подробной информации.

...