fixup! fix: remove lvgl event callback on component deletion This cau…

…se a segfault

Previous fix was applied to a location where the underlying lv object could already have been deleted.
And therefore, could cause a segfault.

src/render/native/components/component.hpp

@@ -284,7 +284,10 @@ void NativeComponentMaskInit (JSContext* ctx, JSValue ns);
 #define WRAPPED_JS_CLOSE_COMPONENT(COMPONENT,COMPONENT_NAME) \
  static JSValue NativeCompCloseComponent(JSContext *ctx, JSValueConst this_val, int argc, JSValueConst *argv) { \
  COMP_REF* s = (COMP_REF*)JS_GetOpaque(this_val, COMPONENT##ClassID); \
- lv_obj_del_async(((BasicComponent*)(s->comp))->instance); \
+ lv_obj_t* instance = ((BasicComponent*)(s->comp))->instance; \
+ ((BasicComponent*)(s->comp))->instance = nullptr; \
+ lv_obj_remove_event_cb(instance, &BasicComponent::EventCallback); \
+ lv_obj_del_async(instance); \
  return JS_UNDEFINED; \
  } \
  \

src/render/native/core/basic/comp.cpp

@@ -234,13 +234,11 @@ void BasicComponent::setBackgroundImage (uint8_t* buf, size_t buf_len, int32_t s
 };
 
 BasicComponent::~BasicComponent () {
- removeEventListener(0);
-
  comp_map.erase(this->uid);
 
  const lv_coord_t* ptr1 = this->grid_row_desc;
  const lv_coord_t* ptr2 = this->grid_column_desc;
-
+ 
  for(auto& desc : this->image_desc_map) {
  if (desc.second != nullptr) {
  const uint8_t* buf = (static_cast<lv_img_dsc_t_1*>(desc.second))->data;