From 4642408066a0aaeccaa156c8e652c0caff84ee4e Mon Sep 17 00:00:00 2001
From: Javier Arribas <javiarribas@gmail.com>
Date: Tue, 25 Jul 2017 16:12:10 +0200
Subject: [PATCH] Adding exception handlings to tracking destructor operations

---
 .../galileo_e1_dll_pll_veml_tracking_cc.cc    | 27 ++++++++++----
 .../galileo_e1_tcp_connector_tracking_cc.cc   | 29 +++++++++++----
 .../galileo_e5a_dll_pll_tracking_cc.cc        | 37 ++++++++++++-------
 .../gps_l1_ca_dll_pll_c_aid_tracking_cc.cc    | 27 ++++++++++----
 ...ps_l1_ca_dll_pll_c_aid_tracking_fpga_sc.cc | 30 ++++++++++-----
 .../gps_l1_ca_dll_pll_c_aid_tracking_sc.cc    | 29 +++++++++++----
 .../gps_l1_ca_dll_pll_tracking_cc.cc          | 28 ++++++++++----
 .../gps_l1_ca_dll_pll_tracking_gpu_cc.cc      | 31 ++++++++++++----
 .../gps_l1_ca_tcp_connector_tracking_cc.cc    | 30 ++++++++++-----
 .../gps_l2_m_dll_pll_tracking_cc.cc           | 27 ++++++++++----
 10 files changed, 211 insertions(+), 84 deletions(-)

diff --git a/src/algorithms/tracking/gnuradio_blocks/galileo_e1_dll_pll_veml_tracking_cc.cc b/src/algorithms/tracking/gnuradio_blocks/galileo_e1_dll_pll_veml_tracking_cc.cc
index a16eaaa0a..c611bbb9c 100755
--- a/src/algorithms/tracking/gnuradio_blocks/galileo_e1_dll_pll_veml_tracking_cc.cc
+++ b/src/algorithms/tracking/gnuradio_blocks/galileo_e1_dll_pll_veml_tracking_cc.cc
@@ -251,14 +251,27 @@ void galileo_e1_dll_pll_veml_tracking_cc::start_tracking()
 
 galileo_e1_dll_pll_veml_tracking_cc::~galileo_e1_dll_pll_veml_tracking_cc()
 {
-    d_dump_file.close();
+	if (d_dump_file.is_open())
+	{
+		try
+		{
+			d_dump_file.close();
+		}catch(const std::exception & ex)
+		{
+			LOG(WARNING)<<"Exception in destructor "<<ex.what();
+		}
+	}
+	try{
+		volk_gnsssdr_free(d_local_code_shift_chips);
+		volk_gnsssdr_free(d_correlator_outs);
+		volk_gnsssdr_free(d_ca_code);
+		delete[] d_Prompt_buffer;
+		multicorrelator_cpu.free();
+	}catch(const std::exception & ex)
+	{
+		LOG(WARNING)<<"Exception in destructor "<<ex.what();
+	}
 
-    volk_gnsssdr_free(d_local_code_shift_chips);
-    volk_gnsssdr_free(d_correlator_outs);
-    volk_gnsssdr_free(d_ca_code);
-
-    delete[] d_Prompt_buffer;
-    multicorrelator_cpu.free();
 }
 
 
diff --git a/src/algorithms/tracking/gnuradio_blocks/galileo_e1_tcp_connector_tracking_cc.cc b/src/algorithms/tracking/gnuradio_blocks/galileo_e1_tcp_connector_tracking_cc.cc
index 9e2d23277..f609d1a09 100644
--- a/src/algorithms/tracking/gnuradio_blocks/galileo_e1_tcp_connector_tracking_cc.cc
+++ b/src/algorithms/tracking/gnuradio_blocks/galileo_e1_tcp_connector_tracking_cc.cc
@@ -244,15 +244,28 @@ void Galileo_E1_Tcp_Connector_Tracking_cc::start_tracking()
 
 Galileo_E1_Tcp_Connector_Tracking_cc::~Galileo_E1_Tcp_Connector_Tracking_cc()
 {
-    d_dump_file.close();
+	if (d_dump_file.is_open())
+	{
+		try
+		{
+			d_dump_file.close();
+		}catch(const std::exception & ex)
+		{
+			LOG(WARNING)<<"Exception in destructor "<<ex.what();
+		}
+	}
+	try{
+		volk_gnsssdr_free(d_local_code_shift_chips);
+		volk_gnsssdr_free(d_correlator_outs);
+		volk_gnsssdr_free(d_ca_code);
+		delete[] d_Prompt_buffer;
+	    d_tcp_com.close_tcp_connection(d_port);
+		multicorrelator_cpu.free();
+	}catch(const std::exception & ex)
+	{
+		LOG(WARNING)<<"Exception in destructor "<<ex.what();
+	}
 
-    delete[] d_Prompt_buffer;
-    volk_gnsssdr_free(d_ca_code);
-    volk_gnsssdr_free(d_local_code_shift_chips);
-    volk_gnsssdr_free(d_correlator_outs);
-
-    d_tcp_com.close_tcp_connection(d_port);
-    multicorrelator_cpu.free();
 }
 
 
diff --git a/src/algorithms/tracking/gnuradio_blocks/galileo_e5a_dll_pll_tracking_cc.cc b/src/algorithms/tracking/gnuradio_blocks/galileo_e5a_dll_pll_tracking_cc.cc
index cacfd2091..0ce58036f 100644
--- a/src/algorithms/tracking/gnuradio_blocks/galileo_e5a_dll_pll_tracking_cc.cc
+++ b/src/algorithms/tracking/gnuradio_blocks/galileo_e5a_dll_pll_tracking_cc.cc
@@ -213,20 +213,31 @@ Galileo_E5a_Dll_Pll_Tracking_cc::Galileo_E5a_Dll_Pll_Tracking_cc(
 
 Galileo_E5a_Dll_Pll_Tracking_cc::~Galileo_E5a_Dll_Pll_Tracking_cc ()
 {
-    d_dump_file.close();
+	if (d_dump_file.is_open())
+	{
+		try
+		{
+			d_dump_file.close();
+		}catch(const std::exception & ex)
+		{
+			LOG(WARNING)<<"Exception in destructor "<<ex.what();
+		}
+	}
+	try{
+	    delete[] d_codeI;
+	    delete[] d_codeQ;
+	    delete[] d_Prompt_buffer;
+		volk_gnsssdr_free(d_local_code_shift_chips);
+		volk_gnsssdr_free(d_correlator_outs);
+	    volk_gnsssdr_free(d_Single_Prompt_data);
+		delete[] d_Prompt_buffer;
+	    multicorrelator_cpu_Q.free();
+	    multicorrelator_cpu_I.free();
+	}catch(const std::exception & ex)
+	{
+		LOG(WARNING)<<"Exception in destructor "<<ex.what();
+	}
 
-    delete[] d_codeI;
-    delete[] d_codeQ;
-    delete[] d_Prompt_buffer;
-
-    d_dump_file.close();
-
-    volk_gnsssdr_free(d_local_code_shift_chips);
-    volk_gnsssdr_free(d_correlator_outs);
-    volk_gnsssdr_free(d_Single_Prompt_data);
-
-    multicorrelator_cpu_Q.free();
-    multicorrelator_cpu_I.free();
 }
 
 
diff --git a/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_dll_pll_c_aid_tracking_cc.cc b/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_dll_pll_c_aid_tracking_cc.cc
index 8c94d9e1e..fdcc152ec 100644
--- a/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_dll_pll_c_aid_tracking_cc.cc
+++ b/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_dll_pll_c_aid_tracking_cc.cc
@@ -299,14 +299,27 @@ void gps_l1_ca_dll_pll_c_aid_tracking_cc::start_tracking()
 
 gps_l1_ca_dll_pll_c_aid_tracking_cc::~gps_l1_ca_dll_pll_c_aid_tracking_cc()
 {
-    d_dump_file.close();
 
-    volk_gnsssdr_free(d_local_code_shift_chips);
-    volk_gnsssdr_free(d_correlator_outs);
-    volk_gnsssdr_free(d_ca_code);
-
-    delete[] d_Prompt_buffer;
-    multicorrelator_cpu.free();
+	if (d_dump_file.is_open())
+	{
+		try
+		{
+			d_dump_file.close();
+		}catch(const std::exception & ex)
+		{
+			LOG(WARNING)<<"Exception in destructor "<<ex.what();
+		}
+	}
+	try{
+		volk_gnsssdr_free(d_local_code_shift_chips);
+		volk_gnsssdr_free(d_correlator_outs);
+		volk_gnsssdr_free(d_ca_code);
+		delete[] d_Prompt_buffer;
+		multicorrelator_cpu.free();
+	}catch(const std::exception & ex)
+	{
+		LOG(WARNING)<<"Exception in destructor "<<ex.what();
+	}
 }
 
 
diff --git a/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_dll_pll_c_aid_tracking_fpga_sc.cc b/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_dll_pll_c_aid_tracking_fpga_sc.cc
index e466898a6..31e73f5db 100644
--- a/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_dll_pll_c_aid_tracking_fpga_sc.cc
+++ b/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_dll_pll_c_aid_tracking_fpga_sc.cc
@@ -320,15 +320,27 @@ void gps_l1_ca_dll_pll_c_aid_tracking_fpga_sc::start_tracking()
 
 gps_l1_ca_dll_pll_c_aid_tracking_fpga_sc::~gps_l1_ca_dll_pll_c_aid_tracking_fpga_sc()
 {
-    d_dump_file.close();
-
-    volk_gnsssdr_free(d_local_code_shift_chips);
-    volk_gnsssdr_free(d_ca_code);
-    volk_gnsssdr_free(d_ca_code_16sc);
-    volk_gnsssdr_free(d_correlator_outs_16sc);
-
-    delete[] d_Prompt_buffer;
-    multicorrelator_fpga_8sc->free();
+	if (d_dump_file.is_open())
+	{
+		try
+		{
+			d_dump_file.close();
+		}catch(const std::exception & ex)
+		{
+			LOG(WARNING)<<"Exception in destructor "<<ex.what();
+		}
+	}
+	try{
+	    volk_gnsssdr_free(d_local_code_shift_chips);
+	    volk_gnsssdr_free(d_ca_code);
+	    volk_gnsssdr_free(d_ca_code_16sc);
+	    volk_gnsssdr_free(d_correlator_outs_16sc);
+		delete[] d_Prompt_buffer;
+	    multicorrelator_fpga_8sc->free();
+	}catch(const std::exception & ex)
+	{
+		LOG(WARNING)<<"Exception in destructor "<<ex.what();
+	}
 }
 
 
diff --git a/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_dll_pll_c_aid_tracking_sc.cc b/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_dll_pll_c_aid_tracking_sc.cc
index 3a2d23acc..aa8fd70bb 100644
--- a/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_dll_pll_c_aid_tracking_sc.cc
+++ b/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_dll_pll_c_aid_tracking_sc.cc
@@ -302,15 +302,28 @@ void gps_l1_ca_dll_pll_c_aid_tracking_sc::start_tracking()
 
 gps_l1_ca_dll_pll_c_aid_tracking_sc::~gps_l1_ca_dll_pll_c_aid_tracking_sc()
 {
-    d_dump_file.close();
+	if (d_dump_file.is_open())
+	{
+		try
+		{
+			d_dump_file.close();
+		}catch(const std::exception & ex)
+		{
+			LOG(WARNING)<<"Exception in destructor "<<ex.what();
+		}
+	}
+	try{
+	    volk_gnsssdr_free(d_local_code_shift_chips);
+	    volk_gnsssdr_free(d_ca_code);
+	    volk_gnsssdr_free(d_ca_code_16sc);
+	    volk_gnsssdr_free(d_correlator_outs_16sc);
+		delete[] d_Prompt_buffer;
+	    multicorrelator_cpu_16sc.free();
+	}catch(const std::exception & ex)
+	{
+		LOG(WARNING)<<"Exception in destructor "<<ex.what();
+	}
 
-    volk_gnsssdr_free(d_local_code_shift_chips);
-    volk_gnsssdr_free(d_ca_code);
-    volk_gnsssdr_free(d_ca_code_16sc);
-    volk_gnsssdr_free(d_correlator_outs_16sc);
-
-    delete[] d_Prompt_buffer;
-    multicorrelator_cpu_16sc.free();
 }
 
 
diff --git a/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_dll_pll_tracking_cc.cc b/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_dll_pll_tracking_cc.cc
index 89aafca07..5a4b60551 100644
--- a/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_dll_pll_tracking_cc.cc
+++ b/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_dll_pll_tracking_cc.cc
@@ -267,14 +267,26 @@ void Gps_L1_Ca_Dll_Pll_Tracking_cc::start_tracking()
 
 Gps_L1_Ca_Dll_Pll_Tracking_cc::~Gps_L1_Ca_Dll_Pll_Tracking_cc()
 {
-    d_dump_file.close();
-
-    volk_gnsssdr_free(d_local_code_shift_chips);
-    volk_gnsssdr_free(d_correlator_outs);
-    volk_gnsssdr_free(d_ca_code);
-
-    delete[] d_Prompt_buffer;
-    multicorrelator_cpu.free();
+	if (d_dump_file.is_open())
+	{
+		try
+		{
+			d_dump_file.close();
+		}catch(const std::exception & ex)
+		{
+			LOG(WARNING)<<"Exception in destructor "<<ex.what();
+		}
+	}
+	try{
+		volk_gnsssdr_free(d_local_code_shift_chips);
+		volk_gnsssdr_free(d_correlator_outs);
+		volk_gnsssdr_free(d_ca_code);
+		delete[] d_Prompt_buffer;
+		multicorrelator_cpu.free();
+	}catch(const std::exception & ex)
+	{
+		LOG(WARNING)<<"Exception in destructor "<<ex.what();
+	}
 }
 
 
diff --git a/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_dll_pll_tracking_gpu_cc.cc b/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_dll_pll_tracking_gpu_cc.cc
index 437ac23dd..4f9036fd6 100644
--- a/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_dll_pll_tracking_gpu_cc.cc
+++ b/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_dll_pll_tracking_gpu_cc.cc
@@ -273,14 +273,29 @@ void Gps_L1_Ca_Dll_Pll_Tracking_GPU_cc::start_tracking()
 Gps_L1_Ca_Dll_Pll_Tracking_GPU_cc::~Gps_L1_Ca_Dll_Pll_Tracking_GPU_cc()
 {
 
-    d_dump_file.close();
-    cudaFreeHost(in_gpu);
-    cudaFreeHost(d_correlator_outs);
-    cudaFreeHost(d_local_code_shift_chips);
-    cudaFreeHost(d_ca_code);
-    multicorrelator_gpu->free_cuda();
-    delete[] d_Prompt_buffer;
-    delete(multicorrelator_gpu);
+	if (d_dump_file.is_open())
+	{
+		try
+		{
+			d_dump_file.close();
+		}catch(const std::exception & ex)
+		{
+			LOG(WARNING)<<"Exception in destructor "<<ex.what();
+		}
+	}
+	try{
+	    cudaFreeHost(in_gpu);
+	    cudaFreeHost(d_correlator_outs);
+	    cudaFreeHost(d_local_code_shift_chips);
+	    cudaFreeHost(d_ca_code);
+		delete[] d_Prompt_buffer;
+	    multicorrelator_gpu->free_cuda();
+	    delete(multicorrelator_gpu);
+	}catch(const std::exception & ex)
+	{
+		LOG(WARNING)<<"Exception in destructor "<<ex.what();
+	}
+
 }
 
 
diff --git a/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_tcp_connector_tracking_cc.cc b/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_tcp_connector_tracking_cc.cc
index 776d9e8f8..298d31577 100644
--- a/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_tcp_connector_tracking_cc.cc
+++ b/src/algorithms/tracking/gnuradio_blocks/gps_l1_ca_tcp_connector_tracking_cc.cc
@@ -276,15 +276,27 @@ void Gps_L1_Ca_Tcp_Connector_Tracking_cc::start_tracking()
 
 Gps_L1_Ca_Tcp_Connector_Tracking_cc::~Gps_L1_Ca_Tcp_Connector_Tracking_cc()
 {
-    d_dump_file.close();
-
-    delete[] d_Prompt_buffer;
-    volk_gnsssdr_free(d_ca_code);
-    volk_gnsssdr_free(d_local_code_shift_chips);
-    volk_gnsssdr_free(d_correlator_outs);
-
-    d_tcp_com.close_tcp_connection(d_port);
-    multicorrelator_cpu.free();
+	if (d_dump_file.is_open())
+	{
+		try
+		{
+			d_dump_file.close();
+		}catch(const std::exception & ex)
+		{
+			LOG(WARNING)<<"Exception in destructor "<<ex.what();
+		}
+	}
+	try{
+		volk_gnsssdr_free(d_local_code_shift_chips);
+		volk_gnsssdr_free(d_correlator_outs);
+		volk_gnsssdr_free(d_ca_code);
+	    d_tcp_com.close_tcp_connection(d_port);
+		delete[] d_Prompt_buffer;
+		multicorrelator_cpu.free();
+	}catch(const std::exception & ex)
+	{
+		LOG(WARNING)<<"Exception in destructor "<<ex.what();
+	}
 }
 
 int Gps_L1_Ca_Tcp_Connector_Tracking_cc::general_work (int noutput_items __attribute__((unused)), gr_vector_int &ninput_items __attribute__((unused)),
diff --git a/src/algorithms/tracking/gnuradio_blocks/gps_l2_m_dll_pll_tracking_cc.cc b/src/algorithms/tracking/gnuradio_blocks/gps_l2_m_dll_pll_tracking_cc.cc
index f6f6c3ed8..0e5fc3bc8 100644
--- a/src/algorithms/tracking/gnuradio_blocks/gps_l2_m_dll_pll_tracking_cc.cc
+++ b/src/algorithms/tracking/gnuradio_blocks/gps_l2_m_dll_pll_tracking_cc.cc
@@ -271,14 +271,27 @@ void gps_l2_m_dll_pll_tracking_cc::start_tracking()
 
 gps_l2_m_dll_pll_tracking_cc::~gps_l2_m_dll_pll_tracking_cc()
 {
-    d_dump_file.close();
 
-    volk_gnsssdr_free(d_local_code_shift_chips);
-    volk_gnsssdr_free(d_correlator_outs);
-    volk_gnsssdr_free(d_ca_code);
-
-    delete[] d_Prompt_buffer;
-    multicorrelator_cpu.free();
+	if (d_dump_file.is_open())
+	{
+		try
+		{
+			d_dump_file.close();
+		}catch(const std::exception & ex)
+		{
+			LOG(WARNING)<<"Exception in destructor "<<ex.what();
+		}
+	}
+	try{
+		volk_gnsssdr_free(d_local_code_shift_chips);
+		volk_gnsssdr_free(d_correlator_outs);
+		volk_gnsssdr_free(d_ca_code);
+		delete[] d_Prompt_buffer;
+		multicorrelator_cpu.free();
+	}catch(const std::exception & ex)
+	{
+		LOG(WARNING)<<"Exception in destructor "<<ex.what();
+	}
 }