diff options
| author | Greg Hudson <ghudson@mit.edu> | 2011-10-28 16:18:45 +0000 |
|---|---|---|
| committer | Greg Hudson <ghudson@mit.edu> | 2011-10-28 16:18:45 +0000 |
| commit | 8d54c72ae291cea5c68f810ebe7629520e395105 (patch) | |
| tree | 48904b6525bc614e4527ac0faa7fd9a3296a37ec /src/kdc/dispatch.c | |
| parent | 6fd903e14fa9b73c2c312c3d6af383a7356ebc5a (diff) | |
| download | krb5-8d54c72ae291cea5c68f810ebe7629520e395105.tar.gz krb5-8d54c72ae291cea5c68f810ebe7629520e395105.tar.xz krb5-8d54c72ae291cea5c68f810ebe7629520e395105.zip | |
Use zero-filled states for all async ops in KDC
There have been a couple of uninitialized field bugs in the
restructured KDC code, partly because compilers can't find these bugs
as easily as they can find uninitialized local variable bugs. Use
zero-filled state structures to make this type of bug less likely.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@25422 dc483132-0cff-0310-8789-dd5450dbe970
Diffstat (limited to 'src/kdc/dispatch.c')
| -rw-r--r-- | src/kdc/dispatch.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/kdc/dispatch.c b/src/kdc/dispatch.c index eeb95ff73..1398a33b6 100644 --- a/src/kdc/dispatch.c +++ b/src/kdc/dispatch.c @@ -90,9 +90,9 @@ dispatch(void *cb, struct sockaddr *local_saddr, const krb5_fulladdr *from, krb5_data *response = NULL; struct dispatch_state *state; - state = malloc(sizeof(*state)); - if (!state) { - (*respond)(arg, ENOMEM, NULL); + state = k5alloc(sizeof(*state), &retval); + if (state == NULL) { + (*respond)(arg, retval, NULL); return; } state->respond = respond; |