Messages in this thread | | | Date | Mon, 26 Nov 2007 19:19:49 +0900 | From | Yasunori Goto <> | Subject | Re: [Patch] mm/sparse.c: Improve the error handling for sparse_add_one_section() |
| |
Hi, Cong-san.
> ms->section_mem_map |= SECTION_MARKED_PRESENT; > > ret = sparse_init_one_section(ms, section_nr, memmap, usemap); > > out: > pgdat_resize_unlock(pgdat, &flags); > - if (ret <= 0) > - __kfree_section_memmap(memmap, nr_pages); > + > return ret; > } > #endif
Hmm. When sparse_init_one_section() returns error, memmap and usemap should be free.
Thanks for your fixing.
-- Yasunori Goto
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |