Use GDALDatasetUniquePtr (#44)

* The raw pointer was never deleted and was a memory leak

Signed-off-by: Ryan Friedman <[email protected]>

Author: Ryanf55

src/grid_map_geo.cpp

@@ -73,7 +73,7 @@ bool GridMapGeo::Load(const std::string &map_path, bool algin_terrain, const std
 
 bool GridMapGeo::initializeFromGeotiff(const std::string &path, bool align_terrain) {
   GDALAllRegister();
-  GDALDataset *dataset = (GDALDataset *)GDALOpen(path.c_str(), GA_ReadOnly);
+  const auto dataset = GDALDatasetUniquePtr(GDALDataset::FromHandle(GDALOpen(path.c_str(), GA_ReadOnly)));
   if (!dataset) {
     std::cout << "Failed to open" << std::endl;
     return false;
@@ -171,7 +171,7 @@ bool GridMapGeo::initializeFromGeotiff(const std::string &path, bool align_terra
 
 bool GridMapGeo::addColorFromGeotiff(const std::string &path) {
   GDALAllRegister();
-  GDALDataset *dataset = (GDALDataset *)GDALOpen(path.c_str(), GA_ReadOnly);
+  const auto dataset = GDALDatasetUniquePtr(GDALDataset::FromHandle(GDALOpen(path.c_str(), GA_ReadOnly)));
   if (!dataset) {
     std::cout << "Failed to open" << std::endl;
     return false;
@@ -232,7 +232,7 @@ bool GridMapGeo::addColorFromGeotiff(const std::string &path) {
 
 bool GridMapGeo::addLayerFromGeotiff(const std::string &layer_name, const std::string &path) {
   GDALAllRegister();
-  GDALDataset *dataset = (GDALDataset *)GDALOpen(path.c_str(), GA_ReadOnly);
+  const auto dataset = GDALDatasetUniquePtr(GDALDataset::FromHandle(GDALOpen(path.c_str(), GA_ReadOnly)));
   if (!dataset) {
     std::cout << "Failed to open" << std::endl;
     return false;