Question

我正在尝试为包含整数“timer_interval”（全局变量）的sysfs文件创建存储函数。

static ssize_t sys_store(struct kobject *kobj, struct kobj_attribute *attr, const char *buf,     size_t count)
{
  printk(KERN_ALERT "BUF IS %s and count is %d", buf, count);
  int ret;

  char *pTemp = kmalloc(100, GFP_KERNEL); 
  if (pTemp == NULL) 
  {
     printk(KERN_ALERT "malloc WRONG");
     return -EFAULT;
  }

  printk(KERN_ALERT "AFTER MALLOC");

  if (ret = copy_from_user(pTemp ,buf, count)) 
  {
    printk(KERN_ALERT "COPY WRONG %d", ret);
    kfree(pTemp);
    printk(KERN_ALERT "%d" , pTemp);
    return -EFAULT;
  }

  printk(KERN_ALERT "AFTER COPY");

  if (sscanf(pTemp, "%d", &Timer_interval) < count);
  {
    printk(KERN_ALERT "SCANF WRONG");
    kfree(pTemp);
    return -EFAULT;
  }

  printk(KERN_ALERT "AFTER SCANF COUNT = %d", Timer_interval);
  kfree(pTemp);
  return count;
}

问题在于复制到用户。 dmesg显示分配正常，buf包含正确的数据，但copy_from_user返回缓冲区的大小，这意味着没有复制任何内容。

我在这里做错了什么？

Answer 1

最有可能buf是内核空间指针，您不需要使用copy_from_user。 copy_from_user由内核调用。

<强>更新

buf是来自用户的直接缓冲区。你应该检查buf的数据是否正确。例如，可以使用文本调用函数，同时期望数字。

顺便说一句。在sscanf调用之前，您不检查pTemp数组包含null终止符。如果你复制缓冲区，你需要检查src和dst缓冲区的大小，以防止内存损坏或段错误。

copy_from_user返回错误

1 个答案: