Exetools

Exetools (https://forum.exetools.com/index.php)
-   General Discussion (https://forum.exetools.com/forumdisplay.php?f=2)
-   -   Bug fixes for EnigmaHardwareID source code (https://forum.exetools.com/showthread.php?t=20969)

Mr.Leng 04-20-2024 16:53

Bug fixes for EnigmaHardwareID source code
 
Posts
https://forum.exetools.com/showthread.php?t=20765

Error in retrieving hardware serial number in Windows 7 virtual machine.Error in function void GetHDDString().

My modification method:

void RemoveParentheses(char* str)
{
char* dest = str;
for (char* src = str; *src != '\0'; ++src) {
if (*src != '(' && *src != ')') {
*dest = *src;
++dest;
}
}
*dest = '\0';
}
--------------------------------------------------------------------------
----------------------Keep the original code logic unchanged------------
----------------------add code-----------------------------------------
StringLen2 = strlen(bytes_values);
for (i = 0; i < StringLen2; i += 2) // reverse chars
{
char firstChar = bytes_values[i];
char SecondChar = bytes_values[i + 1];
if (SecondChar != 0)
{
bytes_values[i + 1] = firstChar;
bytes_values[i] = SecondChar;
}
}
strcat_s(HDDStr, bytes_values);

//add code
RemoveParentheses(HDDStr);
}
else
strcat_s(HDDStr, SecondString);

//add code
RemoveParentheses(HDDStr);
--------------------------------------------------------------------------
----------------------Keep the original code logic unchanged------------

Ayumi 04-22-2024 05:55

This fix also works for Windows 8 in virtual machine (not need for 8.1)

chants 04-22-2024 08:40

Quote:

Originally Posted by Ayumi (Post 130701)
This fix also works for Windows 8 in virtual machine (not need for 8.1)

Tyro, I'm not sure there is much relevance to Win8/8.1 anymore:
Quote:

https://gs.statcounter.com/os-version-market-share/windows/desktop/worldwide
It's basically less than 1% or computers, more use Win7 by far than Win8. Generally Win10/11 are most important and in rarer cases 7. Perhaps you use old software and are a mere fledgling when it comes to modern OS but I'd encourage updating if only for security updates.

CodeCracker 04-23-2024 16:10

@Mr.Leng: Many thanks for bug report.
let me know if problems was solved in uploaded.
https://workupload.com/file/ydhLR2XhpHQ
https://workupload.com/file/JdqjhJtqTkK

NON 04-28-2024 14:23

Not work for Windows 7 SP1 in virtual machine (64-bit).


All times are GMT +8. The time now is 14:28.

Powered by vBulletin® Version 3.8.8
Copyright ©2000 - 2026, vBulletin Solutions, Inc.
Always Your Best Friend: Aaron, JMI, ahmadmansoor, ZeNiX