HW面试改错题

[higoalhwl]

eventType *CreateEvent(int groupId,int eventId,int length)
{
eventType *event;

event = (eventType*)malloc(sizeof(eventType));

if(event==NULL)
{
return NULL;
}
memset(event,0,sizeof(eventType));
event->groupId=groupId;
event->eventId=eventId;
event->length=length;

if(0!=length)
{
event->data=(unsigned char*)malloc(length);
if(NULL==event->data)
{
return NULL;
}
}

return event;
}

错在哪里啊？

[e_sharp]

event->data=(unsigned char*)malloc(length);
if(NULL==event->data)
{
return NULL;
}


--------------------------

这个地方如果malloc(length)内存分配失败的话，
event = (eventType*)malloc(sizeof(eventType))分配的内存没有释放。

[moonwrong]

而且名气也不小，经常有新闻。

[realdragon2]

mark~

[udb2008]

too easy.......

[lunarfan]

mark

[guzhilei1986]

哦哦，哈哈，自己看了半天也没有觉得有什么问题，看了大家的讨论才知道，关于内存的声明和释放真是一个意识，有经验的人才能像条件饭反射一样一眼就可以看出来。
我要多多学习哦。

[hahhh]

[Quote=引用 19 楼 swallowma 的回复:]
请注意此处也有隐患
if(0!=length)
{
event->data=(unsigned char*)malloc(length);

length 为Int型所以可能为负数，而malloc需要的参数是size_t也就是unsigned int，那么当length为负数时malloc将试图去申请一块非常大的内存，这样轻则分配失败，多次调用的话会大量占用系统内存资源。
[/Quote]
是啊，swallowma看得真仔细，这也是个不能忽视的细节。

[tangyulong1214]

eventType *CreateEvent(int groupId,int eventId,int length)
{
eventType *event ;

event = (eventType*)malloc(sizeof(eventType));

if(event==NULL)
{
return NULL;
}
memset((void *)event,0,sizeof(eventType));
event->groupId=groupId;
event->eventId=eventId;
event->length=length;

if(0!=length)
{
event->data=(unsigned char*)malloc(length);
if(NULL==event->data)
{
free(event);
return NULL;
}
}

return event;
}

[issachen]

swallowma的观察力真好，佩服ING

[blh]

又见内存泄漏问题，HW也不出点新鲜点的

eventType *CreateEvent(int groupId,int eventId,int length)
{
eventType *event;

event = (eventType*)malloc(sizeof(eventType));

if(event==NULL)
{
return NULL;
}
memset(event,0,sizeof(eventType));
event->groupId=groupId;
event->eventId=eventId;
event->length=length;

if(0!=length)
{
event->data=(unsigned char*)malloc(length);
if(NULL==event->data)
{
free(event);
return NULL;
}
}

return event;
}

[hans511002]

length 如果小于0会怎样?

[lims_xlh]

mark

[csuhawk]

俺也看出来了

[junsky2006]

呵呵，就是内存泄漏，如果第一个没有释放的话！

[sc_valentine21]

考的就是细心，缜密

[bleach210]

[Quote=引用 25 楼 lcgenius 的回复:]
学习了
[/Quote]

[chenzhiyubuaa]

说个和主题不相关的
if(event==NULL)
if(0!=length)
===================
这两个if的判断看起来就不爽

[飞哥]

eventType *CreateEvent(int groupId,int eventId,int length) 
{ 
eventType *event= (eventType*)malloc(sizeof(eventType)); 

if(NULL == event) 
{ 
    return NULL; 
} 
memset(event,0x0,sizeof(eventType)); 
event->groupId=groupId; 
event->eventId=eventId; 
event->length=length; 

if(0!=length) 
{ 
event->data=(unsigned char*)malloc(length * sizoef(unsigned char)); 
if(NULL==event->data) 
{ 
 free(event);
 event = NULL;
 return NULL; 
} 
} 

return event; 
}

[ack2me]

华为好进个毛啊！·#？

[zming]

36楼的，

if(event==NULL) 也应该写为
if ( NULL == event )

这样做的目的是为了防止少写一个 “=”

如果写成： if ( NULL = event ) 与 if ( 0 = length ), 编译时将立即报出语法错。

如果写成： if ( event = NULL ) 和 if ( length = 0 ) , 您就郁闷地慢慢去排错吧。