2014-12-03 108 views
1

我有一个const char *,我需要用逗号分开。因为strtok修改了输入字符串,所以我复制它,并在函数结尾释放分配的内存。内存泄漏,尽管释放分配的内存

void ApBuilder::addNetworkType(ApDbData::RowIterator &iter) 
{ 
    const char * type = iter.getColumnText(ApDbData::AP_IDX_TYPE_80211); 

    const size_t len = strlen(type); 
    char * temp = new char[len +1]; 
    strncpy(temp, type, len); 
    temp[len] = '\0'; 

    temp = strtok(temp, ","); 

    while(temp != NULL) 
    { 
     tmpObject.add(temp, true); 

     temp = strtok(NULL, ","); 
    } 

    jsonObject.add("type80211", tmpObject); 

    delete[] temp; 
} 

Valgrind抱怨说,尽管我释放了分配的内存,但仍有内存泄漏。我该如何解决此漏洞

==17667== 8 bytes in 2 blocks are definitely lost in loss record 1 of 4 
==17667== at 0x402ADFC: operator new[](unsigned int) (in /usr/lib/valgrind/vgpreload_memcheck-x86-linux.so) 
==17667== by 0x8049C5D: ApBuilder::addNetworkType(AbstractDbData::RowIterator&) (in /home/***/workspace/projects/jsonBuilder/main) 
==17667== by 0x8049A38: ApBuilder::buildApArray() (in /home/***/workspace/projects/jsonBuilder/main) 
==17667== by 0x8049679: main (in /home/***/workspace/projects/jsonBuilder/main) 
+0

'jsonObject.add'可以抛出吗? – Deduplicator 2014-12-03 18:10:04

回答

2

strtok修改临时指针。您需要删除原始指针值。 (为此将它保存在一个变量中。)

1

您需要为strtok值指定temp_ptr。每当您重新指定temp时,您都会放弃临时字符串的开始。

tok_ptr = strtok(temp, ","); 

while(tok_ptr != NULL) 
{ 

//你可能想添加最后tok_ptr和明年tok_prt之间的子 tmpObject.add(温度,真正的);

tok_prt = strtok(NULL, ","); 
} 
0

ScottMcP-MVP说,导致内存泄漏的问题是strtok。

C++避免此类泄漏的方法是使用众多std帮助程序之一: auto_ptr/unique_ptr,vector。为什么不是字符串?

std::string copy(iter.getColumnText(ApDbData::AP_IDX_TYPE_80211); 
copy.append('\0'); 
char *temp = &copy[0]; 
temp = strtok(temp, ","); 
....