[v3] libxl: Fixes for libxl_xenconsole_readline()

[XEN PATCH v3 0/3] libxl: Fixes for libxl_xenconsole_readline()

Javi Merino posted 3 patches 2 months, 3 weeks ago

Diff against v2
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/cover.1725294334.git.javi.merino@cloud.com

tools/include/libxl.h             |  2 +-
tools/libs/light/libxl_console.c  | 52 +++++++++++++++++++++----------
tools/libs/light/libxl_internal.h |  1 -
3 files changed, 36 insertions(+), 19 deletions(-)

Expand all Fold all

[XEN PATCH v3 0/3] libxl: Fixes for libxl_xenconsole_readline()

Posted by Javi Merino 2 months, 3 weeks ago

Fix nul-termination of the return value of
libxl_xen_console_read_line().  While at it, remove unneeded memset()s
to the buffer and improve the documentation of the function.

Changes since v2[0]:
  - Fixed comment format as suggested by Anthony
  - Clarified that 16384 is the default size of xen's console buffer.
  - Fixed `libxl_xen_console_read_line()` documentation as suggested by Alejandro and Andrew.
  - line_r -> buff in `tools/include/libxl.h` as well.

Changes since v1[1]:
  - Add Fixes: line to the first patch
  - Remove cr->count from the struct and make it a local variable in
    libxl_xen_console_read_line()
  - Improve the documentation of libxl_xen_console_read_line()

[0] https://lore.kernel.org/xen-devel/cover.1724430173.git.javi.merino@cloud.com/
[1] https://lore.kernel.org/xen-devel/ad7c89bbae34155566ae7c9ca2cb501f21c7d585.1724330921.git.javi.merino@cloud.com/

Javi Merino (3):
  libxl: Fix nul-termination of the return value of
    libxl_xen_console_read_line()
  libxl: Remove unnecessary buffer zeroing and zalloc()
  libxl: Update the documentation of libxl_xen_console_read_line()

 tools/include/libxl.h             |  2 +-
 tools/libs/light/libxl_console.c  | 52 +++++++++++++++++++++----------
 tools/libs/light/libxl_internal.h |  1 -
 3 files changed, 36 insertions(+), 19 deletions(-)

-- 
2.45.2