The block layer uses a mixture of 'PATH_MAX' and '1024' string sizes
for filenames (and backing filenames).
This series consolidates all that usage to 'PATH_MAX'. Since most platforms
(especially the most common platforms for QEMU) have a PATH_MAX larger than
1024 bytes, this series also changes stack allocations of PATH_MAX to be
dynamically allocated.
Note: checkpatch.pl complains about an extra space in a printf in
patches 1 & 2. The lines complained about are in the diff context and
not the actual changes, so I did not fix them up to satisfy checkpatch.
Changes from v3:
- simplified extent_path handling in vmdk_parse_extents() (Thanks Stefan)
- moved declaration of backing_filename2 to inside if
statement in bdrv_query_image_info() (Thanks Stefan)
- removed zombie variable in bdrv_commit (Thanks Stefan)
- fixed typo in commit message (Thanks Stefan)
Changes from v2:
- Change stack allocations to dybnamic (Thanks Kevin)
- Update qcow/qcow2 ti perform safety checks for platforms that
have a PATH_MAX < 1024 (thanks John, Kevin).
Jeff Cody (6):
block: vmdk - make ret variable usage clear
block: vmdk - move string allocations from stack to the heap
block: qapi - move string allocation from stack to the heap
block: remove unused variable in bdrv_commit
block: mirror - change string allocation to 2-bytes
block: update string sizes for filename,backing_file,exact_filename
block.c | 3 ---
block/mirror.c | 3 ++-
block/qapi.c | 7 ++++---
block/qcow.c | 2 +-
block/qcow2.c | 3 ++-
block/vmdk.c | 51 ++++++++++++++++++++++++++++-------------------
block/vvfat.c | 4 ++--
include/block/block_int.h | 8 ++++----
qemu-img.c | 4 ++--
9 files changed, 47 insertions(+), 38 deletions(-)
--
1.9.3
