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<[email protected]>
Signed-off-by: Yu Ke<[email protected]>

 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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to