]> git.ipfire.org Git - thirdparty/grub.git/commit
support modules without symbol table
authorAndrei Borzenkov <arvidjaar@gmail.com>
Wed, 3 Feb 2016 17:34:55 +0000 (20:34 +0300)
committerAndrei Borzenkov <arvidjaar@gmail.com>
Wed, 3 Feb 2016 17:34:55 +0000 (20:34 +0300)
commit67dba97e4598eaf2deb14da044fbfb1c119cf76f
tree9ab5d276b1b82b65a99658fc522d3345428e4a8b
parent7290bb5623cd8d9b50b756d0ea40eb6a269985e7
support modules without symbol table

all_video module does not have any code or data and exists solely for
.moddeps section to pull in dependencies. This makes all symbols unneeded.

While in current binutils (last released version as of this commit is 2.26)
``strip --strip-unneeded'' unintentionally adds section symbols for each
existing section, this behavior was considered a bug and changed in commit
14f2c699ddca1e2f706342dffc59a6c7e23e844c to completely strip symbol table
in this case.

Older binutils (verified with 2.17) and some other toolchains (at least
elftoolchain r3223M), both used in FreeBSD, remove symbol table in all_video
as well.

Relax run-time check and do not return error for modules without symbol table.
Add additional checks to module verifier to make sure such modules

a) have non-empty .moddeps section. Without either externally visible symbols
or .moddeps modules are completely useless and should not be built.

b) do not have any relocations.

Closes: 46986
v2: add run-time check for empty symbol table if relocations are present as
    suggested by Vladimir.
grub-core/kern/dl.c
util/grub-module-verifierXX.c