chinmoyr created this revision. chinmoyr added reviewers: Okular, aacid. Restricted Application added a project: Okular. chinmoyr requested review of this revision.
REVISION SUMMARY Okular saves a new file with permissions 0600 completely ignoring the umask value. This is because it makes use of QTemporaryFile which creates file with the said permissions and which then Okular copies to the new location. So to overcome this generate new file permissions using old mask value and change permissions of the new file. BUG: 392682 TEST PLAN Set umask to 0040 Open a pdf file from the same shell and save it under a new name. Before patch: file permisions -> 0600 After patch: file permisions -> 0606 REPOSITORY R223 Okular BRANCH master REVISION DETAIL https://phabricator.kde.org/D12049 AFFECTED FILES part.cpp To: chinmoyr, #okular, aacid Cc: michaelweghorn, ngraham, aacid