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 151767 - File descriptor leak in egg_recent_model_add_full
File descriptor leak in egg_recent_model_add_full
Status: RESOLVED FIXED
Product: libegg
Classification: Other
Component: recent-files
unspecified
Other Linux
: High major
: ---
Assigned To: James Willcox
James Willcox
Depends on:
Blocks:
 
 
Reported: 2004-09-03 15:20 UTC by Crispin Flowerday (not receiving bugmail)
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch (726 bytes, patch)
2004-09-03 15:21 UTC, Crispin Flowerday (not receiving bugmail) 		none Details | Review

Description Crispin Flowerday (not receiving bugmail) 2004-09-03 15:20:03 UTC 
If the model fails to lock the file, it will leak the file descriptor. I
currently have 20 leaked file descriptors in Galeon at the moment
Comment 1 Crispin Flowerday (not receiving bugmail) 2004-09-03 15:21:01 UTC 
Created attachment 31241 [details] [review]
patch

This is the obvious fix. It is actually against the copy in galeon, but should
be fine against the copy in libegg
Comment 2 Paolo Borelli 2004-09-28 15:56:58 UTC 
still there in HEAD. can we have this one liner committed so that all modules
can update?
Comment 3 Paolo Borelli 2004-09-28 16:14:27 UTC 
I asked approval from snorp on irc and committed this.