mirror of
https://github.com/oven-sh/bun
synced 2026-02-13 20:39:05 +00:00
31debe497b
Created separate JSYogaConfigOwner to properly handle YogaConfigImpl objects instead of incorrectly using JSYogaNodeOwner which expects YogaNodeImpl. The issue was: - YogaConfigImpl used jsYogaNodeOwner() WeakHandleOwner - JSYogaNodeOwner::isReachableFromOpaqueRoots cast context to YogaNodeImpl* - When called with YogaConfigImpl*, this caused type confusion and potential memory corruption Fix: - Created JSYogaConfigOwner with proper YogaConfigImpl handling - YogaConfigImpl now uses jsYogaConfigOwner() instead of jsYogaNodeOwner() - JSYogaConfigOwner doesn't use opaque roots (configs don't need them) Progress: - ✅ 2-3 yoga test files: All combinations work without ASAN errors - ❌ 4+ yoga test files: Still ASAN error, possibly related to yoga-node-extended.test.js 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>