EnsureProcessIsClosed function missing correct access rights and does not close handle

Abierto
#249 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
58/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Estancado
Stack tecnológico
cpp
Área
desktop

Línea de trabajo

Comienza en SampleApps/WebView2APISample/ProcessComponent.cpp alrededor de la línea 554 e inspecciona EnsureProcessIsClosed, incluida la llamada adyacente a OpenProcess y el uso de WaitForSingleObject. Confirma que el identificador del proceso solicita los derechos necesarios, que se gestiona el fallo de OpenProcess y que el identificador se cierra en todos los caminos; valida el comportamiento compilando y ejecutando el flujo de limpieza del proceso del sample.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

The process handle that gets passed to WaitForSingleObject in this line is missing the SYNCHRONIZE access right as specified here. This needs to be requested in the adjacent OpenProcess call alongside the PROCESS_TERMINATE access right.

This results in the WaitForSingleObject call to always immediately return with a result that always leads to the explicit forceful termination code path. This fact hides the other issue with this function that the opened process handle is only closed in this code path when it should be closed outside the if.

Furthermore, the result of OpenProcess should be evaluated as it might be NULL.

Nothing breaking, just caused some confusion on our side as we took this function as an example on how to wait for the web view process to finish before doing some clean up on our side.

Lenguaje dominante
C++
Estrellas
1k
Forks
521
Merge medio
13 min
PR fusionados (30 d)
1

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de MicrosoftEdge/WebView2Samples

Todos los issues de MicrosoftEdge/WebView2Samples

Issues similares

Más issues de C++

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.