Fix memory leaks
authorJonathan Wakely <github@kayari.org>
Thu, 18 Feb 2021 17:10:07 +0000 (17:10 +0000)
committerMichael McMaster <michael@codesrc.com>
Mon, 26 Apr 2021 10:59:37 +0000 (20:59 +1000)
Use std::unique_ptr for dynamically-allocated objects.

src/scsi2sd-util6/BoardPanel.hh
src/scsi2sd-util6/TargetPanel.hh

index a21680cdead96099d470111febe677a870e6096b..5969f7396ff57e01cf2295c1fad175daacdb7454 100644 (file)
@@ -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;
 
index afaaae228dc7774d5012228e7b06c8a5f123e64e..f26aa323aebd0edb5ad0e0189db3d28ef2e27d5e 100644 (file)
@@ -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;