diff options
| author | Greg Hudson <ghudson@mit.edu> | 2011-10-04 22:40:10 +0000 |
|---|---|---|
| committer | Greg Hudson <ghudson@mit.edu> | 2011-10-04 22:40:10 +0000 |
| commit | 57a52177feee207d8b3f4bd0fbf7a3d7ee09c070 (patch) | |
| tree | 92030d33fdb3123ee6f80173786d63087cce6355 /src/lib/apputils | |
| parent | cbb4ede6d5a939f39f3325ad040406ac05c99713 (diff) | |
| download | krb5-57a52177feee207d8b3f4bd0fbf7a3d7ee09c070.tar.gz krb5-57a52177feee207d8b3f4bd0fbf7a3d7ee09c070.tar.xz krb5-57a52177feee207d8b3f4bd0fbf7a3d7ee09c070.zip | |
Fix initialization and pointer bugs in new code
Coverity found some minor-to-medium bugs in some recent changes; fix
them.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@25299 dc483132-0cff-0310-8789-dd5450dbe970
Diffstat (limited to 'src/lib/apputils')
| -rw-r--r-- | src/lib/apputils/net-server.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/lib/apputils/net-server.c b/src/lib/apputils/net-server.c index b31c6bc5c..11a7c122f 100644 --- a/src/lib/apputils/net-server.c +++ b/src/lib/apputils/net-server.c @@ -1548,7 +1548,7 @@ process_packet_response(void *arg, krb5_error_code code, krb5_data *response) if (code) com_err(state->prog ? state->prog : NULL, code, _("while dispatching (udp)")); - if (code || response == NULL || state == NULL) + if (code || response == NULL) goto out; cc = send_to_from(state->port_fd, response->data, @@ -1900,8 +1900,10 @@ process_tcp_connection_read(verto_ctx *ctx, verto_ev *ev) } state = prepare_for_dispatch(ctx, ev); - if (!state) + if (!state) { + krb5_free_data(get_context(conn->handle), response); goto kill_tcp_connection; + } process_tcp_response(state, 0, response); } } |