Fixed an issue where message object was disposed before message was sent.

Author: ravibpatel

CrashReporter.NET/HelperMethods.cs

@@ -44,7 +44,7 @@ private static string HKLM_GetString(string key, string value)
             try
             {
                 RegistryKey registryKey = Registry.LocalMachine.OpenSubKey(key);
-                return registryKey?.GetValue(value).ToString() ?? String.Empty;
+                return registryKey?.GetValue(value)?.ToString() ?? String.Empty;
             }
             catch
             {

CrashReporter.NET/ReportCrash.cs

@@ -339,33 +339,33 @@ private void SendEmail(bool includeScreenshot, SendCompletedEventHandler smtpCli
                 EnableSsl = EnableSSL,
                 DeliveryMethod = SmtpDeliveryMethod.Network,
                 UseDefaultCredentials = false,
-                Credentials = new NetworkCredential(UserName, Password),
+                Credentials = new NetworkCredential(UserName, Password)
             };
+
+            var message = new MailMessage(new MailAddress(FromEmail), new MailAddress(ToEmail))
+                {IsBodyHtml = true, Subject = subject, Body = CreateHtmlReport(userMessage)};
 
-            using (var message = new MailMessage(new MailAddress(FromEmail), new MailAddress(ToEmail)) {IsBodyHtml = true, Subject = subject, Body = CreateHtmlReport(userMessage)})
+            if (ScreenShotBinary?.Length > 0 && includeScreenshot)
             {
-                if (ScreenShotBinary?.Length > 0 && includeScreenshot)
-                {
-                    message.Attachments.Add(new Attachment(new MemoryStream(ScreenShotBinary), "Screenshot.png", "image/png"));
-                }
+                message.Attachments.Add(new Attachment(new MemoryStream(ScreenShotBinary), "Screenshot.png", "image/png"));
+            }
 
-                if (smtpClientSendCompleted != null)
+            if (smtpClientSendCompleted != null)
+            {
+                try
                 {
-                    try
-                    {
-                        smtpClient.SendCompleted += smtpClientSendCompleted;
-                        smtpClient.SendAsync(message, "Crash Report");
-                    }
-                    catch (SmtpException smtpException)
-                    {
-                        smtpClientSendCompleted(this, new System.ComponentModel.AsyncCompletedEventArgs(smtpException, true, null));
-                    }
+                    smtpClient.SendCompleted += smtpClientSendCompleted;
+                    smtpClient.SendAsync(message, "Crash Report");
                 }
-                else
+                catch (SmtpException smtpException)
                 {
-                    smtpClient.Send(message);
+                    smtpClientSendCompleted(this, new System.ComponentModel.AsyncCompletedEventArgs(smtpException, true, null));
                 }
             }
+            else
+            {
+                smtpClient.Send(message);
+            }
         }
 
         #endregion