Inconsistent user of character string lengths

[Roberta Marton]

In Context.cpp (2014-09-16) code base:

Line 4853: char authNameFromTable[600];
Looks like authQuery will make the assumption that 600 (actually 599) is long enough. Shouldn't this at least be a defined constant rather than hard-coded?

Line 5097: char usersNameFromUsersTable[600];
600 seems to be a popular number. Should be a defined enum or const int in a header file somewhere.

We do already have a literal that defines the length of authorization names. This should be used instead of 600.

[Cliff Gray (cliff-gray)]

Fix complete, waiting until r1.0 is released before delivering.

[Cliff Gray (cliff-gray)]

Fix delivered in 1082.