commit 46afa22dc72c38be58132a9ac8be86b21547d1fc
author James Robinson <jamesr@fuchsia.infra.roller.google.com> Fri Jan 21 06:01:54 2022 +0000
committer Copybara-Service <copybara-worker@google.com> Thu Jan 20 22:08:47 2022 -0800
tree fcbb43f3c46cc6ec1dba86692a2a120c42b0bd24
parent 7dba257e55bb7277319efac0f8e033765e3eb068

[roll] Roll fuchsia [fdio][zxio] Use buffer+length strings in two path operations

This updates the logic for operations performed on two paths (rename and
link) to represent the string using an explicitly tracked length instead
of relying on null termination in both the zxio and fdio layers.

In zxio, the C interface is updated to accept a pointer to a buffer and
a length and the implementation is updated to use this length instead of
calling strlen() on the passed in value.

In fdio, the buffers used for these operations are allocated internally
on the stack. This updates the logic for allocating and manipulating the
path buffers to use fbl::StringBuffer which keeps track of the length
and uses std::string_view to pass buffer+length tuples inside fdio.

fbl::StringBuffer is also allocating and managing a null terminator, as
is the old code. This terminator isn't necessary and we may wish to
remove it later on.

Original-Bug: 91672
Original-Reviewed-on: https://fuchsia-review.googlesource.com/c/fuchsia/+/634644
Original-Revision: 99808ba955a236103064d8e2b7c8a3ef2eb2f856
GitOrigin-RevId: 83fba39cc3d2c42dfdb97dc39c9a265c646bb550
Change-Id: Ib1441715e2cf3dca995f9d88f0b8be96c9ee33b0