While loop executed twice time after calling a specific function - c

Question

I have written the following function which works for all parameters well rather than calling InteractiveCredentialDispathcer. When this function called, loop executed again from the top and doing all checks again. How should I fix this issue?
void InteractiveMode()
{
char commands[MAX_PATH] = { 0 };
PrintColorful(0, "%s", PTH_CMD);
while (fgets(commands, MAX_PATH - 1, stdin) != NULL)
{
if (strstr(commands, "help") || strstr(commands, "?"))
{
ShowHelpMessage();
}
else if (strstr(commands, "clear"))
{
ClearConsoleOutput(GetStdHandle(STD_OUTPUT_HANDLE));
}
else if (strstr(commands, "privilege"))
{
ShowPrivilegeStatus();
}
else if (strstr(commands, "login"))
{
if (InteractiveCredentialDispatcher())
{
NormalMessage("%s\n", "Process has been spawned successfully");
}
else
{
ErrorMessage("%s\n", "Process has not been spawned successfully");
}
}
else if (strstr(commands, "version"))
{
ShowProgramVersion();
}
else if (strstr(commands, "exit"))
{
NormalMessage("%s\n", "Program has been finished.");
ProgramExit();
}
else
{
printf("\n");
ErrorMessage("%s\n\n", "The command isn't appropriate:");
printf("\t");
NormalMessage("%s\n", "You entered a blank input or a wrong command.");
printf("\t");
NormalMessage("%s\n\n", "Execute help or ? command to see the manual.");
}
PrintColorful(0, "%s", PTH_CMD);
}
}
The following code called when the function InteractiveCredentialDispatcher executed:
BOOL InteractiveCredentialDispatcher()
{
TCHAR l_tc_Username[MAX_PATH];
TCHAR l_tc_Domain[MAX_PATH];
TCHAR l_tc_HashNtlm[MAX_PATH];
TCHAR l_tc_ComputerName[MAX_PATH];
TCHAR l_tc_ProcessName[MAX_PATH];
printf("\n\t");
WarningMessage("%s\n", "Fill following information:");
printf("\n\t\tUsername: ");
wscanf(L"%ls", l_tc_Username);
printf("\t\tDomain: ");
wscanf(L"%ls", l_tc_Domain);
printf("\t\tNTLM Hash: ");
wscanf(L"%ls", l_tc_HashNtlm);
printf("\t\tComputer Name: ");
wscanf(L"%ls", l_tc_ComputerName);
wcscpy(l_tc_ProcessName, TEXT("FM.exe "));
wcscat(l_tc_ProcessName, l_tc_ComputerName);
if (InteractiveAuthticationOnWindows(l_tc_Username, l_tc_Domain, l_tc_HashNtlm, l_tc_ProcessName))
return TRUE;
else
return FALSE;
}

Answer 1

The problem belonged to using scanf rather fgets in the function InteractiveCredentialDispatcher. I have solved the problem with the usage of just fgets and fgetws variant in the program. After that, while loop never executed twice.
BOOL InteractiveCredentialDispatcher()
{
TCHAR l_tc_Username[MAX_PATH];
TCHAR l_tc_Domain[MAX_PATH];
TCHAR l_tc_HashNtlm[MAX_PATH];
TCHAR l_tc_ComputerName[MAX_PATH];
TCHAR l_tc_ProcessName[MAX_PATH];
printf("\n\t");
WarningMessage("%s\n", "Fill following information:");
printf("\n\t\tUsername: ");
fgetstr(l_tc_Username, MAX_PATH, stdin);
printf("\t\tDomain: ");
fgetstr(l_tc_Domain, MAX_PATH, stdin);
printf("\t\tNTLM Hash: ");
fgetstr(l_tc_HashNtlm, MAX_PATH, stdin);
printf("\t\tComputer Name: ");
fgetstr(l_tc_ComputerName, MAX_PATH, stdin);
wcscpy(l_tc_ProcessName, TEXT("PTH-FileManager.exe "));
wcscat(l_tc_ProcessName, l_tc_ComputerName);
if (InteractiveAuthticationOnWindows(l_tc_Username, l_tc_Domain, l_tc_HashNtlm, l_tc_ProcessName))
{
return TRUE;
}
else
{
return FALSE;
}
}