diff options
author | Brandon Casey <casey@nrlssc.navy.mil> | 2008-10-08 19:07:54 -0500 |
---|---|---|
committer | Shawn O. Pearce <spearce@spearce.org> | 2008-10-09 08:13:29 -0700 |
commit | 36e40535dcafe230a7f5ef43fea5cf67c1c58f6f (patch) | |
tree | 16778e960ebdeff1b627b5ae5d2221d94b955951 /t | |
parent | Do not use errno when pread() returns 0 (diff) | |
download | tgif-36e40535dcafe230a7f5ef43fea5cf67c1c58f6f.tar.xz |
builtin-merge.c: allocate correct amount of memory
Fix two memory allocation errors which allocate space for a pointer
rather than enough space for the structure itself.
This:
struct commit_list *parent = xmalloc(sizeof(struct commit_list *));
should have been this:
struct commit_list *parent = xmalloc(sizeof(struct commit_list));
But while we're at it, change the allocation to reference the
variable it is allocating memory for to try to prevent a similar
mistake, for example if the type is changed, in the future.
Signed-off-by: Brandon Casey <casey@nrlssc.navy.mil>
Acked-by: Miklos Vajna <vmiklos@frugalware.org>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Diffstat (limited to 't')
0 files changed, 0 insertions, 0 deletions