# duplicate memory allocation in buildin-add.c

1 messages from 2006-09-15 to 2006-09-15. Participants: Liu Yubao.
Thread: https://gitlist.dev/t/5569

## Liu Yubao, 2006-09-15 11:26

Subject: duplicate memory allocation in buildin-add.c
Message-ID: <450A8DFF.9040605@gmail.com>
URL: https://gitlist.dev/e/450A8DFF.9040605%40gmail.com

```
http://www.kernel.org/git/?p=git/git.git;a=commitdiff;h=0d78153952e70c21e94dc6b7eefcb2ac5337a902

buildin-add.c:

+static void fill_directory(struct dir_struct *dir, const char **pathspec)
+{
+ 	const char *path, *base;
+ 	int baselen;
...
+ 	if (baselen) {
+ 		char *common = xmalloc(baselen + 1);
+ 		common = xmalloc(baselen + 1);
+ 		memcpy(common, *pathspec, baselen);
+ 		common[baselen] = 0;
+ 		path = base = common;
+ 	}
...

Allocate memory twice for "common" variable, is it indended? I'm very confused @_@

```