diff options
author | Danilo Almeida <dalmeida@mit.edu> | 1999-08-04 03:18:29 +0000 |
---|---|---|
committer | Danilo Almeida <dalmeida@mit.edu> | 1999-08-04 03:18:29 +0000 |
commit | 21964d83bbacfb52bc7e1cf40e3def2c048a9468 (patch) | |
tree | 390452cbda250410e0b4ceb41da4fcc7f09041b5 /src/lib/krb5/krb/get_creds.c | |
parent | 6404ed459468bcd3aeb35e21e73da960463461f0 (diff) | |
download | krb5-21964d83bbacfb52bc7e1cf40e3def2c048a9468.tar.gz krb5-21964d83bbacfb52bc7e1cf40e3def2c048a9468.tar.xz krb5-21964d83bbacfb52bc7e1cf40e3def2c048a9468.zip |
Intialize out_creds pointer to 0 and then check whether it is 0 before
trying to dereference it, in case lower-level routine failed to assign
a value to it.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@11619 dc483132-0cff-0310-8789-dd5450dbe970
Diffstat (limited to 'src/lib/krb5/krb/get_creds.c')
-rw-r--r-- | src/lib/krb5/krb/get_creds.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/src/lib/krb5/krb/get_creds.c b/src/lib/krb5/krb/get_creds.c index 37f2bb5a9..4fbf4cf33 100644 --- a/src/lib/krb5/krb/get_creds.c +++ b/src/lib/krb5/krb/get_creds.c @@ -226,7 +226,7 @@ krb5_validate_or_renew_creds(context, creds, client, ccache, in_tkt_service, { krb5_error_code ret; krb5_creds in_creds; /* only client and server need to be filled in */ - krb5_creds *out_creds; + krb5_creds *out_creds = 0; /* for check before dereferencing below */ krb5_creds **tgts; memset((char *)&in_creds, 0, sizeof(krb5_creds)); @@ -277,9 +277,10 @@ krb5_validate_or_renew_creds(context, creds, client, ccache, in_tkt_service, &in_creds, &out_creds, &tgts); /* ick. copy the struct contents, free the container */ - - *creds = *out_creds; - krb5_xfree(out_creds); + if (out_creds) { + *creds = *out_creds; + krb5_xfree(out_creds); + } cleanup: |