From 585a0758dfd1c45551cba4d9088d4938422dfcc7 Mon Sep 17 00:00:00 2001 From: Simon Wilkinson Date: Sat, 11 Sep 2010 23:14:42 +0100 Subject: [PATCH] auth: Make token_FreeSet work on an empty set If the set passed to token_FreeSet is already freed, just make the funciton a no-op, rather than segfaulting. Change-Id: I41342d1c7bd94bcf20ad67407b28d9ffcccf3c07 Reviewed-on: http://gerrit.openafs.org/2751 Reviewed-by: Derrick Brashear Tested-by: Derrick Brashear --- src/auth/token.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/auth/token.c b/src/auth/token.c index 4070bb87b..59d323b3d 100644 --- a/src/auth/token.c +++ b/src/auth/token.c @@ -429,7 +429,9 @@ token_setPag(struct ktc_setTokenData *jar, int setpag) { void token_FreeSet(struct ktc_setTokenData **jar) { - xdr_free((xdrproc_t)xdr_ktc_setTokenData, *jar); - memset(*jar, 0, sizeof(struct ktc_setTokenData)); - *jar = NULL; + if (*jar) { + xdr_free((xdrproc_t)xdr_ktc_setTokenData, *jar); + memset(*jar, 0, sizeof(struct ktc_setTokenData)); + *jar = NULL; + } } -- 2.39.5