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 682833 - Handle EINTR for open()
Handle EINTR for open()
Status: RESOLVED FIXED
Product: glib
Classification: Platform
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: gtkdev
gtkdev
Depends on:
Blocks:
 
 
Reported: 2012-08-27 22:32 UTC by Colin Walters
Modified: 2012-08-28 17:58 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
xdgmime: Handle EINTR in open() (960 bytes, patch)
2012-08-27 22:32 UTC, Colin Walters
committed Details | Review
Switch all open() calls to use g_open() (6.04 KB, patch)
2012-08-27 22:32 UTC, Colin Walters
committed Details | Review

Description Colin Walters 2012-08-27 22:32:25 UTC
This is just open().  Look for bugs, find bugs...
Comment 1 Colin Walters 2012-08-27 22:32:27 UTC
Created attachment 222592 [details] [review]
xdgmime: Handle EINTR in open()
Comment 2 Colin Walters 2012-08-27 22:32:30 UTC
Created attachment 222593 [details] [review]
Switch all open() calls to use g_open()

Because it now handles EINTR.  And we should do so.  While most people
use Linux, which tries very hard to avoid propagating EINTR back up
into userspace, it can still happen.
Comment 3 Dan Winship 2012-08-28 13:14:01 UTC
Comment on attachment 222593 [details] [review]
Switch all open() calls to use g_open()

sure
Comment 4 Colin Walters 2012-08-28 17:58:33 UTC
Attachment 222592 [details] pushed as 2542b6f - xdgmime: Handle EINTR in open()
Attachment 222593 [details] pushed as 6e64ba5 - Switch all open() calls to use g_open()