On 13.6.2011 10:55, Yu, Ke wrote:
Mark Hatle ran into a random segfault using the module-init-tools 3.12 and finally trace back to depmod.c:grab_module, which appears that the new malloc(...) setups up things, but never clears the memory that was just allocated. So this patch fix this issue by clear the newly allocated memory. Signed-off-by: Mark Hatle<mark.hatle@xxxxxxxxxxxxx> Signed-off-by: Yu Ke<ke.yu@xxxxxxxxx> From Yocto Project (www.yoctoproject.org), project for embedded linux. diff --git a/depmod.c b/depmod.c index a1d2f8c..b450808 100644 --- a/depmod.c +++ b/depmod.c @@ -430,6 +430,7 @@ static struct module *grab_module(const char *dirname, const char *filename) new = NOFAIL(malloc(sizeof(*new) + strlen(dirname?:"") + 1 + strlen(filename) + 1)); + memset(new, 0x00, sizeof(*new) + strlen(dirname?:"") + 1 + strlen(filename) + 1);
If you change the malloc() to calloc(), then you won't have to write the lengthy sum twice.
Michal -- 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