grub_dl_load_segments(): page-align the tramp/GOT areas. · Pull Request !1 · OpenCloudOS RPMS/grub2

The patch is from upstream:
    commit 862f81b5b007 ("grub_dl_load_segments(): page-align the tramp/GOT areas too")

The tramp/GOT write-protection in grub_dl_set_mem_attrs() requires that
the tramp/GOT areas of the module image *not* share a page with any other
memory allocations. Page-align the tramp/GOT areas, while satisfying their
intrinsic alignment requirements too.

=====

We have a different panic stack, but almost the same reason.
Our Panic call stack:

```
    grub_main
        grub_load_modules
	        grub_dl_load_core
		        grub_boot_time  *BOOM*

void
grub_real_boot_time (const char *file,
		     const int line,
		     const char *fmt, ...)
{
  struct grub_boot_time *n;
  va_list args;

grub_error_push ();
  n = grub_malloc (sizeof (*n));
  if (!n)
    {
      grub_errno = 0;
      grub_error_pop ();
      return;
    }
  n->file = file;
  n->line = line;
  n->tp = grub_get_time_ms ();
  n->next = 0;

va_start (args, fmt);
  n->msg = grub_xvasprintf (fmt, args);
  grub_dprintf ("boot", "%s\n", n->msg);
  va_end (args);

*boot_time_last = n;  <------ The global ptr has no write-permission
  boot_time_last = &n->next;

grub_errno = 0;
  grub_error_pop ();
}
```

Because dl load module changes the attrs of the page which
boot_time_lasts stays.

类型	指派人员	状态
审查	jiaxinyyang 已审查通过 xiezhongtian 已审查通过	已完成 (2/1人)
测试	jiaxinyyang 已测试通过 xiezhongtian 已测试通过	已完成 (2/1人)

OpenCloudOS RPMS/grub2 .gitee-modal { width: 500px !important; }

搜索帮助

OpenCloudOS RPMS/grub2