From 88b2b690c70e6d882508b963ce3b8ee69aa394a5 Mon Sep 17 00:00:00 2001 From: Jaehyun Cho Date: Thu, 27 Oct 2022 11:28:39 +0900 Subject: [PATCH] [NUI] Fix crash on ScrollableBase.Dispose() In ScrollableBase.Dispose(), propertyNotification is disposed as well. To dispose propertyNotification, RemovePropertyNotifications was called inappropriately and it caused crash. To resolve the above, RemovePropertyNotification is called by ContentContainer with propertyNotification like AddPropertyNotification is called by ContentContainer with propertyNotification. --- src/Tizen.NUI.Components/Controls/ScrollableBase.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Tizen.NUI.Components/Controls/ScrollableBase.cs b/src/Tizen.NUI.Components/Controls/ScrollableBase.cs index bfac755..0f2f477 100755 --- a/src/Tizen.NUI.Components/Controls/ScrollableBase.cs +++ b/src/Tizen.NUI.Components/Controls/ScrollableBase.cs @@ -1342,8 +1342,8 @@ namespace Tizen.NUI.Components if (propertyNotification != null) { + ContentContainer?.RemovePropertyNotification(propertyNotification); propertyNotification.Notified -= OnPropertyChanged; - Interop.Handle.RemovePropertyNotifications(propertyNotification.SwigCPtr); propertyNotification.Dispose(); propertyNotification = null; } -- 2.7.4