From: Jeffrey Altman <jaltman@secure-endpoints.com>
Date: Fri, 30 Jun 2006 00:25:49 +0000 (+0000)
Subject: STABLE14-rxkad-rt34568-20060629
X-Git-Tag: openafs-stable-1_4_2-beta2~17
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=fbe651dc367d69ea908eb4487243af07fceb3678;p=packages%2Fo%2Fopenafs.git

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 c5bed59f0..126721c17 100644
--- 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.