diff options
author | Rusty Russell <rusty@rustcorp.com.au> | 2012-10-25 10:49:25 +1030 |
---|---|---|
committer | Rusty Russell <rusty@rustcorp.com.au> | 2012-10-31 13:56:37 +1030 |
commit | 59ef28b1f14899b10d6b2682c7057ca00a9a3f47 (patch) | |
tree | 68ad548001fb24c0e63e1758d368a746998feb05 /kernel/lockdep_proc.c | |
parent | bc909421a9c7083fcde795846d22b36a51a7be54 (diff) |
module: fix out-by-one error in kallsyms
Masaki found and patched a kallsyms issue: the last symbol in a
module's symtab wasn't transferred. This is because we manually copy
the zero'th entry (which is always empty) then copy the rest in a loop
starting at 1, though from src[0]. His fix was minimal, I prefer to
rewrite the loops in more standard form.
There are two loops: one to get the size, and one to copy. Make these
identical: always count entry 0 and any defined symbol in an allocated
non-init section.
This bug exists since the following commit was introduced.
module: reduce symbol table for loaded modules (v2)
commit: 4a4962263f07d14660849ec134ee42b63e95ea9a
LKML: http://lkml.org/lkml/2012/10/24/27
Reported-by: Masaki Kimura <masaki.kimura.kz@hitachi.com>
Cc: stable@kernel.org
Diffstat (limited to 'kernel/lockdep_proc.c')
0 files changed, 0 insertions, 0 deletions