diff options
| author | Brian Cameron <brian.cameron@sun.com> | 2006-10-06 00:56:45 +0000 |
|---|---|---|
| committer | Brian Cameron <bcameron@src.gnome.org> | 2006-10-06 00:56:45 +0000 |
| commit | 6bfca48299f1f5c70b9acf5578b998d4fbc478c3 (patch) | |
| tree | 4181ec61271f537e505b8b062f4ae21f2e83ddfa | |
| parent | 7ec941263725025eb2fcbab1f94025918063eeaf (diff) | |
| download | gdm-6bfca48299f1f5c70b9acf5578b998d4fbc478c3.tar.gz | |
Set authdir to NULL after freeing to avoid accessing an invalid pointer.
2006-10-05 Brian Cameron <brian.cameron@sun.com>
* daemon/auth.c: Set authdir to NULL after freeing to avoid accessing
an invalid pointer. Fixes bug #359831. Patch provided by
Amnon Aaronsohn <bla@cs.huji.ac.il>.
| -rw-r--r-- | ChangeLog | 6 | ||||
| -rw-r--r-- | daemon/auth.c | 2 |
2 files changed, 8 insertions, 0 deletions
@@ -1,3 +1,9 @@ +2006-10-05 Brian Cameron <brian.cameron@sun.com> + + * daemon/auth.c: Set authdir to NULL after freeing to avoid accessing + an invalid pointer. Fixes bug #359831. Patch provided by + Amnon Aaronsohn <bla@cs.huji.ac.il>. + 2006-08-03 Ray Strode <rstrode@redhat.com> * configure.ac: look for /usr/bin/Xorg when searching or diff --git a/daemon/auth.c b/daemon/auth.c index c8086b2b..a3977cdf 100644 --- a/daemon/auth.c +++ b/daemon/auth.c @@ -710,7 +710,9 @@ try_user_add_again: af = gdm_safe_fopen_ap (d->userauth); } + /* Set to NULL, because can goto try_user_add_again. g_free (authdir); + authdir = NULL; if G_UNLIKELY (af == NULL) { /* Really no need to clean up here - this process is a goner anyway */ |