STABLE14-rxkad-rt34568-20060629

FIXES 34568

The use of strncmp() results in p->v5_str being treated as matching
prefix instead of as the complete string.  This causes "ftpa" to be
considered a match for "ftp".

(cherry picked from commit f0830f89a11a2fea06a26270a90c7ddf8365a533)

diff --git a/src/rxkad/ticket5.c b/src/rxkad/ticket5.c
index c5bed59f055ab50e4a0768e3de7729ea835c3baa..126721c17e2225e19d03be49c614c45bd62f9bb2 100644 (file)
--- a/src/rxkad/ticket5.c
+++ b/src/rxkad/ticket5.c
@@ -284,7 +284,7 @@ tkt_DecodeTicket5(char *ticket, afs_int32 ticket_len,
        v5_comp1 = decr_part.cname.name_string.val[1];
        p = sconv_list;
        while (p->v4_str) {
-           if (strncmp(p->v5_str, v5_comp0, p->len) == 0) {
+           if (strcmp(p->v5_str, v5_comp0) == 0) {
                /*
                 * It is, so set the new name now, and chop off
                 * instance's domain name if requested.