Lacking a key to verify the signature is not "bad".

1 files changed, 4 insertions, 4 deletions

diff --git a/framework/src/domain/mime/partmodel.cpp b/framework/src/domain/mime/partmodel.cpp
index bbafbc7c..ca8856fd 100644
--- a/framework/src/domain/mime/partmodel.cpp
+++ b/framework/src/domain/mime/partmodel.cpp
@@ -267,12 +267,12 @@ QVariant PartModel::data(const QModelIndex &index, int role) const
 
                 if (messageIsSigned) {
                     auto sigInfo = std::unique_ptr<SignatureInfo>{signatureInfo(messagePart)};
-                    if (!sigInfo->signatureIsGood || sigInfo->keyRevoked) {
+                    if (!sigInfo->signatureIsGood) {
+                        if (sigInfo->keyMissing || sigInfo->keyExpired) {
+                            return "notsogood";
+                        }
                         return "bad";
                     }
-                    if (sigInfo->keyMissing || sigInfo->keyExpired) {
-                        return "notsogood";
-                    }
                 }
                 //All good
                 if (messageIsSigned || messageIsEncrypted) {