diff options
| author |  Karl Ramm <kcr@mit.edu> | 2009-03-23 13:48:03 +0000 |
|---|---|---|
| committer | Karl Ramm <kcr@mit.edu> | 2009-03-23 13:48:03 +0000 |
| commit | 7bf9b4390e877a960a2dbcc35e848b11dd989544 (patch) | |
| tree | 56a4cf6459d587503f885525b8495098574fb684 | |
| parent | 39fd7c353efd18d3daaf5d72043aa1a24cd095fc (diff) | |
notice->z_other_fields[notice->z_num_other_fields] is either going to
be NULL, something Wrong^TM, or off the end of z_other_fields.
bad all around.
| -rw-r--r-- | lib/ZCkZAut.c | 2 | ||||
| -rw-r--r-- | server/kstuff.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/lib/ZCkZAut.c b/lib/ZCkZAut.c index 2110871..19ec6d1 100644 --- a/lib/ZCkZAut.c +++ b/lib/ZCkZAut.c @@ -102,7 +102,7 @@ Code_t ZCheckZcodeAuthentication(ZNotice_t *notice, */ cksum1_base = notice->z_multinotice; if (notice->z_num_other_fields) - x = notice->z_other_fields[notice->z_num_other_fields]; + x = notice->z_other_fields[notice->z_num_other_fields - 1]; else x = cksum1_base + strlen(cksum1_base) + 1; /* multiuid */ cksum1_len = x + strlen(x) + 1 - cksum1_base; diff --git a/server/kstuff.c b/server/kstuff.c index ba50fdc..5ae30d4 100644 --- a/server/kstuff.c +++ b/server/kstuff.c @@ -720,7 +720,7 @@ ZCheckAuthentication(ZNotice_t *notice, */ cksum1_base = notice->z_multinotice; if (notice->z_num_other_fields) - x = notice->z_other_fields[notice->z_num_other_fields]; + x = notice->z_other_fields[notice->z_num_other_fields - 1]; else x = cksum1_base + strlen(cksum1_base) + 1; /* multiuid */ cksum1_len = x + strlen(x) + 1 - cksum1_base; |