Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Renaming the "operating system" output? #14

Closed
DLKSASDl opened this issue Nov 24, 2021 · 1 comment
Closed

Renaming the "operating system" output? #14

DLKSASDl opened this issue Nov 24, 2021 · 1 comment

Comments

@DLKSASDl
Copy link

First of all I really like your tools and I appreciate the time and effort you put into developing them.

I was wondering about the "operation system" output by Nauz. By looking at the code it appears that the OS result for PE files is calculated by parsing the Major/MinorOperatingSystemVersion field of the Optional Header. My understanding of the PE format was, that those fields are like a "required minimum version" to run the file. I did some tests using legacy compilers (VS 05/08) running Windows XP SP3 and Win7 and had varying results (see attachment). Just by looking at the Nauz output some could think that "Operation system" refers to the OS Version used to build the file and not some arbitrary version number set by VS. Maybe it's best to rename the output to clear up the confusion.

results_vs.log

@horsicq
Copy link
Owner

horsicq commented Nov 25, 2021

Hello! Thanks a lot for you report! I fixed an improved the operation system output. You could see it if you will compile the project from sources or will wait new version. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants