Правильный способ использовать copy_to_user?

Question

Я пытаюсь определить системный вызов, который изменяет буфер символов, передаваемый ему. Конкретно как то так:

...
asmlinkage int sys_mycall( char __user *buff, int len )
{
   char tmp[1000];
   copy_from_user(tmp, buff, len);
   /* change tmp here */
   copy_to_user( buff, &tmp, len );
}

Здесь copy_to_user возвращает -1, а буфер из вызывающей программы не изменяется. Что происходит?

Answer 1

Помните, что tmp уже является указателем! Правильный способ сделать это:

copy_to_user( buff, tmp, len );

Answer 2

Это выглядит хорошо. Возможно, что буфер, который передал пользовательское пространство, отображается только для чтения - например, если он находится в текстовом сегменте (например, строковый литерал). Кстати, это, вероятно, то, что вы хотите:

return copy_to_user(buff, &tmp, len) ? -EFAULT : 0;