dotnet/msbuild

Prevent infinite recursion in Solution parser when a project is nested under itself

Open

#1.599 aberto em 22 de jan. de 2017

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)C# (1.364 forks)batch import
Area: Solution (.sln)help wantedtriaged

Métricas do repositório

Stars
 (5.062 stars)
Métricas de merge de PR
 (Mesclagem média 11d 7h) (125 fundiu PRs em 30d)

Description

We don't check for invalid project nestings in the solution file, so it's possible to create a solution that crashes MSBuild with stack overflow:

https://github.com/Microsoft/msbuild/blob/xplat/src/XMakeBuildEngine/Construction/Solution/ProjectInSolution.cs#L393

An example solution is: https://github.com/dotnet/corefx/blob/f6ab339f446e37505f93ef9f430801fa1dcc05b8/src/System.Collections.Immutable/System.Collections.Immutable.sln#L40-L41

Note that we should check for indirect recursion as well (loops).

Guia do colaborador

Direção de pesquisa
Inspecione o arquivo ProjectInSolution.cs perto da linha 393 (vinculada na issue) e rastreie o caminho de análise dos projetos. Implemente a detecção de ciclos usando um conjunto visitado ou ordenação topológica para evitar recursão infinita.
Pilha de tecnologia
csharp
Domain
build system
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#MSBuild solution file format
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50