On Wed, 1 Jul 2009 01:06:36 -0400 Amerigo Wang <amwang@xxxxxxxxxx> wrote: > +#define NUM_NOTES 6 > info->notes = kmalloc(NUM_NOTES * sizeof(struct memelfnote), > GFP_KERNEL); > +#undef NUM_NOTES That #define amounts to a really perverse code comment. How about we do this? --- a/fs/binfmt_elf.c~elf-clean-up-fill_note_info-fix +++ a/fs/binfmt_elf.c @@ -1719,10 +1719,8 @@ static int elf_note_info_init(struct elf memset(info, 0, sizeof(*info)); INIT_LIST_HEAD(&info->thread_list); -#define NUM_NOTES 6 - info->notes = kmalloc(NUM_NOTES * sizeof(struct memelfnote), - GFP_KERNEL); -#undef NUM_NOTES + /* Allocate space for six ELF notes */ + info->notes = kmalloc(6 * sizeof(struct memelfnote), GFP_KERNEL); if (!info->notes) return 0; info->psinfo = kmalloc(sizeof(*info->psinfo), GFP_KERNEL); _ -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html