context UPDATE move some data from priv to shared

Author: Roytak

src/context.c

@@ -614,7 +614,7 @@ LIBYANG_API_DEF LY_ERR
 ly_ctx_set_options(struct ly_ctx *ctx, uint32_t option)
 {
     LY_ERR lyrc = LY_SUCCESS;
-    struct ly_ctx_private_data *ctx_data;
+    struct ly_ctx_shared_data *ctx_data;
     struct lys_module *mod;
     uint32_t i;
 
@@ -637,7 +637,8 @@ ly_ctx_set_options(struct ly_ctx *ctx, uint32_t option)
     }
 
     if (!(ctx->opts & LY_CTX_LEAFREF_LINKING) && (option & LY_CTX_LEAFREF_LINKING)) {
-        ctx_data = ly_ctx_private_data_get(ctx);
+        /* context is not printed (ctx data cant be shared with any other ctx), so no need to hold a lock */
+        ctx_data = ly_ctx_shared_data_get(ctx);
         ctx_data->leafref_links_ht = lyht_new(1, sizeof(struct lyd_leafref_links_rec *), ly_ctx_ht_leafref_links_equal_cb, NULL, 1);
         LY_CHECK_ERR_RET(!ctx_data->leafref_links_ht, LOGARG(ctx, option), LY_EMEM);
     }
@@ -699,13 +700,14 @@ ly_ctx_unset_options(struct ly_ctx *ctx, uint32_t option)
     LY_ARRAY_COUNT_TYPE u, v;
     const struct lysc_ext_instance *ext;
     struct lysc_node *root;
-    struct ly_ctx_private_data *ctx_data;
+    struct ly_ctx_shared_data *ctx_data;
 
     LY_CHECK_ARG_RET(ctx, ctx, !(ctx->opts & LY_CTX_INT_IMMUTABLE), LY_EINVAL);
     LY_CHECK_ERR_RET(option & LY_CTX_NO_YANGLIBRARY, LOGARG(ctx, option), LY_EINVAL);
 
     if ((ctx->opts & LY_CTX_LEAFREF_LINKING) && (option & LY_CTX_LEAFREF_LINKING)) {
-        ctx_data = ly_ctx_private_data_get(ctx);
+        /* context is not printed (ctx data cant be shared with any other ctx), so no need to hold a lock */
+        ctx_data = ly_ctx_shared_data_get(ctx);
         lyht_free(ctx_data->leafref_links_ht, ly_ctx_ht_leafref_links_rec_free);
         ctx_data->leafref_links_ht = NULL;
     }
@@ -828,16 +830,23 @@ ly_ctx_set_module_imp_clb(struct ly_ctx *ctx, ly_module_imp_clb clb, void *user_
 LIBYANG_API_DEF ly_ext_data_clb
 ly_ctx_set_ext_data_clb(const struct ly_ctx *ctx, ly_ext_data_clb clb, void *user_data)
 {
-    struct ly_ctx_private_data *ctx_data;
+    struct ly_ctx_shared_data *ctx_data;
     ly_ext_data_clb prev;
 
     LY_CHECK_ARG_RET(ctx, ctx, NULL);
 
-    ctx_data = ly_ctx_private_data_get(ctx);
+    ctx_data = ly_ctx_shared_data_get(ctx);
+
+    /* EXT CLB LOCK */
+    pthread_mutex_lock(&ctx_data->ext_clb_lock);
+
     prev = ctx_data->ext_clb;
     ctx_data->ext_clb = clb;
     ctx_data->ext_clb_data = user_data;
 
+    /* EXT CLB UNLOCK */
+    pthread_mutex_unlock(&ctx_data->ext_clb_lock);
+
     return prev;
 }
 

src/log.c

@@ -183,7 +183,8 @@ ly_err_first(const struct ly_ctx *ctx)
 
     LY_CHECK_ARG_RET(NULL, ctx, NULL);
 
-    ctx_data = ly_ctx_private_data_get(ctx);
+    ctx_data = ly_ctx_private_data_get_or_create(ctx);
+    LY_CHECK_RET(!ctx_data, NULL);
     return ctx_data->errs;
 }
 
@@ -194,7 +195,8 @@ ly_err_last(const struct ly_ctx *ctx)
 
     LY_CHECK_ARG_RET(NULL, ctx, NULL);
 
-    ctx_data = ly_ctx_private_data_get(ctx);
+    ctx_data = ly_ctx_private_data_get_or_create(ctx);
+    LY_CHECK_RET(!ctx_data, NULL);
     return ctx_data->errs ? ctx_data->errs->prev : NULL;
 }
 
@@ -205,12 +207,12 @@ ly_err_move(struct ly_ctx *src_ctx, struct ly_ctx *trg_ctx)
     struct ly_err_item *errs = NULL;
 
     /* get src context errs */
-    src_data = ly_ctx_private_data_get(src_ctx);
+    src_data = ly_ctx_private_data_get_or_create(src_ctx);
     errs = src_data->errs;
     src_data->errs = NULL;
 
     /* get and free the trg context errs */
-    trg_data = ly_ctx_private_data_get(trg_ctx);
+    trg_data = ly_ctx_private_data_get_or_create(trg_ctx);
     ly_err_free(trg_data->errs);
 
     /* set them for trg */
@@ -243,7 +245,7 @@ ly_err_clean(const struct ly_ctx *ctx, struct ly_err_item *eitem)
         return;
     }
 
-    ctx_data = ly_ctx_private_data_get(ctx);
+    ctx_data = ly_ctx_private_data_get_or_create(ctx);
     if (ctx_data->errs == eitem) {
         eitem = NULL;
     }
@@ -412,7 +414,8 @@ log_store(const struct ly_ctx *ctx, LY_LOG_LEVEL level, LY_ERR err, LY_VECODE ve
     assert(ctx && (level < LY_LLVRB));
 
     /* get context private data */
-    ctx_data = ly_ctx_private_data_get(ctx);
+    ctx_data = ly_ctx_private_data_get_or_create(ctx);
+    LY_CHECK_GOTO(!ctx_data, mem_fail);
 
     e = ctx_data->errs;
     if (!e) {