`
nigelzeng
  • 浏览: 598404 次
  • 性别: Icon_minigender_1
  • 来自: 杭州
社区版块
存档分类
最新评论

malloc函数引起的意外错误

    博客分类:
  • C
阅读更多

在实现一个简单的字符对应函数的过程中发现了这个不算是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把内存重新整理过才进行的分配,所以指针会乱。深层一点的代码我暂时没时间阅读……

做完毕业设计先……

0
3
分享到:
评论
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。
谢谢你的博文!

相关推荐

Global site tag (gtag.js) - Google Analytics