z_bytes_serialize_from_str aliases it's argument which is not clear from function's name

[milyin]

Describe the release item

The function

ZENOHC_API void z_bytes_serialize_from_str(struct z_owned_bytes_t *this_, const char *s);

actually stores s pointer inside created z_owned_bytes_t.

This raises 2 problems:

• from the function name it's not clear, without looking into docs user may think that data is copied
• even if user knows how this works, he have no way to know at which moment the s is ready to be freed. When z_owned_bytes_t is consumed by z_put for example, there is no guarantee that after z_put call the s is not kept in some queue inside zenoh. So actually this function is useful for static strings only

[milyin]

PRs related:
#554
eclipse-zenoh/zenoh-pico#56

[milyin]

fixed in corresponding prs