By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.
Bug 349 - Files are saved with incorrect group ownership
Summary: Files are saved with incorrect group ownership
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdelibs (show other bugs)
Version: 3.5.12 [Trinity]
Hardware: All Linux
: P1 critical
Assignee: Timothy Pearson
URL:
Depends on:
Blocks:
 
Reported: 2010-10-06 16:56 CDT by John A. Sullivan III
Modified: 2012-10-19 15:18 CDT (History)
3 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description John A. Sullivan III 2010-10-06 16:56:35 CDT 
In our multiclient environment, the shared file areas typically have permissions of 2770, i.e., we set GID on the directories so files are automatically created owned by the group and writable by the group.

Files created with KSnapshot using the Save As button are not honoring the GID bit but rather are setting the group ownership to the user.  Notice this file listing:

jasiii@jasiii:/data/Tech$ touch ScreenTest2
jasiii@jasiii:/data/Tech$ ls -l Scre*
-rw-rw---- 1 jasiii jasiii    161569 2010-10-06 17:53 ScreenTest1.png
-rw-rw---- 1 jasiii Tech-a100      0 2010-10-06 17:53 ScreenTest2

As you can see, ScreenTest2 was made by touching the file.  ScreenTest1.png was saved from KSnapshot.  Group ownership of ScreenTest2 is correct but ScreenTest1.png is incorrect - John
Comment 1 John A. Sullivan III 2011-01-12 11:09:41 CST 
Hello, all.  Is there any progress on this bug? One of our clients was bitten by it again today when producing some documentation.  Thanks - John
Comment 2 John A. Sullivan III 2011-02-07 02:29:33 CST 
Bump :)
Comment 3 Robert Xu 2011-03-12 21:08:00 CST 
This is a kdelibs problem - KSaveFile.
This is now a critical bug for Trinity 3.5.12.
Comment 4 Timothy Pearson 2012-05-25 21:19:45 CDT 
It looks like KSaveFile uses KTempFile internally, and KTempFile does this:

   // Set uid/gid (necessary for SUID programs)
   fchown(mFd, getuid(), getgid());

Therefore, I suspect this bug affects more programs than just KSnapShot.
Comment 5 Timothy Pearson 2012-05-25 22:23:00 CDT 
Fixed in GIT hash 9068fff and repair confirmed on test platform.

Thanks for reporting!