[BUG] device_info() buffers are 64 bytes; CUDA backend writes 257, corrupting the stack
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 76/100
Direção de pesquisa
Comece em src/core/device.rs:109-121 e inspecione os buffers passados para device_info(); compare-os com o contrato documentado e com o comportamento do CUDA descrito em src/backend/cuda/platform.cpp:290-307. Execute examples/helloworld.rs com o backend CUDA para confirmar que a chamada é segura e considere a issue concluída quando device_info() não permitir mais que o backend CUDA sobrescreva a stack do chamador.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Description
device_info() allocates the buffer sizes recommended by the ArrayFire documentation, but the CUDA backend writes up to 257 bytes into the first one. On the CUDA backend this corrupts up to 193 bytes of the caller's stack on every call.
This is a plausible root cause for several long-standing crash reports here: #106, #285, #311.
pub fn device_info() -> (String, String, String, String) {
let mut name: [c_char; 64] = [0; 64];
let mut platform: [c_char; 10] = [0; 10];
let mut toolkit: [c_char; 64] = [0; 64];
let mut compute: [c_char; 10] = [0; 10];
unsafe {
let err_val = af_device_info(
&mut name[0], &mut platform[0], &mut toolkit[0], &mut compute[0],
);
These sizes match ArrayFire's documented contract exactly (docs/details/device.dox:10-16, "Recommended minimum size is 64 / 10 / 64 / 10" across the four params), and af_device_info takes no length arguments, so there is nothing else to go on.
The CUDA backend does not honour it (src/backend/cuda/platform.cpp:290-307, identical from 3.8.0 through master):
snprintf(d_name, 256, "%s", dev.name);
// Sanitize input
for (int i = 0; i < 256; i++) {
if (d_name[i] == ' ') { // reads d_name[0..256]
if (d_name[i + 1] == 0 || d_name[i + 1] == ' ') {
d_name[i] = 0; // writes d_name[0..255]
} else {
d_name[i] = '_';
}
}
}
The sanitize loop does not stop at the NUL terminator, so it runs the full 256 iterations regardless of the actual device-name length, writing 0x00 or '_' wherever it reads a 0x20 byte. The overflow therefore is not conditional on having a long GPU name — it happens on every call.
The CPU and OpenCL backends are correct here (snprintf(..., 64, ...), and OpenCL bounds its sanitize loop at i < 31), so this affects the CUDA backend only — which matches the reported pattern of crashes that disappear when users switch to CPU or OpenCL.
I've filed the backend-side bug upstream as arrayfire/arrayfire#<CPP_ISSUE>.
Impact
Writing ~193 bytes past a stack array clobbers the other three buffers, spilled registers, the /GS cookie and the return address. On Windows that surfaces as STATUS_ACCESS_VIOLATION (0xC0000005) or STATUS_STACK_BUFFER_OVERRUN (0xC0000409).
The corruption is deterministic but the crash is intermittent, because whether it is fatal depends on which stack bytes happen to contain 0x20 and on the frame layout rustc chose. That also gives a mechanism for #299 (crash only at opt-level = 3): more aggressive inlining means less dead stack padding to absorb the overflow.
examples/helloworld.rs:9 calls device_info(), so this is in the first program a new user runs.
Reproducible Code and/or Steps
see above
System Information
general code issue, system independent
Checklist
- Using the latest available ArrayFire release
- GPU drivers are up to date
Suggested fix
Independent of any upstream change, and safe against every 3.8.x:
let mut name: [c_char; 1024] = [0; 1024];
(or at least 257)
Happy to open a PR if that's useful.
Disclaimer
Found by Claude Opus 5 while investigating intermittent 0xC0000005 errors on Windows/CUDA. I checked this manually and it seems like a real bug.
- Linguagem predominante
- Rust
- Estrelas
- 827
- Forks
- 59
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Guia de contribuição
Nenhum guia de contribuição indexado para este repositório
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 arrayfire/arrayfire-rust
-
Bug
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 88/100
arrayfire/arrayfire-rust#388 ·
-
Bug
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 88/100
arrayfire/arrayfire-rust#387 ·
-
Bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
arrayfire/arrayfire-rust#386 ·
-
Bug
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 35/100
arrayfire/arrayfire-rust#385 · 1 comentário ·
-
Bug
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 45/100
arrayfire/arrayfire-rust#382 ·
Todas as issues de arrayfire/arrayfire-rust
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
Eynzof/Hermes-CN-Desktop#610 ·
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
gitbutlerapp/gitbutler#15998 · 1 comentário ·
-
bug triage:deciding
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 88/100
open-telemetry/otel-arrow#4132 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100