path MAINTENANCE clarify string length param

Author: michalvasko

src/path.c

@@ -3,7 +3,7 @@
  * @author Michal Vasko <[email protected]>
  * @brief Path functions
  *
- * Copyright (c) 2020 - 2023 CESNET, z.s.p.o.
+ * Copyright (c) 2020 - 2025 CESNET, z.s.p.o.
  *
  * This source code is licensed under BSD 3-Clause License (the "License").
  * You may not use this file except in compliance with the License.
@@ -339,6 +339,10 @@ ly_path_parse(const struct ly_ctx *ctx, const struct lysc_node *ctx_node, const
         LOG_LOCSET(ctx_node, NULL);
     }
 
+    if (!path_len) {
+        path_len = strlen(str_path);
+    }
+
     /* parse as a generic XPath expression, reparse is performed manually */
     LY_CHECK_GOTO(ret = lyxp_expr_parse(ctx, str_path, path_len, 0, &exp), error);
     tok_idx = 0;

src/path.h

@@ -118,7 +118,7 @@ struct ly_path {
  * @param[in] ctx libyang context.
  * @param[in] ctx_node Optional context node, used for logging.
  * @param[in] str_path Path to parse.
- * @param[in] path_len Length of @p str_path.
+ * @param[in] path_len Length of @p str_path, may be 0 if @p str_path is 0-terminated.
  * @param[in] lref Whether leafref is being parsed or not.
  * @param[in] begin Begin option (@ref path_begin_options).
  * @param[in] prefix Prefix option (@ref path_prefix_options).
@@ -135,7 +135,7 @@ LY_ERR ly_path_parse(const struct ly_ctx *ctx, const struct lysc_node *ctx_node,
  * @param[in] ctx libyang context.
  * @param[in] cur_node Optional current (original context) node, used for logging.
  * @param[in] str_path Path to parse.
- * @param[in] path_len Length of @p str_path.
+ * @param[in] path_len Length of @p str_path, may be 0 if @p str_path is 0-terminated.
  * @param[in] prefix Prefix option (@ref path_prefix_options).
  * @param[in] pred Predicate option (@ref path_pred_options).
  * @param[out] expr Parsed path.

src/tree_data.c

@@ -3578,8 +3578,8 @@ lyd_find_path(const struct lyd_node *ctx_node, const char *path, ly_bool output,
     LY_CHECK_ARG_RET(NULL, ctx_node, ctx_node->schema, path, LY_EINVAL);
 
     /* parse the path */
-    ret = ly_path_parse(LYD_CTX(ctx_node), ctx_node->schema, path, strlen(path), 0, LY_PATH_BEGIN_EITHER,
-            LY_PATH_PREFIX_FIRST, LY_PATH_PRED_SIMPLE, &expr);
+    ret = ly_path_parse(LYD_CTX(ctx_node), ctx_node->schema, path, 0, 0, LY_PATH_BEGIN_EITHER, LY_PATH_PREFIX_FIRST,
+            LY_PATH_PRED_SIMPLE, &expr);
     LY_CHECK_GOTO(ret, cleanup);
 
     /* compile the path */

src/tree_data_new.c

@@ -1682,7 +1682,7 @@ lyd_new_path_(struct lyd_node *parent, const struct ly_ctx *ctx, const struct ly
     LY_CHECK_GOTO(ret = lyd_new_val_get_format(options, &format), cleanup);
 
     /* parse path */
-    LY_CHECK_GOTO(ret = ly_path_parse(ctx, NULL, path, strlen(path), 0, LY_PATH_BEGIN_EITHER, LY_PATH_PREFIX_FIRST,
+    LY_CHECK_GOTO(ret = ly_path_parse(ctx, NULL, path, 0, 0, LY_PATH_BEGIN_EITHER, LY_PATH_PREFIX_FIRST,
             LY_PATH_PRED_SIMPLE, &exp), cleanup);
 
     /* compile path */

src/tree_schema.c

@@ -1,9 +1,10 @@
 /**
  * @file tree_schema.c
  * @author Radek Krejci <[email protected]>
+ * @author Michal Vasko <[email protected]>
  * @brief Schema tree implementation
  *
- * Copyright (c) 2015 - 2018 CESNET, z.s.p.o.
+ * Copyright (c) 2015 - 2025 CESNET, z.s.p.o.
  *
  * This source code is licensed under BSD 3-Clause License (the "License").
  * You may not use this file except in compliance with the License.
@@ -631,7 +632,7 @@ lys_find_path_atoms(const struct ly_ctx *ctx, const struct lysc_node *ctx_node,
     }
 
     /* parse */
-    ret = ly_path_parse(ctx, ctx_node, path, strlen(path), 0, LY_PATH_BEGIN_EITHER, LY_PATH_PREFIX_FIRST,
+    ret = ly_path_parse(ctx, ctx_node, path, 0, 0, LY_PATH_BEGIN_EITHER, LY_PATH_PREFIX_FIRST,
             LY_PATH_PRED_SIMPLE, &expr);
     LY_CHECK_GOTO(ret, cleanup);
 
@@ -658,15 +659,15 @@ lys_find_path(const struct ly_ctx *ctx, const struct lysc_node *ctx_node, const
     LY_ERR ret;
     uint8_t oper;
 
-    LY_CHECK_ARG_RET(ctx, ctx || ctx_node, NULL);
+    LY_CHECK_ARG_RET(ctx, ctx || ctx_node, path, NULL);
     LY_CHECK_CTX_EQUAL_RET(ctx, ctx_node ? ctx_node->module->ctx : NULL, NULL);
 
     if (!ctx) {
         ctx = ctx_node->module->ctx;
     }
 
     /* parse */
-    ret = ly_path_parse(ctx, ctx_node, path, strlen(path), 0, LY_PATH_BEGIN_EITHER, LY_PATH_PREFIX_FIRST,
+    ret = ly_path_parse(ctx, ctx_node, path, 0, 0, LY_PATH_BEGIN_EITHER, LY_PATH_PREFIX_FIRST,
             LY_PATH_PRED_SIMPLE, &expr);
     LY_CHECK_GOTO(ret, cleanup);