From 88a84b71b7c76c5bcbb1b069ac582d355c7a0151 Mon Sep 17 00:00:00 2001 From: ShirkNeko <109797057+ShirkNeko@users.noreply.github.com> Date: Tue, 1 Apr 2025 12:04:40 +0800 Subject: [PATCH] =?UTF-8?q?=E4=BF=AE=E5=A4=8D=20KPM=20=E6=A8=A1=E5=9D=97?= =?UTF-8?q?=E4=B8=AD=20copy=5Fto=5Fuser=20=E8=B0=83=E7=94=A8=E7=9A=84?= =?UTF-8?q?=E9=94=99=E8=AF=AF=E5=A4=84=E7=90=86=EF=BC=8C=E7=A1=AE=E4=BF=9D?= =?UTF-8?q?=E5=9C=A8=E5=A4=8D=E5=88=B6=E5=A4=B1=E8=B4=A5=E6=97=B6=E8=AE=B0?= =?UTF-8?q?=E5=BD=95=E9=94=99=E8=AF=AF=E4=BF=A1=E6=81=AF?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- kernel/kpm/kpm.c | 28 +++++++++++++++++++++------- 1 file changed, 21 insertions(+), 7 deletions(-) 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);