diff options
Diffstat (limited to 'modules/spamassassin/evolution-spamassassin.c')
| -rw-r--r-- | modules/spamassassin/evolution-spamassassin.c | 22 |
1 files changed, 13 insertions, 9 deletions
diff --git a/modules/spamassassin/evolution-spamassassin.c b/modules/spamassassin/evolution-spamassassin.c index 4adb1e8cc3..480fedccd4 100644 --- a/modules/spamassassin/evolution-spamassassin.c +++ b/modules/spamassassin/evolution-spamassassin.c @@ -786,14 +786,14 @@ spam_assassin_new_config_widget (EMailJunkFilter *junk_filter) return box; } -static gboolean +static CamelJunkStatus spam_assassin_classify (CamelJunkFilter *junk_filter, CamelMimeMessage *message, - CamelJunkStatus *status, GCancellable *cancellable, GError **error) { ESpamAssassin *extension = E_SPAM_ASSASSIN (junk_filter); + CamelJunkStatus status; const gchar *argv[7]; gint exit_code; gint ii = 0; @@ -833,32 +833,36 @@ spam_assassin_classify (CamelJunkFilter *junk_filter, exit_code = spam_assassin_command ( argv, message, NULL, cancellable, error); + /* Check for an error while spawning the program. */ + if (exit_code == SPAM_ASSASSIN_EXIT_STATUS_ERROR) + status = CAMEL_JUNK_STATUS_ERROR; + /* For either program, exit code 0 means the message is ham. */ - if (exit_code == 0) - *status = CAMEL_JUNK_STATUS_MESSAGE_IS_NOT_JUNK; + else if (exit_code == 0) + status = CAMEL_JUNK_STATUS_MESSAGE_IS_NOT_JUNK; /* spamassassin(1) only specifies zero and non-zero exit codes. */ else if (!extension->use_spamc) - *status = CAMEL_JUNK_STATUS_MESSAGE_IS_JUNK; + status = CAMEL_JUNK_STATUS_MESSAGE_IS_JUNK; /* Whereas spamc(1) explicitly states exit code 1 means spam. */ else if (exit_code == 1) - *status = CAMEL_JUNK_STATUS_MESSAGE_IS_JUNK; + status = CAMEL_JUNK_STATUS_MESSAGE_IS_JUNK; /* Consider any other spamc(1) exit code to be inconclusive * since it most likely failed to process the message. */ else - *status = CAMEL_JUNK_STATUS_INCONCLUSIVE; + status = CAMEL_JUNK_STATUS_INCONCLUSIVE; /* Check that the return value and GError agree. */ - if (exit_code != SPAM_ASSASSIN_EXIT_STATUS_ERROR) + if (status != CAMEL_JUNK_STATUS_ERROR) g_warn_if_fail (error == NULL || *error == NULL); else g_warn_if_fail (error == NULL || *error != NULL); g_mutex_unlock (extension->socket_path_mutex); - return (exit_code != SPAM_ASSASSIN_EXIT_STATUS_ERROR); + return status; } static gboolean |