-
Notifications
You must be signed in to change notification settings - Fork 16
Prevent span start overflow panic #1373
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.
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
Conversation
Co-authored-by: VianneyRuhlmann <[email protected]>
|
Bits AI Dev Agent Status: ✅ Done [Fix CI Errors] You can ask for changes by mentioning @DataDog in a comment. |
This comment has been minimized.
This comment has been minimized.
BenchmarksComparisonBenchmark execution time: 2025-12-17 14:41:04 Comparing candidate commit aadcd56 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 57 metrics, 2 unstable metrics. CandidateCandidate benchmark detailsGroup 1
Group 2
Group 3
Group 4
Group 5
Group 6
Group 7
Group 8
Group 9
Group 10
Group 11
Group 12
Group 13
Group 14
Group 15
Group 16
Group 17
Group 18
Group 19
BaselineOmitted due to size. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1373 +/- ##
==========================================
- Coverage 71.45% 71.45% -0.01%
==========================================
Files 406 406
Lines 65203 65216 +13
==========================================
+ Hits 46590 46597 +7
- Misses 18613 18619 +6
🚀 New features to boost your workflow:
|
Artifact Size Benchmark Reportaarch64-alpine-linux-musl
aarch64-apple-darwin
aarch64-unknown-linux-gnu
libdatadog-x64-windows
libdatadog-x86-windows
x86_64-alpine-linux-musl
x86_64-apple-darwin
x86_64-unknown-linux-gnu
|
|
/merge |
|
View all feedbacks in Devflow UI.
This pull request is not mergeable according to GitHub. Common reasons include pending required checks, missing approvals, or merge conflicts — but it could also be blocked by other repository rules or settings.
The expected merge time in
|
for Dev Agent Session identified by Workflow Automation
You can ask for changes by mentioning @DataDog in a comment.
Feedback (especially what can be better) welcome in #code-gen-feedback!
What does this PR do?
i64::MINstart valuenormalize_span_start_durationwithchecked_addto avoid overflowing when computingstart + durationMotivation
A libFuzzer input produced an overflow panic in
normalize_span_start_durationwhenstartwas near negative, because we were subtracting fromi64::MAXbefore clamping the duration.Additional Notes
cargo test -p libdd-trace-normalizationcurrently cannot run in this sandbox because Cargo attempts to fetch the patchedproptestgit dependency from GitHub, but the environment has no network access.How to test the change?
Run
cargo test -p libdd-trace-normalization test_normalize_span_start_duration_handles_min_startin an environment that can download the workspace dependencies.