Fix issues in other queries caused by changing fail_driver1.c

[jacob-ronstadt]

Checklist for Pull Requests

Fix issues in other queries caused by changing fail_driver1.c

• Description is filled out.
• Only one query or related query group is in this pull request.
• The version number on changed queries has been increased via the @version comment in the file header.
• All unit tests have been run: (Test README.md).
• Commands codeql database create and codeql database analyze have completed successfully.
• A .qhelp file has been added for any new queries or updated if changes have been made to an existing query.

[NateD-MSFT]

If you can take a minute to check if there's a way to pass in definitions on the MSBuild line beyond creating a whole new configuration for the project, I would appreciate it. Adding another configuration outside of Debug and Release feels heavy-handed to me to just get a single #define set up correctly, but maybe this really is the only way.

[jacob-ronstadt]

@NateD-MSFT I looked into this some more and there doesn't seem to be an easy way to just pass a preprocessor definition from msbuild to the compiler (/D flag for cl.exe). A slight workaround is to add a preprocessor definition to the C/C++ properties of the VS project in the form NAME=$(PropertyName). "NAME" can then be used in the code for conditional compilation and msbuild can be called with an added property of /p:NAME=value. This would be easier to expand to add new definitions than having multiple configurations. It would only require adding a new preprocessor name/value pair to the project properties.