提交 #945051: tdunning t-digest 3.3 Unvalidated NaN Centroid Means Cause Quadratic Blowup信息

标题tdunning t-digest 3.3 Unvalidated NaN Centroid Means Cause Quadratic Blowup
描述While digging further into MergingDigest.fromBytes() past the array-bounds issue I already reported, I noticed it also skips the one check every other write path in this class enforces: rejecting NaN means. MergingDigest.add(double, int) throws on NaN, and even the sibling AVLTreeDigest.fromBytes() routes through add() so it inherits that check for free — but MergingDigest.fromBytes() writes straight into the backing arrays and never calls add(), so a digest can be deserialized with an arbitrary number of NaN means baked in. The very next merge of that digest — a single add(x, w) call, exactly what a real aggregator does with each new raw value — hands those NaNs to Sort.quickSort(), whose median-of-three pivot logic silently degrades from O(n log n) to O(n²) once comparisons stop forming a total order.
来源⚠️ https://github.com/tdunning/t-digest/issues/229
用户
 Yu_Bao from PayPal Cyber Security Team (UID 100783)
提交2026-08-24 18時41分 (2 月前)
管理2026-09-19 19時04分 (26 days later)
状态重复
VulDB条目401776 [tdunning t-digest 直到 3.3 MergingDigest.fromBytes 权限提升]
积分0

Want to stay up to date on a daily basis?

Enable the mail alert feature now!