Skip to content

hw/9pfs: fix 3 issues which related with path string

Chen Gang qemu-devel 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest3 patches 15 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3hw/9pfs/virtio-9p-local.c: move v9fs_string_free() to below "err_out:"hw/9pfs/virtio-9p-local.c+1 -12 replies
2/3hw/9pfs/virtio-9p-local.c: use snprintf() instead of sprintf()hw/9pfs/virtio-9p-local.c+4 -36 replies
3/3hw/9pfs: use g_strdup_printf() instead of PATH_MAX limitationhw/9pfs/cofs.c hw/9pfs/virtio-9p-handle.c hw/9pfs/virtio-9p-local.c +5 more+292 -1385 replies

Reviews trailers per patch

2 of 3 patches have a Reviewed-by or Acked-by.

PatchAneesh Kumar K.V
1/3R-b
2/3R-b
3/3

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [Qemu-devel] [PATCH 0/3] hw/9pfs: fix 3 issues which related with path string

Patch 1/3: move v9fs_string_free() to below "err_out:"

Patch 2/3: use snprintf() instead of sprintf()
(which will be replaced of by Path 3/3)

Patch 3/3: use g_strdup_printf() instead of PATH_MAX limitation

---
 hw/9pfs/cofs.c                 |  15 ++-
 hw/9pfs/virtio-9p-handle.c     |   9 +-
 hw/9pfs/virtio-9p-local.c      | 286 +++++++++++++++++++++++++++--------------
 hw/9pfs/virtio-9p-posix-acl.c  |  52 ++++++--
 hw/9pfs/virtio-9p-xattr-user.c |  27 +++-
 hw/9pfs/virtio-9p-xattr.c      |   9 +-
 hw/9pfs/virtio-9p-xattr.h      |  27 +++-
 hw/9pfs/virtio-9p.h            |   6 +-
 8 files changed, 293 insertions(+), 138 deletions(-)