2017-11-11 137 views
-1

我有一个包含一组char *属性的结构。清空包含char的结构数组*

struct gcp_values { 
char* srcX; 
char* srcY; 
char* dstX; 
char* dstY; 
}; 

好,我填补它在一个循环从集合Q对象

std::vector<GeoRefLibrary::gcp_values> gcpvalues(vl.size()); 

    for(int i=0;i<vl.size();i++) { 
     gcppoint_ *a = qobject_cast<gcppoint_ *>(vl.at(i).value<QObject *>()); 
     gcpvalues[i].srcX= strdup(QString::number(a->row()).toStdString().c_str()); 
     gcpvalues[i].srcY= strdup(QString::number(a->column()).toStdString().c_str()); 
     gcpvalues[i].dstX= strdup(QString::number(a->lon()).toStdString().c_str()); 
     gcpvalues[i].dstY= strdup(QString::number(a->lat()).toStdString().c_str()); 
    } 
当我运行Valgrind的测试结果,这些OUTPT

12 bytes in 4 blocks are definitely lost in loss record 813 of 19,623 
14 bytes in 4 blocks are definitely lost in loss record 889 of 19,623 
16 bytes in 1 blocks are definitely lost in loss record 3,621 of 19,623 
32 bytes in 4 blocks are definitely lost in loss record 8,851 of 19,623 
36 bytes in 4 blocks are definitely lost in loss record 9,134 of 19,623 

这些错误是在这些线路

的循环。我试图delete [] gcpvalues,但它返回错误的删除/免费错误。我搜索,发现这个职位太

delete[] an array of objects

,但不能找到解决方案,可有一个人请帮我解决这个问题?

我也试过

struct gcp_values { 
    QString srcX; 
    QString srcY; 
    QString dstX; 
    QString dstY; 
}; 

和返回同样的错误


我改变了循环到这一点,似乎错误都没有了

for(int i=0;i<vl.size();i++) { 
    gcppoint_ *a = qobject_cast<gcppoint_ *>(vl.at(i).value<QObject *>()); 

     char* srcX = strdup(QString::number(a->row()).toStdString().c_str()); 
     char* srcY = strdup(QString::number(a->column()).toStdString().c_str()); 
     char* dstX = strdup(QString::number(a->lon()).toStdString().c_str()); 
     char* dstY = strdup(QString::number(a->lat()).toStdString().c_str()); 
     gcpvalues[i].srcX=srcX; 
     gcpvalues[i].srcY= srcY; 
     gcpvalues[i].dstX= dstX; 
     gcpvalues[i].dstY= dstY; 
    if(srcX) { free(srcX);} 
    if(srcY) { free(srcY);} 
    if(dstX) { free(dstX);} 
    if(dstY) { free(dstY);} 
} 
+1

'strdup'文件特别指出:*必须将返回的指针传递给空闲以避免内存泄漏。* –

+2

只需将'QString's放入'struct'中即可避免此问题。手动内存管理是困难和不必要的。不要这样做。 – nwp

+0

@nwp然后以后如何将Qstring转换为char *,因为我在后面的函数中需要char *。 –

回答

1

strdup是POSIX函数和分配内存的POSIX函数(并且不提供类型特定的释放乐趣例如fclosefreeaddrinfo)期望调用方最终调用free来释放分配的内存。

POSIX不使用C++内存分配,所以operator delete[]在这里不正确。

+0

感谢它是主问题,我改变了循环,它似乎工作。但如果我在循环中释放值,它是否工作? –

+0

只要您将每个指针值释放一次,它就会工作。 –

+0

当然,这不是关于POSIX,而是任何C API,因为C没有像C++那样通过RAII自动取消分配。 –

相关问题