fix: memset() called to fill 0 bytes.

jjwang · Jan 18, 2025 · 0f39fd4 · 0f39fd4
1 parent 653afa9
commit 0f39fd4
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/kernel/base/kmalloc.c b/kernel/base/kmalloc.c
@@ -54,7 +54,7 @@ void *kmalloc_chunk(uint64_t size, const char *func, uint64_t line)
     }
 
     /* TODO: for final release, this should be removed to improve speed */
-    memset(alloc, size + PAGE_SIZE, 0);
+    memset(alloc, 0, size + PAGE_SIZE);
 
     alloc->magic = MEM_MAGIC_NUM;
     alloc->checkno = kmalloc_checkno;

diff --git a/kernel/device/display/fb.c b/kernel/device/display/fb.c
@@ -199,7 +199,7 @@ void fb_init(fb_info_t *fb, struct limine_framebuffer* s)
     fb->backbuffer_len = fb->height * fb->pitch;
     fb->backbuffer = fb->addr;
 
-    memset(&(fb->img_bg), sizeof(image_t), 0);;
+    memset(&(fb->img_bg), 0, sizeof(image_t));
 
     for(uint32_t x = 0; x < fb->width; x++) {
         for(uint32_t y = 0; y < fb->height; y++) {

diff --git a/kernel/proc/syscall.c b/kernel/proc/syscall.c
@@ -1325,7 +1325,7 @@ int k_getrusage(int64_t who, uint64_t usage) {
      * gcc to know the purpose of this function call.
      */
     klogw("SYSCALL: get 0x%x rusage\n", who);
-    memset(u, sizeof(rusage_t), 0);
+    memset(u, 0, sizeof(rusage_t));
 
     return 0;
 }