ptserver: Do not use cell for entry name len check
authorAndrew Deason <adeason@sinenomine.net>
Fri, 30 Jul 2010 20:32:53 +0000 (15:32 -0500)
committerDerrick Brashear <shadow@dementia.org>
Mon, 2 Aug 2010 16:35:29 +0000 (09:35 -0700)
commit5e946d5b6672d4dea7dd1c3ee669245b3d2ced92
treefe297b796b4e05a9a8e426bf95821940f09dc9d5
parent89afd3c21e816c29f497c10cf76673d009cdd919
ptserver: Do not use cell for entry name len check

Do not use the local cell name when determining if a new entry name is
too long. This check assumes that foreign cells will use our local
cell name (assumed to be our local Kerberos realm) in a certain way,
and prevents creating users that will make those names too long.

This is undesirable for several reasons. One is that the local realm
name may not be the same as the local cell name (and we may have many
local realms). Another is that we cannot reliably predict how foreign
cells will construct foreign pt entry names, so preventing entry
creation based on that may prevent names that will never cause any
problems. This check also assumes that our names will be used as
foreign entries in other cells, which may not be the case.

So, remove the check based on the local cell name, and remove the
pr_realmNameLen variable while we are at it, since this is all it is
used for.

Thanks to Jeffrey Altman for discussion, and for bringing this up in
the first place.

Change-Id: Ief4bc94d9ead61a1589797b5dc663a6473c9ed72
Reviewed-on: http://gerrit.openafs.org/2488
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
src/ptserver/ptserver.c
src/ptserver/ptutils.c
src/ptserver/ubik.c