Failed spamassassin pipe on Evolution (Intrepid)

Bug #275746 reported by datanalytics.com
74
This bug affects 11 people
Affects Status Importance Assigned to Milestone
Evolution
Fix Released
Low
evolution (Ubuntu)
Fix Released
Low
Ubuntu Desktop Bugs
Intrepid
Fix Released
Low
Ubuntu Desktop Bugs

Bug Description

Binary package hint: evolution

I am using Evolution after an upgrade to Ubuntu 8.10. I have activated the spamfilter (spamassassin).

My incoming mail is being filtered. However, each time a new message is downloaded, a warning message appears on the lower status bar. If I click on it, an error window pops up and says that the pipe to spamassassin failed (and exited with error code 5).

The filter is applied. It works. It is only an annoyance...

ProblemType: Bug
Architecture: amd64
DistroRelease: Ubuntu 8.10
ExecutablePath: /usr/bin/evolution
NonfreeKernelModules: nvidia
Package: evolution 2.24.0-0ubuntu1
ProcEnviron:
 SHELL=/bin/bash
 PATH=/home/username/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games
 LANG=en_US.UTF-8
SourcePackage: evolution
Uname: Linux 2.6.27-4-generic x86_64

Related branches

Revision history for this message
datanalytics.com (cgb-datanalytics) wrote :
Revision history for this message
Pedro Villavicencio (pedro) wrote :

what's the message? i don't get it here, may you try to reproduce this with a new user created on your system and with the latest update of evolution? please also take an screenshot of the message you're seeing, thanks.

Changed in evolution:
assignee: nobody → desktop-bugs
importance: Undecided → Low
status: New → Incomplete
Revision history for this message
Bolderbast (bolderbast-duckstad) wrote :

Hi,

I have the same issue. Downloaded the beta Alternate CD on the 23rd of october and installed fresh on a new laptop. Updated that evening and started Evolution. Imported my messages and settings from my Hardy laptop (using the import wizard) and got the same error when fetching mail (displayed in the statusbar). "Screenshot-Evolution Spamassassin error.png" is a screenshot of the error message.

The preferences of Evolution confirm that Spamassassin is installed. (see "Screenshot-Evolution Preferences.png"). Spamassassin is running, confirmed by the following ps output:

root 20148 8.9 0.7 31476 27700 ? Ss 17:58 0:00 /usr/sbin/spamd --create-prefs --max-children 5 --helper-home-dir -d --pidfile=/var/run/spamd.pid
root 20157 0.0 0.7 31476 25736 ? S 17:58 0:00 spamd child
root 20158 0.0 0.7 31476 25736 ? S 17:58 0:00 spamd child

There are no errors displayed regarding to spamassassin when Evolution is run from a terminal. If there is anything else I can do to help, please let me know!

Revision history for this message
Graziano (graziano-giuliani-gmail) wrote :

I have tried with and without spamd, and the behaviour is the same.

Revision history for this message
Derek White (d-man97) wrote :

I can confirm this on a fresh install of 8.10. Ran evolution, imported my old data/settings, exited, installed spamassassin and spamc, ran evolution, activated spamassassin in Junk settings, error was present immediately, still there after restarting evolution and the computer.

https://bugs.launchpad.net/ubuntu/+source/evolution/+bug/281807 -- Comments 11 & 12

Revision history for this message
Hein-PietervanBraam (hp) wrote :

I have looked into this bug, and I have found the following problem:

plugins/sa-junk-plugin/em-junk-filter.c starts spamc through pipe_to_sa_full(). This function starts the various client programs and checks the exit status of the command that was run.

This last part is the problem : on line 253 a check for the exit code of either spamc or spamassassin is done and an error is returned to evolution (which results in the popup).

The interesting thing about this is that the rest of the plugin uses a non-zero exit status to denote spam, but the pipe_to_sa_full() funtion returns an error when the exit status is non-zero. So, every spam message gets processed as it should be, but pipe_to_sa_full() considers it an error.

Spamassassin and spamc use error code '64' to denote abnormal program termination. pipe_to_sa_full() should thus check for '64' or '70' in the case of spamassassin, and check for >1 in the case of spamc before throwing an error.

Attached you will find a patch that changes the error to only appear if the errorcode is >64.

Revision history for this message
Hein-PietervanBraam (hp) wrote :

Confirmed the problem, attached patch to fix.

Changed in evolution:
status: Incomplete → Confirmed
Hein-PietervanBraam (hp)
Changed in evolution:
status: New → Confirmed
Revision history for this message
Bolderbast (bolderbast-duckstad) wrote :

Applied patch (with help) and the error is gone now. Thanks!

Revision history for this message
Hein-PietervanBraam (hp) wrote :

I made an error in my last comment about spamc, the way evolution calls spamc (-c) also denotes errors with exitcodes of 64 or larger, not just bigger than 1.

I adjusted the patch, but I forgot to update my comment. My apologies for the bugspam

Revision history for this message
Pedro Villavicencio (pedro) wrote :

Thanks for taking this report upstream, just remember to create a bug watch when you do it, information about how to do that is available here: https://wiki.ubuntu.com/Bugs/Watches ; thanks again.

Changed in evolution:
status: Confirmed → Triaged
importance: Undecided → Unknown
status: Confirmed → Unknown
Changed in evolution:
status: Unknown → Fix Released
Revision history for this message
Laurent Bonnaud (laurent-bonnaud) wrote :

This bug has been fixed upstream. How about uploading an updated package to intrepid-update ?

Revision history for this message
Sebastien Bacher (seb128) wrote :

the intrepid version will be updated when upstream roll new tarballs for stable

Changed in evolution:
status: Triaged → Fix Committed
Revision history for this message
Sebastien Bacher (seb128) wrote :

upstream rolled a new stable version today which has been uploaded as a candidate update to intrepid now

Revision history for this message
Steve Langasek (vorlon) wrote :

Accepted into intrepid-proposed, please test and give feedback here. Please see https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in evolution:
status: New → Fix Committed
Revision history for this message
Laurent Bonnaud (laurent-bonnaud) wrote :
Download full text (3.4 KiB)

Currently intrepid-proposed offers a mixture of versions 2.24.2 and 2.24.3. Here is what I have on my system:

$ dpkg -l | grep evolution
ii beagle-backend-evolution 0.3.8-1ubuntu2 evolution data backend for beagle
ii evolution 2.24.2-0ubuntu1 groupware suite with mail client and organizer
ii evolution-common 2.24.2-0ubuntu1 architecture independent files for Evolution
ii evolution-data-server 2.24.3-0ubuntu1 evolution database backend server
ii evolution-data-server-common 2.24.3-0ubuntu1 architecture independent files for Evolution Data Server
ii evolution-dbgsym 2.24.2-0ubuntu1 debug symbols for package evolution
ii evolution-exchange 2.24.2-0ubuntu1 Exchange plugin for the Evolution groupware suite
ii evolution-plugins 2.24.2-0ubuntu1 standard plugins for Evolution
ii evolution-rss 0.1.0-1ubuntu2 Evolution RSS Reader Plugin
ii evolution-tnef 0.0.0-1 Evolution plugin to decrypt winmail.dat files
ii evolution-webcal 2.24.0-0ubuntu1 webcal: URL handler for GNOME and Evolution
ii libebackend1.2-0 2.24.3-0ubuntu1 Utility library for evolution data servers
ii libebook1.2-9 2.24.3-0ubuntu1 Client library for evolution address books
ii libecal1.2-7 2.24.3-0ubuntu1 Client library for evolution calendars
ii libedata-book1.2-2 2.24.3-0ubuntu1 Backend library for evolution address books
ii libedata-cal1.2-6 2.24.3-0ubuntu1 Backend library for evolution calendars
ii libedataserver1.2-11 2.24.3-0ubuntu1 Utility library for evolution data servers
ii libedataserver1.2-dev 2.24.3-0ubuntu1 Utility library for evolution data servers (development files)
ii libedataserverui1.2-8 2.24.3-0ubuntu1 GUI utility library for evolution data servers
ii libevolution3.0-cil 0.17.5-0ubuntu1 CLI bindings for Evolution
ii libmultisync-plugin-evolution 0.82-8.1ubuntu2 Ximian Evolution plugin for MultiSyn...

Read more...

Revision history for this message
Sebastien Bacher (seb128) wrote :

The new evolution version require the new evolution-data-server binaries to be published to build so there is a small delay between having this one available, it has built on i386 and the binaries should be available soon now

Changed in evolution:
assignee: nobody → desktop-bugs
importance: Undecided → Low
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package evolution - 2.24.3-0ubuntu1

---------------
evolution (2.24.3-0ubuntu1) intrepid-proposed; urgency=low

  * New upstream version:
    Bug Fixes:
    - #332729: Invalid write with outspring from e-config.c
    - #337082: Unnamed imap folder (lp: #37227)
    - #435452: Adding People to Meeting Loses Attendee if Add or
               Busy Search Selected
    - #435694: Retract of Email from Sent Items Does not Work
    - #439733: Checking for junk before training
    - #446285: Meeting Entry Does Not Allow Multiple Names
    - #458968: Don't warn the user about changes being lost if he is trying to
               delegate the meeting
    - #546637: Mail opened from the "Unread mails" displays empty (lp: #275952)
    - #551599: Do not fallback to default 15 minutes alarm offset if alarm is
               triggered at start
    - #552583: Passwords don't get stored when using other authentication
            type than "Password"
    - #552583: Free the right url
    - #552583: Account checking logic a bit more consistent
    - #552583: Fix bugs in the account checking logic
    - #552583: Compare the protocol, user, host and port and disregard the rest
    - #555663: Evolution crashed with SIGSEGV in gconf-bridge (lp: #280675)
    - #556303: Evolution crashed with SIGSEGV in camel_mime_part_get_filename
               (lp: #304385)
    - #557176: Crash while trying to click on 'Contacts'
    - #558337: Evolution crashes while managing IMAP folder subscriptions
    - #559153: Unable to convert mailboxes from Hardy to Intrepid versions
    - #560420: Failed spamassassin pipe on Evolution (lp: #275746)
    - #562091: New share memo window doesn't come up
    - #562155: Evolution stopped to start
    - #562450: Edit as New Message doesn't copy the content of a
               GPG signed message
    - #562886: Evolution mailto CLI cannot handle attachments with
               "#" in filename (lp: #304036)
    - #563077: Evolution crashes if gnome-settings-daemon is not
               started on Solaris
    - #563250: Save and restore custom composer accelerators
    - #563369: "Templates" folder not translated
    - #564007: Messages do not show inline decrypted message by default
    - #564860: Evolution crashes in e_util_labels_parse if the labels don't have
               the correct format
    - #565553: Right click on an attachment in calendar event
               and chose application does not work
    - #566653: Local folder's DnD aren't saved well to DB
  * debian/control:
    - updated evolution-data-server requirement

 -- Sebastien Bacher <email address hidden> Mon, 12 Jan 2009 16:23:51 +0100

Changed in evolution:
status: Fix Committed → Fix Released
Revision history for this message
Sebastien Bacher (seb128) wrote :

the new version is in jaunty now

Changed in evolution:
status: Fix Committed → Fix Released
Changed in evolution:
importance: Unknown → Low
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.