After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 429289 - Created videos have file permission 600.
Created videos have file permission 600.
Status: RESOLVED FIXED
Product: istanbul
Classification: Other
Component: general
0.2.0
Other All
: High critical
: ---
Assigned To: Zaheer Abbas Merali
Zaheer Abbas Merali
Depends on:
Blocks:
 
 
Reported: 2007-04-13 10:56 UTC by Debarshi Ray
Modified: 2007-08-12 08:24 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fixes the bug for gnomevfs (1.38 KB, patch)
2007-04-15 23:26 UTC, Florian Boucault
none Details | Review
patch to chmod the created video to have file permission 0644 (467 bytes, patch)
2007-04-29 23:29 UTC, Chris RIvas
none Details | Review
patch to honour umask setting (1.87 KB, patch)
2007-08-11 15:05 UTC, Luca Bruno
none Details | Review

Description Debarshi Ray 2007-04-13 10:56:33 UTC
Please describe the problem:
Whenever a screencast is saved as a video in Ogg Theora format, the permissions of the file is -rw------- or 600. 

Steps to reproduce:
1. Start Istanbul. I am using 0.2.1 on a Fedora Core 6 i386 system (http://fedora.glug-nith.org/linux/extras/6/i386/istanbul-0.2.1-0.fc6.i386.rpm).

2. Record a screencast.

3. Save this screencast as a Ogg Theora file. Mention the name as foo.ogg in the 'save as' dialog.


Actual results:
The permissions of the created file is -rw------- or 600.

Expected results:
The file's permissions should have been -rw-r--r-- or 644 according to the system's umask setting.

Does this happen every time?
Yes.

Other information:
n.a.
Comment 1 Florian Boucault 2007-04-15 21:00:47 UTC
I can confirm this issue with version 0.2.2 on Ubuntu Feisty Fawn (7.04).
Comment 2 Florian Boucault 2007-04-15 23:26:10 UTC
Created attachment 86402 [details] [review]
Fixes the bug for gnomevfs

This patch solves the issue for the gnomevfs case only.

gnomevfs solution: GNOME_VFS_XFER_TARGET_DEFAULT_PERMS C constant has not been binded in Python for whatever reason (one should find or file a bug against the binding)

non-gnomevfs problem: tempfile by default uses 600 permission to make it secure. A solution could be to chmod the target file to the current inverted umask. However, I did not find a clean way to get the current umask.
Comment 3 Chris RIvas 2007-04-29 23:29:11 UTC
Created attachment 87256 [details] [review]
patch to chmod the created video to have file permission 0644

This works for non-gnomevfs cases.
Comment 4 Yevgen Muntyan 2007-07-13 05:55:44 UTC
Confirming, voting, buzzing, fuzzing. A bug!
"You don't have permission to access /files/el.ogg on this server."
Pretty inconvenient.
Comment 5 Luca Bruno 2007-08-11 15:03:58 UTC
Ok, so I've put up the two parts togheter in a common patch. I've also modified th second on honour umask setting (via os.umask). Now it just need a little bit of testing ;)
Comment 6 Luca Bruno 2007-08-11 15:05:32 UTC
Created attachment 93495 [details] [review]
patch to honour umask setting
Comment 7 Zaheer Abbas Merali 2007-08-12 08:24:52 UTC
Thanks for taking the time to report this bug.

Thanks also for fixing. This is now committed.