microsoft/Terminal

Let Terminal auto-detect a busted `AutoRun` value for CMD

Open

#15.658 aberto em 5 de jul. de 2023

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)C++ (3.212 forks)batch import
Area-TerminalConnectionArea-UserInterfaceHelp WantedIssue-TaskProduct-Terminal

Métricas do repositório

Stars
 (35.764 stars)
Métricas de merge de PR
 (Mesclagem média 27d 19h) (24 fundiu PRs em 30d)

Description

CMD has this magic AutoRun reg key that's a gigantic footgun.

  • HKEY_LOCAL_MACHINE\Software\Microsoft\Command Processor\AutoRun
  • HKEY_CURRENT_USER\Software\Microsoft\Command Processor\AutoRun

Basically, if someone writes a bad value to that, it can mess up CMD.exe real bad. As a real life example: https://github.com/conda/constructor/pull/521

We attempted to patch this in Windows 11. But we'll surely be unable to backport that. While we're maintaining Windows 10 support, it might be nice to return a specific warning for this in Terminal. Something like:

if (profileGuid == {cmd} && error == 1 && reg query ....) { write("Hey your AutoRun value might be busted. Go check these reg keys:"); }

(for discussion. We can always just say meh, since we did fix this for cmd.exe)

Guia do colaborador

Direção de pesquisa
Inspecione o código de inicialização do Terminal para encontrar onde o CMD.exe é iniciado como um perfil. Adicione uma verificação para as chaves de registro AutoRun (HKCU e HKLM) e exiba uma mensagem de aviso se um valor inválido for detectado. Certifique se de que a verificação se aplique apenas ao perfil CMD.
Pilha de tecnologia
cpp
Domain
desktop
Tipo Issue
Funcionalidade
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.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Windows RegistryGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75