| Summary: | ACL default mask is ignored or mask is changed for image editor and batch queue manager | ||
|---|---|---|---|
| Product: | [Applications] digikam | Reporter: | Thomas Bettler <thomas.bettler> |
| Component: | DImg-FileIO | Assignee: | Digikam Developers <digikam-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | caulier.gilles, marcel.wiesweg, metzpinguin |
| Priority: | NOR | ||
| Version: | 4.2.0 | ||
| Target Milestone: | --- | ||
| Platform: | Gentoo Packages | ||
| OS: | Linux | ||
| URL: | https://bugs.kde.org/show_bug.cgi?id=308373 | ||
| Latest Commit: | Version Fixed In: | 7.5.0 | |
| Sentry Crash Report: | |||
|
Description
Thomas Bettler
2013-11-07 02:37:04 UTC
Thomas, Problem still valid using last digiKam 4.2.0 ? Gilles Caulier Gilles, 1) For batch editing behaviour is still unexpected as described in bug report above. 2) For versioned image editing behaviour has changed and leads to other unexpected results, thus still needs to be fixed: Directory defaults as mentioned above in bug report above. File orig has rwx for several users, while group:--- / others:--- / mask:rw- # file: orig.jpg # owner:t #group:users user::rw- user:r:rwx #effective:rw- user:t:rwx #effective:rw- group::--- mask::rw- other::--- Image editing leads to unexpected results for others:r-- / mask:r-- # file: edited.jpg # owner: t # group: users user::rw- user:r:rwx #effective:r-- user:t:rwx #effective:r-- group::--- mask::r-- other::r-- Expected results would be: New versioned file gets the same ACL as origin file Thomas, For Image editor, ACL are managed through std::umask() in this method : https://projects.kde.org/projects/extragear/graphics/digikam/repository/revisions/master/entry/utilities/imageeditor/editor/editorwindow.cpp#L2552 What's missing here ? Documentation about umask() function can be found here : http://man7.org/linux/man-pages/man2/umask.2.html In BQM, there is no similar code. So, right this is missing. It miss also code certainly in ImportUI, when files are copy from camera to computer, but i'm not sure. Gilles Caulier For BQM, relevant code written in ImageEditor must be called somewhere here : https://projects.kde.org/projects/extragear/graphics/digikam/repository/revisions/master/entry/utilities/queuemanager/manager/task.cpp#L233 ... to restore file attribute from original to target file. Gilles Caulier Thomas, I tested Image Editor ACL management on my computer where i use ext4. I open a JPEG file and just perform File Save As : [gilles@localhost TESTS]$ ls -al *.jpg -rw-rw-r-- 1 gilles gilles 24141637 sept. 4 13:59 2.jpg -rw-rw-r-- 1 gilles gilles 16004774 sept. 3 17:30 1.jpg [gilles@localhost TESTS]$ getfacl -e 1.jpg # file: 1.jpg # owner: gilles # group: gilles user::rw- group::rw- other::r-- [gilles@localhost TESTS]$ getfacl -e 2.jpg # file: 2.jpg # owner: gilles # group: gilles user::rw- group::rw- other::r-- 1.jpg is original, and 2.jpg the new file created. There is no difference. I miss something ? Git commit 1d29b8828ffde096d7d288dd76714c2f8edf56b1 by Gilles Caulier. Committed on 04/09/2014 at 12:59. Pushed by cgilles into branch 'master'. Factorize ACL method preservation from ImageEditor as a static and public method Use this method in BQM core when file are renamed. Add documantation API M +10 -9 utilities/imageeditor/editor/editorwindow.cpp M +6 -1 utilities/imageeditor/editor/editorwindow.h M +7 -3 utilities/queuemanager/manager/task.cpp http://commits.kde.org/digikam/1d29b8828ffde096d7d288dd76714c2f8edf56b1 Thomas, With my last commit, BQM must have the same behavior than Image Editor now. It still a problem with a restoration of all ACL properties. Right ? Gilles Caulier Marcel, Do you know what's missing to support fully ACL with code implemented now in filemanagement.cpp, to be common between IE et BQM : https://projects.kde.org/projects/extragear/graphics/digikam/repository/revisions/master/entry/utilities/imageeditor/core/filemanagement.cpp#L50 Gilles Sorry, I do not have ACLs in use. I would hope the original poster of this bug was available to carry out tests. Maik, Can you check if this file still valid with my last commit to factorize ACL management code with Editor and BQM ? Thanks in advance Gilles Can you reproduce the problem using digiKam Linux AppImage bundle ? The last bundle is available at this url: https://drive.google.com/drive/folders/0BzeiVr-byqt5Y0tIRWVWelRJenM Gilles Caulier digiKam 7.0.0 stable release is now published: https://www.digikam.org/news/2020-07-19-7.0.0_release_announcement/ We need a fresh feedback on this file using this version. Best Regards Gilles Caulier No feedback, closed |