diff options
author | Ray Strode <rstrode@redhat.com> | 2012-10-12 13:08:53 -0400 |
---|---|---|
committer | Ray Strode <rstrode@redhat.com> | 2012-10-15 11:42:57 -0400 |
commit | 84a330a1ecb657f0512715361c834e3324d6a724 (patch) | |
tree | 80c13f5ff819ffa641b8f9b4e0c97849980f8df2 | |
parent | 91b182bb96d65793e97cb20e9b190637beaca23d (diff) | |
download | gdm-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.c | 4 |
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); |