Go语言SMTP邮箱校验程序触发SIGSEGV空指针panic问题排查

从你的报错栈和代码来看，问题出在SMTP客户端对象的空指针调用上，我来一步步帮你排查修复：

核心问题根源 #

1. smtp.Dial失败后仍调用client.Close()
   当smtp.Dial无法连接到目标SMTP服务器时，会返回错误，此时client变量是nil。但你在调用smtp.Dial后立刻写了defer client.Close()，函数退出时就会对空指针执行Close方法，直接触发SIGSEGV段错误。

2. 忽略MX记录解析错误，直接访问mx[0].Host
   如果域名的MX记录解析失败（比如域名不存在），net.LookupMX返回的mx切片是nil，此时访问mx[0].Host也会触发panic，这个隐藏风险必须修复。

3. 错误处理不完整，报错后仍执行后续代码
   比如client.Hello、client.Mail等步骤出错时，你只是打印错误但没有提前返回，后续代码依然会尝试执行其他SMTP命令，会引发更多未知问题。

修复后的代码实现 #

修复validateHost函数 #

func validateHost(email string) error {
	_, host := split(email)
	mx, err := net.LookupMX(host)
	if err != nil {
		fmt.Printf("Error, UnresolvableHost! %s\n", err)
		return err // 解析失败直接返回，终止后续逻辑
	}
	if len(mx) == 0 {
		fmt.Println("No MX records found for host")
		return fmt.Errorf("no MX records for %s", host)
	}

	client, err := smtp.Dial(fmt.Sprintf("%s:%d", mx[0].Host, 25))
	if err != nil {
		fmt.Printf("SmtpError! %s \n", err)
		return err // 连接失败直接返回，此时client为nil，不会执行后续defer
	}
	// 仅当client非nil时才延迟关闭，避免空指针调用
	defer func() {
		if closeErr := client.Close(); closeErr != nil {
			fmt.Printf("Error closing SMTP client: %s\n", closeErr)
		}
	}()

	t := time.AfterFunc(forceDisconnectAfter, func() {
		// 超时关闭前先判断client是否有效
		if client != nil {
			_ = client.Close()
		}
	})
	defer t.Stop()

	err = client.Hello("checkmail.me")
	if err != nil {
		fmt.Printf("client.Hello SmtpError! %s \n", err)
		return err
	}

	err = client.Mail("lansome-cowboy@gmail.com")
	if err != nil {
		fmt.Printf("client.Mail SmtpError! %s \n", err)
		return err
	}

	err = client.Rcpt(email)
	if err != nil {
		fmt.Printf("client.Rcpt SmtpError! %s \n", err)
		return err
	}

	return nil
}

优化CheckMails的错误判断逻辑 #

原来的错误分支逻辑有点混乱，调整后更清晰：

func CheckMails(mails []string) []string {
	var existingMails []string
	if len(mails) < 2 {
		fmt.Println("Mail slice is too short")
		return existingMails
	}
	fmt.Printf("!!!!!!!!!!!!!!STARTING!!!!!!!!!!!! %s \n\n\n", mails[1])
	for _, mail := range mails { // 用range遍历更安全，避免索引越界
		err := validateHost(mail)
		if err != nil {
			fmt.Printf("Error validating %s: %s\n", mail, err)
			if smtpErr, ok := err.(checkmail.SmtpError); ok {
				fmt.Printf("SMTP Code: %s, Msg: %s\n", smtpErr.Code(), smtpErr)
				if smtpErr.Code() == "dia" {
					break
				}
			}
			continue // 出错直接跳过当前邮箱
		}
		// 没有错误说明邮箱存在
		fmt.Printf("Email %s exists\n", mail)
		existingMails = append(existingMails, mail)
	}
	fmt.Printf("!!!!!!!!!!!!!!ENDING!!!!!!!!!!!! %s \n\n\n", mails[1])
	return existingMails
}

额外优化建议 #

• 你的CheckMailsWithExpectedInterval超时逻辑只是放弃等待goroutine，但goroutine会继续运行，可能导致资源泄漏（比如未关闭的SMTP连接），建议改用context.Context来实现goroutine的取消逻辑。
• 频繁的SMTP验证请求容易被目标服务器拉黑，建议添加请求间隔控制，避免短时间内大量请求。

内容的提问来源于stack exchange，提问作者WannaBeBetter