From: Andrew Deason <adeason@sinenomine.net>
Date: Wed, 11 Nov 2009 16:51:19 +0000 (-0600)
Subject: Do not check *aoutSize in PGetPAG
X-Git-Tag: openafs-devel-1_5_67~58
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=97d3fcfcd1617bafa6ee62d291f09b2e8bb6daf7;p=packages%2Fo%2Fopenafs.git

Do not check *aoutSize in PGetPAG

*aoutSize is always zero in pioctls, since afs_HandlePioctl handles
checking the output buffer size, and sets outSize to 0 before calling
the pioctl. So, PGetPAG was always returning E2BIG; remove the check to
make it work.

Change-Id: Ibed16b35ea14237f44a494add8c7a4a48e0c2b71
Reviewed-on: http://gerrit.openafs.org/814
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
---

diff --git a/src/afs/afs_pioctl.c b/src/afs/afs_pioctl.c
index 467b3819c..a16388eca 100644
--- a/src/afs/afs_pioctl.c
+++ b/src/afs/afs_pioctl.c
@@ -2342,18 +2342,12 @@ DECL_PIOCTL(PViceAccess)
  * \param[in] ain	not in use
  * \param[out] aout	PAG value or NOPAG
  *
- * \retval E2BIG	Error not enough space to copy out value
- *
  * \post get PAG value for the caller's cred
  */
 DECL_PIOCTL(PGetPAG)
 {
     afs_int32 pag;
 
-    if (*aoutSize < sizeof(afs_int32)) {
-	return E2BIG;
-    }
-
     pag = PagInCred(*acred);
 
     memcpy(aout, (char *)&pag, sizeof(afs_int32));