diff options
| author | Paul Buetow <paul@buetow.org> | 2026-05-30 16:35:39 +0300 |
|---|---|---|
| committer | Paul Buetow <paul@buetow.org> | 2026-05-30 16:35:39 +0300 |
| commit | a74da2f75d8088793ee6f7e2cc46ba19803a350f (patch) | |
| tree | 0368c226edc4f53c0fb98ddd94e98646a7a69404 /internal/generate/testdata.go | |
| parent | 5210f20b16a224f58117ffb71b2a0691c02a50ed (diff) | |
test(generate): lock in bind syscall classification
Audit of bind(2): int bind(int sockfd, const struct sockaddr *addr,
socklen_t addrlen). Verified the existing classification is correct and
consistent with its socket-setup siblings connect/listen/accept/
getsockname/getpeername:
- KindFd, capturing ev->fd = args[0] (the sockfd); the addr pointer and
addrlen are not captured.
- FamilyNetwork.
- Exit is UNCLASSIFIED (returns 0/-1, no transferred byte count).
No implementation or doc changes were needed (docs/syscall-tracing-plan.md
already lists bind under Network and fd; drift test green). Added
regression coverage:
- FormatBind/FormatExitBind fixtures mirroring the real kernel tracepoint.
- TestGenerateBindHandler with negative guards (no probe_read on the
sockaddr, no fd capture from args[1]/args[2], exit stays UNCLASSIFIED).
- bind + connect/listen/getsockname/getpeername added to the
family (FamilyNetwork) and ret-classification (UNCLASSIFIED) lock-in
lists.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Diffstat (limited to 'internal/generate/testdata.go')
| -rw-r--r-- | internal/generate/testdata.go | 35 |
1 files changed, 35 insertions, 0 deletions
diff --git a/internal/generate/testdata.go b/internal/generate/testdata.go index 6555fc7..f26234f 100644 --- a/internal/generate/testdata.go +++ b/internal/generate/testdata.go @@ -2156,3 +2156,38 @@ format: print fmt: "0x%lx", REC->ret ` + +// FormatBind / FormatExitBind mirror the real kernel tracepoint format for +// bind(2): int bind(int sockfd, const struct sockaddr *addr, socklen_t addrlen). +// The leading "fd" field (sockfd at args[0]) makes it a KindFd fd_event; the +// addr pointer and addrlen must NOT be captured. On exit bind returns 0/-1, +// which is UNCLASSIFIED (a plain ret_event, no read/write/transfer byte count). +const FormatBind = `name: sys_enter_bind +ID: 1843 +format: + field:unsigned short common_type; offset:0; size:2; signed:0; + field:unsigned char common_flags; offset:2; size:1; signed:0; + field:unsigned char common_preempt_count; offset:3; size:1; signed:0; + field:int common_pid; offset:4; size:4; signed:1; + + field:int __syscall_nr; offset:8; size:4; signed:1; + field:int fd; offset:16; size:8; signed:0; + field:struct sockaddr * umyaddr; offset:24; size:8; signed:0; + field:int addrlen; offset:32; size:8; signed:0; + +print fmt: "fd: 0x%08lx, umyaddr: 0x%08lx, addrlen: 0x%08lx", ((unsigned long)(REC->fd)), ((unsigned long)(REC->umyaddr)), ((unsigned long)(REC->addrlen)) +` + +const FormatExitBind = `name: sys_exit_bind +ID: 1842 +format: + field:unsigned short common_type; offset:0; size:2; signed:0; + field:unsigned char common_flags; offset:2; size:1; signed:0; + field:unsigned char common_preempt_count; offset:3; size:1; signed:0; + field:int common_pid; offset:4; size:4; signed:1; + + field:int __syscall_nr; offset:8; size:4; signed:1; + field:long ret; offset:16; size:8; signed:1; + +print fmt: "0x%lx", REC->ret +` |
