[Xen-devel] [PATCH for-4.6 v3 0/6] Fix libxl TOOLSTACK records for migration v2

[Andrew Cooper <andrew.cooper3@xxxxxxxxxx>]

The libxl blob for toolstack records was not 32/64bit safe.  In 64bit
builds, it had an extra 4 bytes of padding due to alignment, and because
of the pointer arithmatic used to build it, leaked 4 bytes of libxl heap
into the stream.

Respecify XENSTORE_DATA as EMULATOR_XENSTORE_DATA and take remedial
action to prevent this bitness issue escaping into a Xen release.

Only minor changes from v3.  Details in patches.

Summary of Modified/Acks:

Andrew Cooper (6):
A  tools/libxl: Make libxl__conversion_helper_abort() safe to use
 M docs/libxl: Re-specify XENSTORE_DATA as EMULATOR_XENSTORE_DATA
 M tools/libxl: Save and restore EMULATOR_XENSTORE_DATA content
A  tools/libxl: Prepare to write multiple records with EMULATOR headers
A  libxl/save&restore&convert: Switch to new EMULATOR_XENSTORE_DATA records
A  tools/libxl: Drop all legacy "toolstack" record infrastructure

 docs/specs/libxl-migration-stream.pandoc   |   82 +++++---
 tools/libxl/libxl_convert_callout.c        |    2 +
 tools/libxl/libxl_dom.c                    |  290 ++++++++++------------------
 tools/libxl/libxl_internal.h               |   15 +-
 tools/libxl/libxl_sr_stream_format.h       |   10 +-
 tools/libxl/libxl_stream_read.c            |   27 ++-
 tools/libxl/libxl_stream_write.c           |  194 ++++++++++++-------
 tools/python/Makefile                      |    1 +
 tools/python/scripts/convert-legacy-stream |   68 ++++++-
 tools/python/xen/migration/legacy.py       |   40 +++-
 tools/python/xen/migration/libxl.py        |   70 +++++--
 tools/python/xen/migration/tests.py        |    2 +-
 12 files changed, 464 insertions(+), 337 deletions(-)

--
1.7.10.4

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel