From: Simon Wilkinson <sxw@your-file-system.com>
Date: Sat, 2 Mar 2013 09:33:12 +0000 (+0000)
Subject: kauth: Don't overflow cell string
X-Git-Tag: upstream/1.8.0_pre1^2~1355
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=77aa6c65b74e46c77dfaf440745496ab98b14244;p=packages%2Fo%2Fopenafs.git

kauth: Don't overflow cell string

The cell string within a ktc_principal is only 64 characters long.
Be careful not to overflow it.

Caught by coverity (#985829)

Change-Id: I761dfcf5eebf1bd4ef31aa5982a2593e5317257b
Reviewed-on: http://gerrit.openafs.org/9350
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
---

diff --git a/src/kauth/knfs.c b/src/kauth/knfs.c
index 4b4be5a53..81a03d64b 100644
--- a/src/kauth/knfs.c
+++ b/src/kauth/knfs.c
@@ -183,7 +183,7 @@ GetTokens(afs_int32 ahost, afs_int32 auid)
 		    sprintf(clientName.name, "Unix UID %d", ct.ViceId);
 		    clientName.instance[0] = 0;
 		}
-		strcpy(clientName.cell, tp);
+		strlcpy(clientName.cell, tp, sizeof(clientName.cell));
 
 		tokenExpireTime = token.endTime;
 		strcpy(UserName, clientName.name);