pyroutingkit 0.2.0__tar.gz → 0.2.1__tar.gz

{pyroutingkit-0.2.0 → pyroutingkit-0.2.1}/.github/workflows/publish.yml

@@ -24,7 +24,10 @@ jobs:
           CIBW_BUILD: cp310-* cp311-* cp312-* cp313-*
           CIBW_SKIP: "*-win32 *-manylinux_i686 *musllinux* pp*"
           CIBW_MANYLINUX_X86_64_IMAGE: manylinux2014
-          CIBW_TEST_COMMAND: python -c "from pyroutingkit import CCH, CCHMetric, CCHTopology, HAS_OPENMP, INF_WEIGHT; print('OK, OpenMP:', HAS_OPENMP)"
+          # Run the full suite against every built wheel — the 0.2.0
+          # use-after-free only manifested on Linux release builds.
+          CIBW_TEST_REQUIRES: pytest
+          CIBW_TEST_COMMAND: pytest {project}/tests -q
 
       - uses: actions/upload-artifact@v4
         with:

{pyroutingkit-0.2.0 → pyroutingkit-0.2.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pyroutingkit
-Version: 0.2.0
+Version: 0.2.1
 Summary: Python bindings for RoutingKit Customizable Contraction Hierarchies (CCH)
 Author: Ryan Fisk
 License-Expression: MIT

{pyroutingkit-0.2.0 → pyroutingkit-0.2.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "pyroutingkit"
-version = "0.2.0"
+version = "0.2.1"
 description = "Python bindings for RoutingKit Customizable Contraction Hierarchies (CCH)"
 readme = "README.md"
 requires-python = ">=3.10"

{pyroutingkit-0.2.0 → pyroutingkit-0.2.1}/src/routingkit_module.cpp

@@ -198,7 +198,11 @@ public:
             (void)thread_count;
             metric_.customize();
 #endif
-            query_.reset(metric_);
+            // Construct, don't reset(): reset(metric) on a default-
+            // constructed query compares an UNINITIALIZED cch pointer; if
+            // freed memory is reused it can match and skip allocation of
+            // the query's internal arrays (out-of-bounds on first query).
+            query_ = RoutingKit::CustomizableContractionHierarchyQuery(metric_);
         }
         customized_ = true;
     }
@@ -238,7 +242,13 @@ public:
         {
             py::gil_scoped_release release;
             if (!partial_ready_) {
-                partial_.reset(topology_->cch());
+                // Construct, don't reset(): reset(cch) on a default-
+                // constructed object compares an UNINITIALIZED queue size;
+                // recycled heap memory can make it falsely match and skip
+                // allocating the queue (out-of-bounds on update_arc).
+                partial_ = RoutingKit::
+                    CustomizableContractionHierarchyPartialCustomization(
+                        topology_->cch());
                 partial_ready_ = true;
             }
             for (size_t i = 0; i < n; i++) {

{pyroutingkit-0.2.0 → pyroutingkit-0.2.1}/tests/test_topology_metric.py

@@ -226,3 +226,29 @@ def test_parallel_customize_matches_serial(topology):
         ds, _, _ = serial.query(s, t)
         dp, _, _ = parallel.query(s, t)
         assert ds == dp
+
+
+def test_metric_churn_with_partial_updates(topology):
+    """Regression: creating/destroying many metrics over one topology.
+
+    RoutingKit's X::reset(...) 'reuse' helpers compare possibly
+    UNINITIALIZED state of default-constructed objects (Query::reset
+    checks this->cch, PartialCustomization::reset checks q.id_count());
+    when the allocator recycles a destroyed wrapper's memory the check
+    can falsely match and skip allocating internal arrays — segfault on
+    first use. The wrapper must construct, not reset. Crashed ~100% on
+    Linux within a few iterations before the fix.
+    """
+    w = np.array([10] * 10, dtype=np.uint32)
+    for _ in range(50):
+        for tc in (0, 2, 4):
+            m = CCHMetric(topology)
+            m.customize(w, thread_count=tc)
+            d, _, _ = m.query(0, 3)
+            assert d == 10
+            m.update_weights(
+                np.array([8], dtype=np.uint32),
+                np.array([INF_WEIGHT], dtype=np.uint32),
+            )
+            d2, _, arcs = m.query(0, 3)
+            assert d2 == 20 and 8 not in arcs

{pyroutingkit-0.2.0 → pyroutingkit-0.2.1}/uv.lock

@@ -212,7 +212,7 @@ wheels = [
 
 [[package]]
 name = "pyroutingkit"
-version = "0.2.0"
+version = "0.2.1"
 source = { editable = "." }
 dependencies = [
     { name = "numpy", version = "2.2.6", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version < '3.11'" },