Mirror/TrinityCore
1
0
Fork 0
mirror of https://github.com/TrinityCore/TrinityCore.git synced 2026-01-21 09:44:45 +01:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #10801 from jackpoz/account_password_crashfix

Browse Source 
Core/Commands: Fix NULL dereference crash in .account password
This commit is contained in:
Machiavelli
2013-09-13 12:09:26 -07:00
parent cb4bf1fd97 6581d7db85
commit 2e15f78305
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/server/scripts/Commands/cs_account.cpp
View File

@@ -453,9 +453,9 @@ public:
char* oldPassword = strtok((char*)args, " "); // This extracts [$oldpassword]
char* newPassword = strtok(NULL, " "); // This extracts [$newpassword]
char* passwordConfirmation = strtok(NULL, " "); // This extracts [$newpasswordconfirmation]
char* emailConfirmation = NULL; // This defines the emailConfirmation variable, which is optional depending on sec type.
const char* emailConfirmation; // This defines the emailConfirmation variable, which is optional depending on sec type.
if (!(emailConfirmation = strtok(NULL, " "))) // This extracts [$emailconfirmation]. If it doesn't exist, however...
emailConfirmation = '\0'; // ... it's simply "" for emailConfirmation.
emailConfirmation = ""; // ... it's simply "" for emailConfirmation.
//Is any of those variables missing for any reason ? We return false.
if (!oldPassword || !newPassword || !passwordConfirmation)