Parcourir la source

Fix memory leaks

Use std::unique_ptr for dynamically-allocated objects.
Jonathan Wakely il y a 4 ans
Parent
commit
0787a60978

+ 2 - 1
lib/SCSI2SD/src/scsi2sd-util6/BoardPanel.hh

@@ -35,6 +35,7 @@
 #endif
 
 #include <utility>
+#include <memory>
 
 namespace SCSI2SD
 {
@@ -76,7 +77,7 @@ private:
 	wxCheckBox* myMapLunsCtrl;
 	wxCheckBox* myBlindWriteCtrl;
 
-	wxIntegerValidator<uint8_t>* myDelayValidator;
+	std::unique_ptr<wxIntegerValidator<uint8_t>> myDelayValidator;
 	wxTextCtrl* myStartDelayCtrl;
 	wxTextCtrl* mySelDelayCtrl;
 

+ 5 - 4
lib/SCSI2SD/src/scsi2sd-util6/TargetPanel.hh

@@ -35,6 +35,7 @@
 #endif
 
 #include <utility>
+#include <memory>
 
 namespace SCSI2SD
 {
@@ -108,20 +109,20 @@ private:
 
 	wxChoice* myDeviceTypeCtrl;
 
-	wxIntegerValidator<uint32_t>* myStartSDSectorValidator;
+	std::unique_ptr<wxIntegerValidator<uint32_t>> myStartSDSectorValidator;
 	wxTextCtrl* myStartSDSectorCtrl;
 	wxCheckBox* myAutoStartSectorCtrl;
 	wxStaticText* myStartSDSectorMsg;
 
-	wxIntegerValidator<uint16_t>* mySectorSizeValidator;
+	std::unique_ptr<wxIntegerValidator<uint16_t>> mySectorSizeValidator;
 	wxTextCtrl* mySectorSizeCtrl;
 	wxStaticText* mySectorSizeMsg;
 
-	wxIntegerValidator<uint32_t>* myNumSectorValidator;
+	std::unique_ptr<wxIntegerValidator<uint32_t>> myNumSectorValidator;
 	wxTextCtrl* myNumSectorCtrl;
 	wxStaticText* myNumSectorMsg;
 
-	wxFloatingPointValidator<float>* mySizeValidator;
+	std::unique_ptr<wxFloatingPointValidator<float>> mySizeValidator;
 	wxTextCtrl* mySizeCtrl;
 	wxChoice* mySizeUnitCtrl;