Bug 8783

Summary:	Сохраняемые письма имеют неверные права
Product:	Sisyphus	Reporter:	Vitaly Lipatov <lav>
Component:	kdepim-kmail	Assignee:	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