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 581604 - Permissions on mail/local folders are too open
Permissions on mail/local folders are too open
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Mailer
2.26.x (obsolete)
Other All
: Normal normal
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2009-05-06 15:18 UTC by Yves-Alexis Perez
Modified: 2010-03-25 10:01 UTC
See Also:
GNOME target: 2.30.x
GNOME version: 2.25/2.26


Attachments
evo part of the patch (6.62 KB, patch)
2010-01-21 12:04 UTC, Chenthill P
committed Details | Review
eds part of the changes (6.79 KB, patch)
2010-01-21 12:04 UTC, Chenthill P
committed Details | Review

Description Yves-Alexis Perez 2009-05-06 15:18:51 UTC
Please describe the problem:
Hi,

it seems that the default permissions on the local mail folders are set to 755, which means that, by default, other users on the systems might be able to read one's mail.

At least imap mail folder is 700, but local, config, vfolder and views are 755, so it'd be nice to have a consistent scheme. Same would apply to .evolution and .evolution/{mail,addressbook,calendar,…}

Steps to reproduce:
1. 
2. 
3. 


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 2 Matthew Barnes 2009-05-29 12:05:06 UTC
As I stated in the Red Hat bug, I don't think this warrants a CVE.

It's only an issue if you open up your home directory to other users, and in doing so you've already lost any security or privacy guarantees.

In any case, the best we can do for existing installs is reset the ~/.evolution permissions to 700.
Comment 3 Yves-Alexis Perez 2009-10-29 07:33:04 UTC
That and for the various subfolders, I guess it's not really costly. What's the status on this one? on 2.28 it seems the existing permissions aren't reset (not sure about the new ones)
Comment 4 Chenthill P 2010-01-21 12:04:16 UTC
Created attachment 151934 [details] [review]
evo part of the patch

If the permissions for ~/.evolution is open, it changes the permissions for all internal folders else skip.
Comment 5 Chenthill P 2010-01-21 12:04:42 UTC
Created attachment 151935 [details] [review]
eds part of the changes
Comment 6 Matthew Barnes 2010-01-21 17:49:21 UTC
Thanks for taking care of this, Chen!  Both patches look good.
Comment 7 Chenthill P 2010-01-29 20:23:34 UTC
thanks matt :) pushed to master. will push it to stable branch as well.
Comment 8 Yves-Alexis Perez 2010-02-01 17:44:40 UTC
Does this means it'll be in 2.30?
Comment 9 Chenthill P 2010-02-05 06:45:16 UTC
YVes, yes.
Comment 10 Bharath Acharya 2010-03-25 10:01:08 UTC
Fixed as per comment #7 No use pushing it to 2.28.x Works good on 2.30.x Closing