[PATCH 0/2] Fixes for ELF cleanup, round 2

[Andreas Robinson <andr345@xxxxxxxxx>]

On Tue, 2009-05-19 at 04:58 -0400, Jon Masters wrote:
> On Tue, 2009-05-19 at 04:22 -0400, Jon Masters wrote:
> 
> > Appreciate very much the patch series, especially patch 4. I had to wait
> > on testing on PPC64 until this morning (have a replacement network
> > switch now), which is why you didn't see the release yet, however I'm
> > about to push an updated tree so that we don't have to wait for that. I
> > am not willing to push a release with the ELF changes without checking
> > LE/BE on 32/64-bit targets properly this morning.
> 
> Official tree currently has both "elf_cleanup" and "elf_cleanup2". I've
> also added rworkman's documentation cleanups, and two patches from Alan.

I hope you haven't released yet. :)
Here are a pair of patches that address the problems Alan found.

Still working on the fix for the hack in patch 9 (fail noisily on
non-ELFs) and the bounds check for next_string().

[PATCH 1/2] fix minor strtbl_free and strtbl_add problems

- add missing NOFAIL to strtbl_add() in load_dep_syms()
- export strtbl_free()
- free string tables with strtbl_free()
- documentation fix

[PATCH 2/2] elfops: fix potential crashing bug in load_symbols()

get_section() returned an undefined section size on failure,
which could cause load_symbols() to crash if kernel symtables
were missing from the module.

get_section() will now return secsize == 0 if it fails.

--
To unsubscribe from this list: send the line "unsubscribe linux-modules" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html