diff options
| author | Jeff Layton <jlayton@redhat.com> | 2009-03-23 08:13:01 -0400 |
|---|---|---|
| committer | Steve Dickson <steved@redhat.com> | 2009-03-23 08:13:01 -0400 |
| commit | d62365079f711b25e73522b2af380abc2a7e2788 (patch) | |
| tree | b0499054f3b8912cb71b27a28907087a414bab5a /utils/gssd/gssd_proc.c | |
| parent | 38062464a09868ff579b3c3c0b43fcd550ea3c61 (diff) | |
| download | nfs-utils-d62365079f711b25e73522b2af380abc2a7e2788.tar.gz nfs-utils-d62365079f711b25e73522b2af380abc2a7e2788.tar.xz nfs-utils-d62365079f711b25e73522b2af380abc2a7e2788.zip | |
Regardless though, it's dangerous to keep invalid pointers around like
this. Later code changes may make it more likely for this problem to
occur.
Also eliminate some unneeded NULL pointer checks before freeing memory.
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Steve Dickson <steved@redhat.com>
Diffstat (limited to 'utils/gssd/gssd_proc.c')
| -rw-r--r-- | utils/gssd/gssd_proc.c | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/utils/gssd/gssd_proc.c b/utils/gssd/gssd_proc.c index fb97a13..509946e 100644 --- a/utils/gssd/gssd_proc.c +++ b/utils/gssd/gssd_proc.c @@ -182,9 +182,10 @@ read_service_info(char *info_file_name, char **servicename, char **servername, fail: printerr(0, "ERROR: failed to read service info\n"); if (fd != -1) close(fd); - if (*servername) free(*servername); - if (*servicename) free(*servicename); - if (*protocol) free(*protocol); + free(*servername); + free(*servicename); + free(*protocol); + *servicename = *servername = *protocol = NULL; return -1; } @@ -200,10 +201,10 @@ destroy_client(struct clnt_info *clp) if (clp->dir_fd != -1) close(clp->dir_fd); if (clp->krb5_fd != -1) close(clp->krb5_fd); if (clp->spkm3_fd != -1) close(clp->spkm3_fd); - if (clp->dirname) free(clp->dirname); - if (clp->servicename) free(clp->servicename); - if (clp->servername) free(clp->servername); - if (clp->protocol) free(clp->protocol); + free(clp->dirname); + free(clp->servicename); + free(clp->servername); + free(clp->protocol); free(clp); } |