Bug 8783

Summary: Сохраняемые письма имеют неверные права
Product: Sisyphus Reporter: Vitaly Lipatov <lav>
Component: kdepim-kmailAssignee: rom_as <rom_as>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: normal    
Priority: P2 CC: stanv
Version: unstable   
Hardware: all   
OS: Linux   

Description Vitaly Lipatov 2006-01-02 18:52:11 MSK
Сохранение вложений отрабатывает нормально (включая следование SGID),  
но при сохранении самого письма вне зависимости от umask 
нет прав у группы и других на его чтение. 
$ ls -l some_file 
-rw------- 1 lav uucp ... 
 
$ rpmqf kmail  
kdepim-kmail-3.5.0-alt1
Comment 1 Vitaly Lipatov 2007-01-06 16:54:31 MSK
reconfirmed for
$ rpmqf kmail
kdepim-kmail-3.5.5-alt2
Comment 2 Andriy Stepanov 2007-06-07 13:27:16 MSD
Это не ошибка, а задумано авторами kmail

Вот участок кода из: kdepim-3.5.7/kmail/kmcommands.cpp

KMCommand::Result KMSaveMsgCommand::execute()
{
  mJob = KIO::put( mUrl, S_IRUSR|S_IWUSR, false, false );

Тут можно предложить:

1. закоментировать эту строку чтоб работали установки umask (для этого должны
быть очень весомые аргументы)

2. ввести опцию чтоб пользователь мог сам в своем ~/.kde/share/config/kmailrc
указать параметр (true/false) (но по умолчанию всеравно оставить false)
Comment 3 Vitaly Lipatov 2007-06-08 09:13:24 MSD
(In reply to comment #2)
> Это не ошибка, а задумано авторами kmail
Ну почему, они вполне могли ошибочно задумать. В конце концов любая ошибка 
программы реализована в коде :)

> 
> Вот участок кода из: kdepim-3.5.7/kmail/kmcommands.cpp
> 
> KMCommand::Result KMSaveMsgCommand::execute()
> {
>   mJob = KIO::put( mUrl, S_IRUSR|S_IWUSR, false, false );
> 
> Тут можно предложить:
> 
> 1. закоментировать эту строку чтоб работали установки umask (для этого 
должны
> быть очень весомые аргументы)
Лучше предложите весомые аргументы не использовать umask.

> 
> 2. ввести опцию чтоб пользователь мог сам в своем 
~/.kde/share/config/kmailrc
> указать параметр (true/false) (но по умолчанию всеравно оставить false)
Я думаю, умолчание по правам создания должно задаваться в одном месте, и это 
место - umask.

Comment 4 Sergey V Turchin 2007-06-08 13:31:01 MSD
Я в новом KMail уже видел такое исправление, надо найти и сделать сдесь тоже в 
зависимости от того же параметра в настройках
Comment 5 Sergey V Turchin 2007-06-08 14:29:06 MSD
(In reply to comment #3)
> Лучше предложите весомые аргументы не использовать umask.
Я не знаю, зачем. Но абсолютно точно это неспроста. В другом месте в KMail 
похожее исправили, добавив опцию для старого поведения, отключенную по 
умолчанию.
Comment 6 Sergey V Turchin 2007-06-09 19:01:44 MSD
А в остальных 2-х местах в этом файле не осилили :-(
Comment 7 Sergey V Turchin 2007-06-15 14:33:49 MSD
kdepim-kmail-3.5.7-alt2