diff options
author | Oz Linden <oz@lindenlab.com> | 2016-03-22 14:56:06 -0400 |
---|---|---|
committer | Oz Linden <oz@lindenlab.com> | 2016-03-22 14:56:06 -0400 |
commit | 9944e8d9fa9beab69aaf70d35ad546628a79cb7f (patch) | |
tree | f0274f23a10ee5b57359ed8512b2e17865f0e663 /indra/newview/llvoicevivox.cpp | |
parent | 12c93702a71a76997eeaca3ef2f1a76d9446b8c6 (diff) |
VOICE-36: improvements based on review feedback
Diffstat (limited to 'indra/newview/llvoicevivox.cpp')
-rwxr-xr-x | indra/newview/llvoicevivox.cpp | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/indra/newview/llvoicevivox.cpp b/indra/newview/llvoicevivox.cpp index 60f9122056..c9661dfb11 100755 --- a/indra/newview/llvoicevivox.cpp +++ b/indra/newview/llvoicevivox.cpp @@ -184,6 +184,7 @@ LLVivoxVoiceClient::LLVivoxVoiceClient() : mCurrentParcelLocalID(0), mConnectorEstablished(false), + mAccountLoggedIn(false), mNumberOfAliases(0), mCommandCookie(0), mLoginRetryCount(0), @@ -7213,19 +7214,20 @@ void LLVivoxProtocolParser::processResponse(std::string tag) LLVivoxSecurity::LLVivoxSecurity() { - // this size is an arbitrary choice; Vivox does not care - #define VIVOX_TOKEN_BYTES 8 + // This size is an arbitrary choice; Vivox does not care + // Use a multiple of three so that there is no '=' padding in the base64 (purely an esthetic choice) + #define VIVOX_TOKEN_BYTES 9 U8 random_value[VIVOX_TOKEN_BYTES]; for (int b = 0; b < VIVOX_TOKEN_BYTES; b++) { - random_value[b] = random() & 0xff; + random_value[b] = ll_rand() & 0xff; } mConnectorHandle = LLBase64::encode(random_value, VIVOX_TOKEN_BYTES); for (int b = 0; b < VIVOX_TOKEN_BYTES; b++) { - random_value[b] = random() & 0xff; + random_value[b] = ll_rand() & 0xff; } mAccountHandle = LLBase64::encode(random_value, VIVOX_TOKEN_BYTES); } |