diff --git a/server/mailserver/mail/mailmgr.go b/server/mailserver/mail/mailmgr.go
index 7c5fa725..8e886592 100644
--- a/server/mailserver/mail/mailmgr.go
+++ b/server/mailserver/mail/mailmgr.go
@@ -170,7 +170,7 @@ func (this *mailMgr) CaGetAttachment(c *gin.Context) {
 	mails := []common.Mail{}
 	for _, str := range(mailIds) {
 		m := this.getMail(str)
-		if m != nil {
+		if m != nil && m.IsValid(hum) {
 			q5.AppendSlice(&mails, m)
 		}
 	}
diff --git a/server/mailserver/player/player.go b/server/mailserver/player/player.go
index d9308137..a9a8608e 100644
--- a/server/mailserver/player/player.go
+++ b/server/mailserver/player/player.go
@@ -85,9 +85,7 @@ func (this *player) MarkMails(mails []common.Mail) error {
 }
 
 func (this *player) GetAttachment(mails []common.Mail) error {
-	this.checkLock()
-	var resultErr error
-	return resultErr
+	return this.DeleteMails(mails)
 }
 
 func (this *player) DeleteMails(mails []common.Mail) error {