diff --git a/kernel/kpm/kpm.c b/kernel/kpm/kpm.c index efda485e..658c94db 100644 --- a/kernel/kpm/kpm.c +++ b/kernel/kpm/kpm.c @@ -63,7 +63,9 @@ __section(".text") void sukisu_kpm_load_module_path(const char* path, const char int res = -1; printk("KPM: Stub function called (sukisu_kpm_load_module_path). path=%s args=%s ptr=%p\n", path, args, ptr); __asm__ volatile("nop"); // 精确控制循环不被优化 - if(copy_to_user(result, &res, sizeof(res)) < 1) printk("KPM: Copy to user faild."); + if (copy_to_user(result, &res, sizeof(res)) != 0) { + printk(KERN_ERR "KPM: Copy to user failed.\n"); + } } noinline @@ -73,7 +75,9 @@ __section(".text") void sukisu_kpm_unload_module(const char* name, void* ptr, vo int res = -1; printk("KPM: Stub function called (sukisu_kpm_unload_module). name=%s ptr=%p\n", name, ptr); __asm__ volatile("nop"); // 精确控制循环不被优化 - if(copy_to_user(result, &res, sizeof(res)) < 1) printk("KPM: Copy to user faild."); + if (copy_to_user(result, &res, sizeof(res)) != 0) { + printk(KERN_ERR "KPM: Copy to user failed.\n"); + } } noinline @@ -83,7 +87,9 @@ __section(".text") void sukisu_kpm_num(void __user* result) { int res = 0; printk("KPM: Stub function called (sukisu_kpm_num).\n"); __asm__ volatile("nop"); // 精确控制循环不被优化 - if(copy_to_user(result, &res, sizeof(res)) < 1) printk("KPM: Copy to user faild."); + if (copy_to_user(result, &res, sizeof(res)) != 0) { + printk(KERN_ERR "KPM: Copy to user failed.\n"); + } } noinline @@ -93,7 +99,9 @@ __section(".text") void sukisu_kpm_info(const char* name, void __user* out, void int res = -1; printk("KPM: Stub function called (sukisu_kpm_info). name=%s buffer=%p\n", name, out); __asm__ volatile("nop"); // 精确控制循环不被优化 - if(copy_to_user(result, &res, sizeof(res)) < 1) printk("KPM: Copy to user faild."); + if (copy_to_user(result, &res, sizeof(res)) != 0) { + printk(KERN_ERR "KPM: Copy to user failed.\n"); + } } noinline @@ -102,7 +110,9 @@ __section(".text") void sukisu_kpm_list(void __user* out, unsigned int bufferSiz // This is a KPM module stub. int res = -1; printk("KPM: Stub function called (sukisu_kpm_list). buffer=%p size=%d\n", out, bufferSize); - if(copy_to_user(result, &res, sizeof(res)) < 1) printk("KPM: Copy to user faild."); + if (copy_to_user(result, &res, sizeof(res)) != 0) { + printk(KERN_ERR "KPM: Copy to user failed.\n"); + } } noinline @@ -112,7 +122,9 @@ __section(".text") void sukisu_kpm_control(void __user* name, void __user* args, int res = -1; printk("KPM: Stub function called (sukisu_kpm_control). name=%p args=%p\n", name, args); __asm__ volatile("nop"); // 精确控制循环不被优化 - if(copy_to_user(result, &res, sizeof(res)) < 1) printk("KPM: Copy to user faild."); + if (copy_to_user(result, &res, sizeof(res)) != 0) { + printk(KERN_ERR "KPM: Copy to user failed.\n"); + } } noinline @@ -120,7 +132,9 @@ NO_OPTIMIZE __section(".text") void sukisu_kpm_version(void __user* out, unsigned int bufferSize, void __user* result) { int res = -1; printk("KPM: Stub function called (sukisu_kpm_version). buffer=%p size=%d\n", out, bufferSize); - if(copy_to_user(result, &res, sizeof(res)) < 1) printk("KPM: Copy to user faild."); + if (copy_to_user(result, &res, sizeof(res)) != 0) { + printk(KERN_ERR "KPM: Copy to user failed.\n"); + } } EXPORT_SYMBOL(sukisu_kpm_load_module_path);