After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 678347 - Put spamd pid/socket files in $XDG_RUNTIME_DIR
Put spamd pid/socket files in $XDG_RUNTIME_DIR
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: general
3.2.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: Evolution Shell Maintainers Team
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2012-06-18 17:58 UTC by Marc Deslauriers
Modified: 2012-06-20 03:27 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Marc Deslauriers 2012-06-18 17:58:39 UTC
After a while, evolution stops using spamassassin for junk mail detection.

Here is what happens:

1- Evolution creates spamd socket and pid file in ~/.cache/evolution/tmp with e_mktemp()
2- When creating a new temporary directory to open an email attachment, evolution notices that the socket and pid file are more than 2 hours old, and deletes them, breaking spamassassin integration

See TEMP_EXPIRE in e-util/e-mktemp.c.

The problem is that it's using st.st_atime, but that's no longer getting updated on Ubuntu and/or with ext4.

This is my directory before opening an attachment:
mdeslaur@mdlinux:~/.cache/evolution/tmp$ ls -al
total 24
drwxr-xr-x 5 mdeslaur mdeslaur 4096 Jun 18 12:49 .
drwx------ 6 mdeslaur mdeslaur 4096 Apr 10 2011 ..
drwx------ 2 mdeslaur mdeslaur 4096 Jun 18 12:49 evolution-mdeslaur-PlMVtk
drwx------ 2 mdeslaur mdeslaur 4096 Jun 18 10:40 evolution-mdeslaur-VMPrqC
drwx------ 2 mdeslaur mdeslaur 4096 Jun 18 11:14 evolution-mdeslaur-wrcl0C
-rw-rw-r-- 1 mdeslaur mdeslaur 5 Jun 18 11:13 spamd-pid-file-Y2CYFW
srw-rw-rw- 1 mdeslaur mdeslaur 0 Jun 18 11:13 spamd-socket-path-UZCYFW

This is after opening an attachment at 13:43:
mdeslaur@mdlinux:~/.cache/evolution/tmp$ ls -al
total 16
drwxr-xr-x 4 mdeslaur mdeslaur 4096 Jun 18 13:43 .
drwx------ 6 mdeslaur mdeslaur 4096 Apr 10 2011 ..
drwx------ 2 mdeslaur mdeslaur 4096 Jun 18 12:49 evolution-mdeslaur-PlMVtk
drwx------ 2 mdeslaur mdeslaur 4096 Jun 18 13:43 evolution-mdeslaur-ws2Jh2

See downstream bug report:

https://bugs.launchpad.net/ubuntu/+source/evolution/+bug/1014763
Comment 1 Matthew Barnes 2012-06-18 18:24:03 UTC
Hmm, I guess we should be using XDG_RUNTIME_DIR nowadays.
Comment 2 Matthew Barnes 2012-06-20 03:27:43 UTC
Fixed for Evolution 3.5.3 in:
http://git.gnome.org/browse/evolution/commit/?id=782d59da7b614ba04fe89411dde8b482fd3d6204