summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRay Strode <rstrode@redhat.com>2012-10-12 13:08:53 -0400
committerRay Strode <rstrode@redhat.com>2012-10-15 11:42:57 -0400
commit84a330a1ecb657f0512715361c834e3324d6a724 (patch)
tree80c13f5ff819ffa641b8f9b4e0c97849980f8df2
parent91b182bb96d65793e97cb20e9b190637beaca23d (diff)
downloadgdm-84a330a1ecb657f0512715361c834e3324d6a724.tar.gz
session-worker: don't run close() on negative fds
There were two places in the code that run close() on fd's that failed to get opened. This commit fixes those two places.
-rw-r--r--daemon/gdm-session-worker.c4
1 files changed, 0 insertions, 4 deletions
diff --git a/daemon/gdm-session-worker.c b/daemon/gdm-session-worker.c
index 1dfeee3e..3acdfee7 100644
--- a/daemon/gdm-session-worker.c
+++ b/daemon/gdm-session-worker.c
@@ -1644,8 +1644,6 @@ _open_program_session_log (const char *filename)
if (fd < 0) {
char *temp_name;
- close (fd);
-
temp_name = g_strdup_printf ("%s.XXXXXXXX", filename);
fd = g_mkstemp (temp_name);
@@ -1703,8 +1701,6 @@ _open_user_session_log (const char *dir)
if (fd < 0) {
char *temp_name;
- close (fd);
-
temp_name = g_strdup_printf ("%s.XXXXXXXX", filename);
fd = g_mkstemp (temp_name);