Commented Unassigned: Memory leak in CMapView::LoadMapState [24898]

There is a memory leak in `CMapView::LoadMapState()` method. The `CPLXMLNode* node` that gets allocated during XML parsing is never free'd.

I suggest to insert a call to `CPLDestroyXMLNode(node);` right before returning the parse result:
``` C++
// *********************************************************
// LoadMapState()
// *********************************************************
VARIANT_BOOL CMapView::LoadMapState(LPCTSTR Filename, IDispatch* Callback)
{
AFX_MANAGE_STATE(AfxGetStaticModuleState());

try
{
CPLXMLNode* node = CPLParseXMLFile(Filename);
if (node)
{
IStopExecution* cb = NULL;
if (Callback)
{
Callback->QueryInterface(IID_IStopExecution, (void**)&cb);
}

bool result = DeserializeMapStateCore(node, Filename, VARIANT_TRUE, cb);

if (cb)
{
cb->Release();
}
CPLDestroyXMLNode(node);
return result ? VARIANT_TRUE : VARIANT_FALSE;
}
else
{
return VARIANT_FALSE;
}
}
catch(...)
{
return VARIANT_FALSE;
}
}
```
Comments: ** Comment from web user: fjkaiser **

Another memory leak exists in `CMapView::SerializeMapState()` method.

Here is my suggested solution:

```
// ************************************************************
// SerializeMapState()
// ************************************************************
BSTR CMapView::SerializeMapState(VARIANT_BOOL RelativePaths, LPCTSTR BasePath)
{
AFX_MANAGE_STATE(AfxGetStaticModuleState());
CString strResult;
CPLXMLNode* node = SerializeMapStateCore(RelativePaths, BasePath);
if (node)
{
strResult = CPLSerializeXMLTree(node);
CPLDestroyXMLNode(node);
}
return strResult.AllocSysString();
}
```