c++ - 运行时错误 : Access violation writing loacation while deleting a pointer in dynamic memory allocation

Question

我编写这段代码是为了在 opengl 窗口中显示线形文件。
实际上是代码 Rendering Shapefile in OpenGL here in code-project ，但我正在对其进行一些更改，以便使用 OGR 库而不是 shapelib 读取形状。

#include "ogrsf_frmts.h" 
//#include "shapelib\shapefil.h"

void OpenShapeFile(char* filename)
{
   int i = 0;
   int j = 0;
   OGRErr error;
   OGRDataSource *poDataSource;
   poDataSource = OGRSFDriverRegistrar::Open(filename,false);
   OGRLayer *poLayer;
   poLayer = poDataSource ->GetLayer(0);
   OGREnvelope *poEnvelope = new OGREnvelope();
   error = poLayer ->GetExtent(poEnvelope,true);
   sBoundingBox.fMaxX = poEnvelope ->MaxX;
   sBoundingBox.fMaxY = poEnvelope ->MaxY;
   sBoundingBox.fMinX = poEnvelope ->MinX;
   sBoundingBox.fMinY = poEnvelope ->MinY;
   delete poEnvelope;
   OGRwkbGeometryType GeometryType = poLayer ->GetGeomType();
   int NumberOfFeatures = poLayer ->GetFeatureCount(true);
   poLayer ->ResetReading();

   //Line Shapefile
   if ( wkbFlatten ( GeometryType ) == wkbLineString )
   {
       OGRFeature *poFeature;
       MyLineString2D lineString;

       //temporary pointer in order to store coordinates of individual line vertexes
       OGRPoint *poPointTemp = new OGRPoint();
       for ( i = 0; i < NumberOfFeatures; i++ )
       {
           poFeature = poLayer ->GetNextFeature();
           OGRGeometry *poGeometry;
           poGeometry = poFeature ->GetGeometryRef();
           if ( poGeometry != NULL )
           {
               OGRLineString *poLineString = (OGRLineString *)poGeometry;
               int NumberOfVertexes = poLineString ->getNumPoints();
               lineString.vPointList.resize(NumberOfVertexes);
               for ( j = 0; j < NumberOfVertexes ; j++ )
               {
                   poLineString ->getPoint(j,poPointTemp);
                   MyPoint2D ptTemp;
                   ptTemp.dX = poPointTemp ->getX();
                   ptTemp.dY = poPointTemp ->getY();
                   lineString.vPointList.at(j) = ptTemp;
               }
               vLines.push_back(lineString);
           }
           OGRFeature::DestroyFeature( poFeature );
       }
   delete poPointTemp;
   } 
}  

void initializeGL()
{
    //glClearColor (0.0, 0.0, 0.0, 0.0);
    glClearColor (1.0, 1.0, 1.0, 1.0);  

int main(int argc, char** argv)
{
//OpenShapeFile("Shapefiles\\poi.shp");//Point Shapefile
OpenShapeFile("Shapefiles\\strassen.shp");//Line Shapefile
//OpenShapeFile("Shapefiles\\gruenflaechen.shp");//Polygon Shapefile

当我评论这行时:

delete poPointTemp;  

没有错误，并且在 OpenGL 窗口上正确绘制了形状。但如您所知，这是一种内存泄漏，所以当我不需要 poPointTemp 时，我应该删除它。
但在线:

delete poPointTemp;  

我收到运行时错误:

我已经调试了我的代码，在运行这一行之前poPointTemp的地址和内容如下:

poPointTemp 0x00503a90 {x=3435936.3300000001 y=5790327.5999999996 z=0.00000000000000000 }  

我的意思是 poPointTemp 不是 NULL 指针，而且我们无法访问的地址与 poPointTemp 的地址不同!! !

您认为我的代码有什么问题？

我已尽量减少代码。所以如果你想自己调试代码可以在code-project下载。然后根据我上面写的代码对其进行更改。

• 请使用我的OpenShapeFile 方法而不是代码的相应的方法并对代码进行其他更改。
  我也上传了我的代码 here in 4shared ，你可以把它带到这里并链接GDAL Library并运行它。

---

当然，在 john 的回答的帮助下，我的问题得到了解决。但我仍然会感谢有人告诉我上面代码的问题是什么？

Answer 1

这不是一个真正的答案，但它可能会有所帮助，而且发表评论的时间有点太长了。另请记住，我不了解此库，因此我的建议可能不正确。

毫无疑问，这个错误掩盖了代码中其他地方的真正错误。正如你所说，此时你应该删除内存。但是，在我看来，您确实在不必要地分配内存。比如这个

OGREnvelope *poEnvelope = new OGREnvelope();
error = poLayer ->GetExtent(poEnvelope,true);
sBoundingBox.fMaxX = poEnvelope ->MaxX;
sBoundingBox.fMaxY = poEnvelope ->MaxY;
sBoundingBox.fMinX = poEnvelope ->MinX;
sBoundingBox.fMinY = poEnvelope ->MinY;
delete poEnvelope;

可以像这样在没有任何内存分配的情况下重写

OGREnvelope poEnvelope;
error = poLayer ->GetExtent(&poEnvelope,true);
sBoundingBox.fMaxX = poEnvelope.MaxX;
sBoundingBox.fMaxY = poEnvelope.MaxY;
sBoundingBox.fMinX = poEnvelope.MinX;
sBoundingBox.fMinY = poEnvelope.MinY;

可以对 poPointTemp 进行类似的更改。一般来说，如果您发现自己为某个对象分配内存，然后在同一范围内 删除它，则表明您根本不需要分配内存。

这些都不能修复仍然隐藏的错误，但重点是通过简化代码和分配更少的内存，您将更接近真正的错误，这无疑是某种内存损坏。