Dispose of folder browser dialog correctly

SteamRE · Oct 20, 2019 · d959b04 · d959b04
1 parent 7b71ac1
commit d959b04
Showing 1 changed file with 17 additions and 11 deletions.
diff --git a/Resources/NetHookAnalyzer2/NetHookAnalyzer2/MainForm.cs b/Resources/NetHookAnalyzer2/NetHookAnalyzer2/MainForm.cs
@@ -152,21 +152,27 @@ void OnExitToolStripMenuItemClick(object sender, EventArgs e)
 			Application.Exit();
 		}
 
-		void OnOpenToolStripMenuItemClick(object sender, EventArgs e)
+		void OnOpenToolStripMenuItemClick( object sender, EventArgs e )
 		{
-			var dialog = new FolderBrowserDialog { ShowNewFolderButton = false };
-			var latestNethookDir = GetLatestNethookDumpDirectory();
-			if (latestNethookDir != null)
-			{
-				dialog.SelectedPath = GetLatestNethookDumpDirectory();
-			}
+			string dumpDirectory;
 
-			if (dialog.ShowDialog() != WinForms.DialogResult.OK)
+			using ( var dialog = new FolderBrowserDialog() )
 			{
-				return;
-			}
+				dialog.ShowNewFolderButton = false;
 
-			var dumpDirectory = dialog.SelectedPath;
+				var latestNethookDir = GetLatestNethookDumpDirectory();
+				if ( latestNethookDir != null )
+				{
+					dialog.SelectedPath = GetLatestNethookDumpDirectory();
+				}
+
+				if ( dialog.ShowDialog() != WinForms.DialogResult.OK )
+				{
+					return;
+				}
+
+				dumpDirectory = dialog.SelectedPath;
+			}
 
 			var dump = new NetHookDump();
 			dump.LoadFromDirectory(dumpDirectory);