ci(lx-ci): vet with -unsafeptr=false — upstream debug-crash trips unsafeptr
The 1.14 merge (6b63cee4) brought daemon/managed_service.go and
experimental/libbox/debug.go into vet scope; both crash Go ON PURPOSE via
*(*int)(unsafe.Pointer(uintptr(0)))=0 (TriggerDebugCrash/TriggerGoPanic),
which vet's unsafeptr analyzer flags. They are upstream files we don't edit,
and no lx-owned file uses unsafe at all, so disable just that analyzer.
Fixes the red lint job on every push since the merge.
This commit is contained in:
@@ -51,7 +51,14 @@ jobs:
|
||||
|
||||
- name: go vet (lx packages, full tags)
|
||||
run: |
|
||||
go vet -tags "${BASE_TAGS},with_xhttp,with_awg,with_lx_command" \
|
||||
# -unsafeptr=false: upstream's TriggerDebugCrash/TriggerGoPanic
|
||||
# (daemon/managed_service.go, experimental/libbox/debug.go — came in with
|
||||
# the 1.14 merge) crash Go ON PURPOSE via *(*int)(unsafe.Pointer(uintptr(0)))=0,
|
||||
# which trips vet's unsafeptr check. Those are upstream files we don't edit
|
||||
# (CONSTITUTION §zero-diff), and no lx-owned file uses unsafe at all, so
|
||||
# disabling this one analyzer costs nothing on our own code.
|
||||
go vet -unsafeptr=false \
|
||||
-tags "${BASE_TAGS},with_xhttp,with_awg,with_lx_command" \
|
||||
./option/ ./constant/ ./transport/v2ray/ ./transport/v2rayxhttp/ \
|
||||
./protocol/wireguard/ ./transport/wireguard/ \
|
||||
./daemon/ ./experimental/libbox/
|
||||
|
||||
Reference in New Issue
Block a user