DynamoRIO/drmemory

Test "handle" fails on Win7SP1x64/VS2012

Open

#1.493 aberto em 28 de nov. de 2014

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)C (289 forks)auto 404
Component-TestsMigratedOpSys-WindowsPriority-Mediumhelp wanted

Métricas do repositório

Stars
 (2.727 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

From byron.c....@gmail.com on March 26, 2014 12:49:54

In the pre-commit suite on Windows 7 SP 1, when building r1812 in 64-bit with VS 2012, the test "handle" finds a different error than expected:

5: C:\dev\dynamorio\pre-commit\build_drmemory-dbg-64\logs\DrMemory-handle.exe.2548.000\results.txt 5: failed to match "Error #16: HANDLE LEAK:", found "Error #10: HANDLE LEAK:" 5: instead

Logs for debug and release are attached.

Attachment: handle.debug-64.log handle.release-64.log

Original issue: http://code.google.com/p/drmemory/issues/detail?id=1493

Guia do colaborador

Direção de pesquisa
Investigue por que o teste de vazamento de handles relata o erro #10 em vez de #16 no Windows 7 SP1 x64 com VS2012. Compare as expectativas do teste com o comportamento real nessa plataforma e considere se a diferença na numeração dos erros se deve ao gerenciamento de handles específico da plataforma ou ao ambiente de teste.
Pilha de tecnologia
c
Domain
testingbackend
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.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
CWindows debuggingVisual Studio 2012
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60