Commit 2e45e777 authored by Rusty Russell's avatar Rusty Russell
Browse files

Revert "module: remove the SHF_ALLOC flag on the __versions section."

This reverts commit 9cb610d8


This was an impressively stupid patch.  Firstly, we reset the SHF_ALLOC
flag lower down in the same function, so the patch was useless.  Even
better, find_sec() ignores sections with SHF_ALLOC not set, so
refuses to load the module since it can't find the __versions section.
Signed-off-by: default avatarRusty Russell <>
parent d508afb4
......@@ -1952,9 +1952,6 @@ static noinline struct module *load_module(void __user *umod,
if (strstarts(secstrings+sechdrs[i].sh_name, ".exit"))
sechdrs[i].sh_flags &= ~(unsigned long)SHF_ALLOC;
/* Don't keep __versions around; it's just for loading. */
if (strcmp(secstrings + sechdrs[i].sh_name, "__versions") == 0)
sechdrs[i].sh_flags &= ~(unsigned long)SHF_ALLOC;
modindex = find_sec(hdr, sechdrs, secstrings,
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment