Почему копирование структуры из помещения пользователей не удается в команде IOCTL?
-
08-10-2019 - |
Вопрос
Я разрабатываю драйвер устройства и должен использовать 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;
}
С уважением,
Оливер Ханаппи
Решение
Твой copy_from_user
Звонок не прав. Не возвращает количество копированных байтов, но количество байтов, которые не были скопированы. Что вы хотите, это
if (copy_from_user(...) != 0)
return -EFAULT;
(Вы можете пропустить назначение ret
в вашем фрагменте.)
Другие советы
copy_from_user()
возвращает количество байтов, которые могут нет быть скопированным. Так что вы должны ожидать 0
для успеха, а не sizeof(struct secvault_createoptions)
.
Вы должны изменить утверждение, как,
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 после успешного завершения.
Пожалуйста, обратитесь к этому Больше подробностей.