Msbuild: Checking the Source Code of MSBuild with PVS-Studio

Created on 24 Aug 2016 · 5Comments · Source: dotnet/msbuild

Developers of PVS-Studio C/C++/C# static analyzer present their check report of the source code of 'MSBuild' in the article, containing the review of the most suspicious code fragments they discovered.

You can read article at the official site:
http://www.viva64.com/en/b/0424/

Source

skhrenov

Most helpful comment

This looks more like an ad then an actual bug report, have you found actual bugs or errors? Please mention that instead of linking to a list of warnings on an external website.

PS. Not affliated to msbuild :)

NicoJuicy on 25 Aug 2016

👍9 👎4

All 5 comments

This looks more like an ad then an actual bug report, have you found actual bugs or errors? Please mention that instead of linking to a list of warnings on an external website.

PS. Not affliated to msbuild :)

NicoJuicy on 25 Aug 2016

👍9 👎4

It's hard to tell from most compiler or static analyzer warnings will it cause actual bugs or errors
Especially for people who do not know this code

KindDragon on 25 Aug 2016

Please be friendly

KindDragon on 25 Aug 2016

👍5 👎1

https://medium.com/@sofiafateeva93/checking-the-source-code-of-msbuild-with-pvs-studio-36d71762d705

nigurr on 25 Aug 2016

We're interested in bugs or errors found in the code, regardless of the source, but not in adopting a specific static analysis tool. Feel free to file specific bugs for problems (or send PRs with fixes!).