Changeset e28175d in mainline for kernel/generic/src/mm/as.c

Timestamp:

2020-03-15T10:44:02Z (6 years ago)

Author:

GitHub <noreply@…>

Parents:

b401b33 (diff), 44dde42 (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

git-author:

heiducteam <tristanided@…> (2020-03-15 10:44:02)

git-committer:

GitHub <noreply@…> (2020-03-15 10:44:02)

Message:

Merge pull request #1 from HelenOS/master

sync

File:

• kernel/generic/src/mm/as.c (modified) (4 diffs)

kernel/generic/src/mm/as.c

@@ -2092 +2092 @@
 
 sysarg_t sys_as_area_create(uintptr_t base, size_t size, unsigned int flags,
-    uintptr_t bound, as_area_pager_info_t *pager_info)
+    uintptr_t bound, uspace_ptr_as_area_pager_info_t pager_info)
 {
         uintptr_t virt = base;
@@ -2098 +2098 @@
         mem_backend_data_t backend_data;
 
-        if (pager_info == AS_AREA_UNPAGED)
+        if (!pager_info)
                 backend = &anon_backend;
         else {
@@ -2125 +2125 @@
 }
 
-sys_errno_t sys_as_area_get_info(uintptr_t address, as_area_info_t *dest)
+sys_errno_t sys_as_area_get_info(uintptr_t address, uspace_ptr_as_area_info_t dest)
 {
         as_area_t *area;
+        as_area_info_t info;
+
+        /* Prevent leaking stack bytes via structure padding. */
+        memset(&info, 0, sizeof(info));
 
         mutex_lock(&AS->lock);
@@ -2136 +2140 @@
         }
 
-        dest->start_addr = area->base;
-        dest->size = P2SZ(area->pages);
-        dest->flags = area->flags;
+        info.start_addr = area->base;
+        info.size = P2SZ(area->pages);
+        info.flags = area->flags;
 
         mutex_unlock(&area->lock);
         mutex_unlock(&AS->lock);
+
+        copy_to_uspace(dest, &info, sizeof(info));
         return EOK;
 }