malloc函数引起的意外错误

在实现一个简单的字符对应函数的过程中发现了这个不算是BUG的错误：

这个错误出现的前提是这样的：

 

Main函数如下：

 

int main(int argc, char **argv)
{
	int num = 4;
	char *nodeName=NULL;
	nodeName = getNodeName(num);
	printf("\n\nnodeName is:\n%s\n\n",nodeName);
	char *aliasVal = (char *)malloc(10);
	getAliasVal(aliasVal, num);
	printf("\n\nnodeName is:\n%s\n\n",nodeName); //不能在这里输出nodeName,
	exit(0);

}

 

 

getNodeName函数：

 

char *getNodeName(int num)
{
	printf("\n\ntest num:%d\n\n",num);
/*	char *nodeName=NULL;*/
	char *nodeName = (char *)malloc(10);
	// 生成节点名
	switch (num)
	{
		case 1:
			nodeName = "port1";
			printf("\n\nin the switch:%s\n\n",nodeName);
			break;
		case 2:
			nodeName = "port2";
			break;
		case 3:
			nodeName = "port3";
			break;
		case 4:
			nodeName = "port4";
			printf("\n\nin the switch:%s\n\n",nodeName);
			break;
		case 5:
			nodeName = "port5";
			break;
		case 6:
			nodeName = "port6";
			break;
		case 7:
			nodeName = "port7";
			break;
		case 8:
			nodeName = "port8";
			break;
	}
	printf("\n\nOUT OF the switch:%s\n\n",nodeName);
	return nodeName;
}

 

getAliasVal函数

 

char *getAliasVal(char *aliasVal, int num)
{
	char b[1];
	int tmp = num - 1;
	if (tmp == 0)
	{
		strcpy(aliasVal, "ttyM0");
	}
	else
	{
		// 整数转字符串，方便拼接
		itoa(tmp, b);
		// 生成alias属性的值
	/*	char *aliasVal = (char *)malloc(10);*/
		strcpy(aliasVal, "ttyM");
		strcat(aliasVal, b);
	}
	printf("\n\nresult:\n%s\n\n",aliasVal);	
	return aliasVal;
}

 

 

错误产生在main函数的第9行那段，

该段代码不能正确产生输出，而前一句（第6行）是可以正确输出的，具有可重复性。

在getNodeName中nodeName也都能正确输出。

 

起初是怀疑nodeName指针出的问题，

但是用GDB进行简单的调试之后，发现nodeName指针并没有错误。

 

于是定位到malloc（）函数这里，

因为malloc（）函数之前可以正确输出，而malloc（）之后就乱了，而且每次执行的结果都不一样。（指针的地址每次不一样。）

 

解决办法目前我只想到取消使用malloc（）函数，我推测的原因是malloc把内存重新整理过才进行的分配，所以指针会乱。深层一点的代码我暂时没时间阅读……

做完毕业设计先……

评论

4 楼 nigelzeng 2012-05-15  

wuzhi3078 写道

这个是{}作用域引起的。

char *getAliasVal(char *aliasVal, int num)
{
	char b[1];
	int tmp = num - 1;
	if (tmp == 0)
	{
		strcpy(aliasVal, "ttyM0");
	}
	else
	{
		// 整数转字符串，方便拼接
		itoa(tmp, b);
		// 生成alias属性的值
		char *aliasVal = (char *)malloc(10);/*如果你在这里声明，编译器会以为aliasVal是这个{}里面的，不是参数那个*/
		strcpy(aliasVal, "ttyM");
		strcat(aliasVal, b);
	}
	printf("\n\nresult:\n%s\n\n",aliasVal);/*当else{}结束了，里面那个aliasVal变量也不见了，(没有free，分配的内存是存在的).这里的aliasVal其实是参数那个，因为参数那个aliasVal在传参前分配了内存，但是没有赋值，内存是乱码，printf打印的就是乱码。你看下面的例子*/
	return aliasVal;
}

char* GetVal(char *a, int m)
{
	{
		char *a = (char*)malloc(10);
		strcpy(a, "EEEEEEE\n");
		//return a;
	}
	return a;
}

void main()
{
	char *a;
	a =(char *) malloc(10);
	strcpy(a, "ABCDEF\n");
	a = GetVal(a, 1);
	printf("%s", a);
}

输出的是ABCDEF。就是说出了{}返回的是参数那个.如果把里面那个注释，返回的就是里面那个，输出EEEEEE. 如果没有大括号你会发现，变量重名编译出错，这跟全局变量，局部变量有点相似的

嗯，我试了一下，正如你所说。呵呵，谢谢你！

3 楼 wuzhi3078 2012-05-14  

这个是{}作用域引起的。

char *getAliasVal(char *aliasVal, int num)
{
	char b[1];
	int tmp = num - 1;
	if (tmp == 0)
	{
		strcpy(aliasVal, "ttyM0");
	}
	else
	{
		// 整数转字符串，方便拼接
		itoa(tmp, b);
		// 生成alias属性的值
		char *aliasVal = (char *)malloc(10);/*如果你在这里声明，编译器会以为aliasVal是这个{}里面的，不是参数那个*/
		strcpy(aliasVal, "ttyM");
		strcat(aliasVal, b);
	}
	printf("\n\nresult:\n%s\n\n",aliasVal);/*当else{}结束了，里面那个aliasVal变量也不见了，(没有free，分配的内存是存在的).这里的aliasVal其实是参数那个，因为参数那个aliasVal在传参前分配了内存，但是没有赋值，内存是乱码，printf打印的就是乱码。你看下面的例子*/
	return aliasVal;
}

char* GetVal(char *a, int m)
{
	{
		char *a = (char*)malloc(10);
		strcpy(a, "EEEEEEE\n");
		//return a;
	}
	return a;
}

void main()
{
	char *a;
	a =(char *) malloc(10);
	strcpy(a, "ABCDEF\n");
	a = GetVal(a, 1);
	printf("%s", a);
}

输出的是ABCDEF。就是说出了{}返回的是参数那个.如果把里面那个注释，返回的就是里面那个，输出EEEEEE. 如果没有大括号你会发现，变量重名编译出错，这跟全局变量，局部变量有点相似的

2 楼 nigelzeng 2012-05-14  

wuzhi3078 写道

/*  char *aliasVal = (char *)malloc(10);*/ 

这里才不认为是malloc的问题。
我认为是char *aliasVal重名的问题。不知道编译器为什么认为是符号。

(char *)aliasVal = (char *)malloc(10)

这样是通过的。我不知道你单单是测试malloc的bug还是干嘛。都没free。
估计是每个编译器的问题不一样吧，我用VC6。
谢谢你的博文！

是char *aliasVal的重名问题导致后面的nodeName输出错误吗？
free操作是我没有考虑到……
我接触linux c语言不久，所以很多地方还不明白，还请你不吝赐教。

1 楼 wuzhi3078 2012-05-14  

/*  char *aliasVal = (char *)malloc(10);*/ 

这里才不认为是malloc的问题。
我认为是char *aliasVal重名的问题。不知道编译器为什么认为是符号。

(char *)aliasVal = (char *)malloc(10)

这样是通过的。我不知道你单单是测试malloc的bug还是干嘛。都没free。
估计是每个编译器的问题不一样吧，我用VC6。
谢谢你的博文！