Hacktoberfest 2026:維護者為十月標記出來的 issue,仍然開放、適合新手。 瀏覽 Hacktoberfest issue

ARM64 static native instrumentation: ADRP page mis-relocated for global loads when relocated data straddles a 4 KB page

未關閉
#255 0 則留言 0 個 reaction 已指派 0 人 在 GitHub 檢視

還沒有人認領這個 Issue。

評估

難度
5/5
預估耗時
一週以上
新手友好度
10/100
Issue 類型
缺陷
描述清晰度
基本清楚
活躍度
活躍
技術堆疊
cpp, csharp
領域
compilers

研究方向

Start with the attached repro.zip and run repro.ps1 on an ARM64 machine to see the 0xFF0 page-offset mismatch. The log message names CPEA64::FinalAssignAddressPass, the ARM64 pass where ADRP targets are relocated, so begin there. Done means each ADRP page is computed from its own load target (symbol plus addend), and repro.ps1 reports 0 mismatches when instrumented.

由索引模型根據 Issue 內容生成。

描述

Environment

  • Visual Studio 2026 Enterprise 18.10, MSVC 14.51.36231, Windows SDK 10.0.26100.0
  • Microsoft.CodeCoverage.Console 18.10.0-preview.26374.2
  • Target: native ARM64, Debug (/Od /RTC1 /GS /guard:cf /MDd), linked with /PROFILE, static native instrumentation
  • Run on Windows 11 ARM64 (26100)

Summary

After static instrumentation, .rdata contents move by 0x10 bytes. For a global struct whose fields end up on two
different 4 KB pages, the adrp instructions addressing that struct get the page of the struct's last field, while each
ldr keeps its own low-12-bit offset. Loads of fields that stayed on the earlier page therefore read memory exactly 4 KB
too high.

In our product this reads an unrelated qword instead of a function pointer and crashes with
FAST_FAIL_GUARD_ICALL_CHECK_FAILURE (0xC0000409), causes C0000005 while copying a global std::set, and produces
"Failed to load the test assembly or its dependencies" in vstest. The same binaries pass without instrumentation, and the
same sources pass under x64 static and dynamic instrumentation.

Repro (attached repro.zip, toy code only)

repro.zip

repro.ps1 generates 3000 const 32-byte structs, each checked field by field by its own function, builds for ARM64 with
/PROFILE, instruments with Microsoft.CodeCoverage.Console instrument, and runs both binaries on an ARM64 machine
(repro.ps1 -BuildOnly on x64, then repro.ps1 -RunOnly on ARM64, or just repro.ps1 on ARM64):

--- uninstrumented:
checked 3000 globals, 0 mismatches
--- instrumented, inside a coverage session:
const 87 at 00007FF6FCEA7FF0: aaaa0000000005d7 bbbb0000000005d7 cccc0000000005d7 dddd0000000005d7
const 215 at 00007FF6FCEAAFF0: aaaa000000000657 bbbb000000000657 cccc000000000657 dddd000000000657
const 316 at 00007FF6FCEACFF0: aaaa00000000017c bbbb00000000017c cccc00000000017c dddd00000000017c
...

Every failing struct sits at page offset 0xFF0 after instrumentation. The address-of (&c_N, adrp + add) is
mis-relocated the same way: const 87 prints the contents of struct 0x5D7, the object 4 KB above it. Both the field
loads and the address computation are 4 KB too high.

Disassembly (check_87, from the attached repro build)

Original - c_87 at 0x140090FE0, all four fields on page 0x140090000:

adrp x8, 0x140090000 ; ldr x8,[x8,#0xFE0]    ; f0
adrp x8, 0x140090000 ; ldr x8,[x8,#0xFE8]    ; f1
adrp x8, 0x140090000 ; ldr x8,[x8,#0xFF0]    ; f2
adrp x8, 0x140090000 ; ldr x8,[x8,#0xFF8]    ; f3
adrp x8, 0x140090000 ; add x1,x8,#0xFE0      ; &c_87

Instrumented - c_87 moved to 0x1401A6FF0 (f0, f1 on page 0x1401A6000; f2, f3 on page 0x1401A7000):

adrp x8, 0x1401A7000 ; ldr x8,[x8,#0xFF0]    ; f0    -> 0x1401A7FF0, expected 0x1401A6FF0  (WRONG, +4 KB)
adrp x8, 0x1401A7000 ; ldr x8,[x8,#0xFF8]    ; f1    -> 0x1401A7FF8, expected 0x1401A6FF8  (WRONG, +4 KB)
adrp x8, 0x1401A7000 ; ldr x8,[x8]           ; f2    -> 0x1401A7000, expected 0x1401A7000  (ok)
adrp x8, 0x1401A7000 ; ldr x8,[x8,#8]        ; f3    -> 0x1401A7008, expected 0x1401A7008  (ok)
adrp x8, 0x1401A7000 ; add x1,x8,#0xFF0      ; &c_87 -> 0x1401A7FF0, expected 0x1401A6FF0  (WRONG, +4 KB)

Every adrp that originally pointed at the struct's page gets the same new page (the page of the struct's last field),
while each low-12-bit offset is correctly shifted by +0x10. Accesses whose relocated target stayed on the earlier page
therefore land exactly 4 KB too high.

Expected

Each adrp page is computed from its own load target (symbol + addend), so the relocated code reads the same objects as
the original.

Also observed (possibly related)

  • Every ARM64 /GS function has bl __security_push_cookie inside its prolog. The instrumenter prints
    "CPEA64::FinalAssignAddressPass: prolog unwind codes mismatch" for each one and inserts a NOP plus a probe into the
    prolog. On large binaries this is millions of output lines and dominates the run time.
  • For CRT varargs functions (e.g. _snprintf_s) the unwind code for stp x4,x5,[sp,#-0x20]! is rewritten to nop
    (dumpbin /unwindinfo: "Nop covers for stack-adjusting load/store").

Full crash dumps and binaries from the original product are available to Microsoft engineers on request.

主要語言
C#
星號
126
分支
17
平均合併
1 小時 13 分鐘
30 天內合併 PR
2

環境準備

這個專案沒有提供開發容器、Dockerfile 或貢獻指南,環境需要你自己搭建:先看它的 README,通用步驟見我們的新手貢獻指南。

從這裡開始

  1. 先讀完整個 Issue,再讀專案的貢獻指南。
  2. 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
  3. Fork 儲存庫,在一個分支上完成修改。
  4. 送出 Pull Request,並在描述裡引用這個 Issue 編號。

microsoft/codecoverage 的其他 Issue

查看 microsoft/codecoverage 的全部 Issue

相似的 Issue

更多 C# Issue

把新 issue 寄到你的電子郵件信箱

精選適合新手參與的 GitHub issue 摘要。