diff options
author | Lukas Slebodnik <lslebodn@redhat.com> | 2014-05-27 14:53:11 +0200 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2014-05-27 21:23:04 +0200 |
commit | 03d7e3207a8758130d7e2431cd886bb3edfc6190 (patch) | |
tree | a27292839dfff725ec0fd1055b15450355446364 | |
parent | 5fc854880db68d9fb887b2d2767906dbf3a1ebe6 (diff) | |
download | sssd-03d7e3207a8758130d7e2431cd886bb3edfc6190.tar.gz sssd-03d7e3207a8758130d7e2431cd886bb3edfc6190.tar.xz sssd-03d7e3207a8758130d7e2431cd886bb3edfc6190.zip |
sss_autofs: Do not try to free empty autofs context
If initialisation fails in function _sss_setautomntent, context will not be
initialized and automount client will crash.
The function _sss_endautomntent should not try to dereference NULL pointer.
Resolves:
https://fedorahosted.org/sssd/ticket/2288
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
(cherry picked from commit 01f0d708fddfd50bdbb36a63737dcdc2c2fbd28d)
-rw-r--r-- | src/sss_client/autofs/sss_autofs.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/sss_client/autofs/sss_autofs.c b/src/sss_client/autofs/sss_autofs.c index f86ea867f..539e8ad1c 100644 --- a/src/sss_client/autofs/sss_autofs.c +++ b/src/sss_client/autofs/sss_autofs.c @@ -455,8 +455,10 @@ _sss_endautomntent(void **context) fctx = (struct automtent *) *context; - free(fctx->mapname); - free(fctx); + if (fctx != NULL) { + free(fctx->mapname); + free(fctx); + } ret = sss_autofs_make_request(SSS_AUTOFS_ENDAUTOMNTENT, NULL, NULL, NULL, &errnop); |