Messages in this thread Patch in this message | | | Date | Wed, 1 Jul 2009 12:29:55 -0700 | From | Andrew Morton <> | Subject | Re: [Patch 2/3] elf: clean up fill_note_info() |
| |
On Wed, 1 Jul 2009 01:06:36 -0400 Amerigo Wang <amwang@redhat.com> 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); _
| |