Skip to content

Integrate GoOffsetLocator in socket tracer #2212

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
ddelnano wants to merge 12 commits into
base: main
Choose a base branch
from
Draft

Integrate GoOffsetLocator in socket tracer #2212

ddelnano wants to merge 12 commits into from

Conversation

@ddelnano
Copy link
Member

@ddelnano ddelnano commented Jun 8, 2025

Summary: Integrate GoOffsetLocator in socket tracer

More details to be filled in later

Relevant Issues:

Type of change:

Test Plan:

Changelog Message:

Additional documentation:

ddelnano added 12 commits June 4, 2025 21:05
@ddelnano
Add GoOffsetLocator that fulfills DwarfReader APIs
0fe7a1b 
Signed-off-by: Dom Del Nano <ddelnano@gmail.com>
@ddelnano
Replace DwarfReader with GoOffsetLocator to facilliate future, lower …
d3b8bb5 
…overhead (memory) Go uprobes

Signed-off-by: Dom Del Nano <ddelnano@gmail.com>
@ddelnano
Attach uprobes even if buildinfo cannot be read
ac26068 
Signed-off-by: Dom Del Nano <ddelnano@gmail.com>
@ddelnano
Fix compilation issues with src/stirling/binaries/go_binary_parse_pro…
d86baa9 
…filing

Signed-off-by: Dom Del Nano <ddelnano@gmail.com>
@ddelnano
Reuse the go version from GoOffsetLocator to avoid parsing build info…
f427177 
… multiple times

Signed-off-by: Dom Del Nano <ddelnano@gmail.com>
@ddelnano
Update GoOffsetTracker to use correct version key for a given module
ef6e65f 
Signed-off-by: Dom Del Nano <ddelnano@gmail.com>
@ddelnano
Add offsetgen file parsing into uprobe manager
02ad26d 
Signed-off-by: Dom Del Nano <ddelnano@gmail.com>
@ddelnano
Use string_view for package names to reduce copies of duplicate strings
e0b1740 
Signed-off-by: Dom Del Nano <ddelnano@gmail.com>
@ddelnano
Fix compilation error in go_binary_parse_profiling
7ce56b9 
Signed-off-by: Dom Del Nano <ddelnano@gmail.com>
@ddelnano
Improve log message and change log level. Remove commented out test
200d241 
Signed-off-by: Dom Del Nano <ddelnano@gmail.com>
@ddelnano
Integrate offsetgen based uprobes into socket tracer. Add go_tls_trac…
0ad34ce 
…e_bpf_test for offsetgen case

Signed-off-by: Dom Del Nano <ddelnano@gmail.com>
@ddelnano
Include offsetgen case in http2_trace_bpf_test
345dddc 
Signed-off-by: Dom Del Nano <ddelnano@gmail.com>
github-advanced-security[bot]
github-advanced-security bot found potential problems Jun 9, 2025
View reviewed changes
src/stirling/testing/demo_apps/go_grpc_tls_pl/server/mod_info/go.mod
@@ -0,0 +1,40 @@
module grpc

Check failure

Code scanning / trivy-fs

gRPC-Go HTTP/2 Rapid Reset vulnerability High test

Package: google.golang.org/grpc
Installed Version: v1.43.0
Vulnerability GHSA-m425-mq94-257g
Severity: HIGH
Fixed Version: 1.56.3, 1.57.1, 1.58.3
Link: GHSA-m425-mq94-257g
src/stirling/testing/demo_apps/go_grpc_tls_pl/server/mod_info/go.mod
@@ -0,0 +1,40 @@
module grpc

Check warning

Code scanning / trivy-fs

HTTP/2: Multiple HTTP/2 enabled web servers are vulnerable to a DDoS attack (Rapid Reset Attack) Medium test

Package: google.golang.org/grpc
Installed Version: v1.43.0
Vulnerability CVE-2023-44487
Severity: MEDIUM
Fixed Version: 1.58.3, 1.57.1, 1.56.3
Link: CVE-2023-44487
@ddelnano ddelnano mentioned this pull request Jun 10, 2025
Merged
2 tasks
@ddelnano
Copy link
Member Author

ddelnano commented Jun 23, 2025

Reminder for myself to use the following images:

  - ghcr.io/pixie-io/golang_1_18_https_server_with_buildinfo:1.0@sha256:d4468289a2a4f2520effc48dd8894c998867b6638953302ef4156983dbf6e4e5
  - ghcr.io/pixie-io/golang_1_21_https_server_with_buildinfo:1.0@sha256:abb4573171c0a46afd4c704d8bbd6d123f022c5ad0448869311b026d0586106b
  - ghcr.io/pixie-io/golang_1_19_https_server_with_buildinfo:1.0@sha256:8a35dab46ccd9af53704f8c83a8d3b28bf142f8e038696b2ba30b7df1c46df73
  - ghcr.io/pixie-io/golang_1_20_https_server_with_buildinfo:1.0@sha256:c21f14e2213f4d9826926eeb3a0e7c7445789539561b43981bbca3efafcad8d7
  - ghcr.io/pixie-io/golang_1_22_https_server_with_buildinfo:1.0@sha256:0f14818a1046dfdb7d5ac27e173d99e071219897a1d9969c8d7604acbd0d9541

  - ghcr.io/pixie-io/golang_1_18_grpc_server_with_buildinfo:1.0@sha256:e59a94abb2c505f680829951161a0a0209de0d02307da951c5a76d7b84a72034
  - ghcr.io/pixie-io/golang_1_21_grpc_server_with_buildinfo:1.0@sha256:998654168565a7d4ed1c73edbafc48042879ac26581972a8ba23fe5a39e520f2
  - ghcr.io/pixie-io/golang_1_19_grpc_server_with_buildinfo:1.0@sha256:88b1ae2096af629152a645f77b9c47dcac45dde95fb78376c6fdf1f7183a652b
  - ghcr.io/pixie-io/golang_1_20_grpc_server_with_buildinfo:1.0@sha256:a35e24a44c53d6f984eb633bbb15e318374a61fa426ccc5e580d587ce7f6a788
  - ghcr.io/pixie-io/golang_1_22_grpc_server_with_buildinfo:1.0@sha256:67adba5e8513670fa37bd042862e7844f26239e8d2997ed8c3b0aa527bc04cc3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@ddelnano