module: compile cache leaks temporary files when persistence fails
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 74/100
- Tipo de issue
- Bug
- Clareza
- Claramente especificada
- Status de atividade
- Ativa
- Stack de tecnologia
- cpp, javascript, nodejs
- Domínio
- backend, performance, testing
Direção de pesquisa
Comece em src/compile_cache.cc por volta das linhas 478-516 e execute a reprodução fornecida de RLIMIT_FSIZE para rastrear o tratamento de arquivos temporários através das falhas de write, close e rename. Adicione um teste de regressão nas plataformas POSIX compatíveis que verifique se uma persistência malsucedida não deixa arquivos temporários com sufixo aleatório. Está concluído quando as gravações de cache malsucedidas fecham os descritores, removem os caminhos temporários e preservam a falha controlada do cache.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Version
v24.15.0
Platform
Linux pike 7.0.0-29-generic #29-Ubuntu SMP PREEMPT_DYNAMIC Fri Jul 17 20:52:35 UTC 2026 x86_64 GNU/Linux
Subsystem
module compile cache
What steps will reproduce the bug?
This uses RLIMIT_FSIZE=0 to make mkstemp() succeed and the subsequent write fail. It requires no third-party dependencies or elevated privileges.
tmp="$(mktemp -d)"
mkdir -p "$tmp/cache"
printf '%s\n' 'module.exports = 42;' >"$tmp/fixture.cjs"
(
cd "$tmp"
ulimit -f 0
trap '' XFSZ
NODE_COMPILE_CACHE="$tmp/cache" node -e "require('./fixture.cjs')"
)
find "$tmp/cache" -type f -printf '%P %s bytes\n'
Example output:
v24.15.0-x64-cf738c9d-1000/712488c2.LuhsmB 0 bytes
v24.15.0-x64-cf738c9d-1000/8e9c99c2.hwlSas 0 bytes
v24.15.0-x64-cf738c9d-1000/9fb526e9.wPP9zO 0 bytes
The Node process exits with status 0.
How often does it reproduce? Is there a required condition?
Every run on the Linux system above. The required condition is that creation of the temporary cache file succeeds but a later write, close, or rename fails. EDQUOT, ENOSPC, and EFBIG are representative failure modes.
What is the expected behavior? Why is that the expected behavior?
The compile cache should continue to fail gracefully, but it should close the temporary file descriptor and unlink the temporary file whenever persistence does not complete. A failed cache write is disposable internal state and should not consume additional filesystem space or inodes.
What do you see instead?
Each failed entry leaves its random-suffix temporary file in the compile-cache directory. On the write-error path, its file descriptor also remains open until process teardown.
In a real EDQUOT incident, one Node process left 3,424 zero-byte compile-cache temporary files in one minute. This amplified a block-quota failure into substantial inode consumption.
Additional information
Current main creates the temporary file, then immediately continues on write, close, or rename errors without unlinking it:
The same control flow is present in v24.15.0.
A possible fix is an RAII cleanup guard established immediately after successful uv_fs_mkstemp() that:
- closes the descriptor if it is still open;
- unlinks the temporary path unless the rename succeeded;
- is disarmed only after successful rename;
- preserves the original persistence error.
A regression test can use the same RLIMIT_FSIZE technique on supported POSIX platforms and assert that no random-suffix temporary files remain.
- Linguagem predominante
- JavaScript
- Estrelas
- 122k
- Forks
- 37.4k
- Merge médio
- 4d 3h
- PRs com merge (30d)
- 279
Guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de nodejs/node
-
doc
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
-
build
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 88/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 90/100
-
feature request
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
Todas as issues de nodejs/node
Issues semelhantes
-
bug confirmed issue
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
open-webui/open-webui#30750 · 1 comentário ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
-
Mend: dependency security vulnerability untriaged
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100