[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 25/25] qapi-schema: mark InetSocketAddress as mandato
From: |
Markus Armbruster |
Subject: |
[Qemu-devel] [PULL 25/25] qapi-schema: mark InetSocketAddress as mandatory again |
Date: |
Fri, 30 Oct 2015 16:42:54 +0100 |
From: "Daniel P. Berrange" <address@hidden>
Revert the qapi-schema.json change done in:
commit 0983f5e6af76d5df8c6346cbdfff9d8305fb6da0
Author: Daniel P. Berrange <address@hidden>
Date: Tue Sep 1 14:46:50 2015 +0100
sockets: allow port to be NULL when listening on IP address
Switching "port" from mandatory to optional causes the QAPI
code generator to add a 'has_port' field to the InetSocketAddress
struct. No code that created InetSocketAddress objects was updated
to set 'has_port = true', which caused the non-NULL port strings
to be silently dropped when copying InetSocketAddress objects.
Reported-by: Knut Omang <address@hidden>
Signed-off-by: Daniel P. Berrange <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Markus Armbruster <address@hidden>
---
qapi-schema.json | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/qapi-schema.json b/qapi-schema.json
index f60be29..702b7b5 100644
--- a/qapi-schema.json
+++ b/qapi-schema.json
@@ -2614,9 +2614,7 @@
#
# @host: host part of the address
#
-# @port: port part of the address, or lowest port if @to is present.
-# Kernel selects a free port if omitted for listener addresses.
-# #optional
+# @port: port part of the address, or lowest port if @to is present
#
# @to: highest port to try
#
@@ -2631,7 +2629,7 @@
{ 'struct': 'InetSocketAddress',
'data': {
'host': 'str',
- '*port': 'str',
+ 'port': 'str',
'*to': 'uint16',
'*ipv4': 'bool',
'*ipv6': 'bool' } }
--
2.4.3
- [Qemu-devel] [PULL 00/25] QAPI patches, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 02/25] qapi: More idiomatic string operations, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 07/25] qapi-visit: Split off visit_type_FOO_fields forward decl, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 08/25] qapi-types: Refactor base fields output, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 04/25] qapi: Reserve '*List' type names for list types, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 03/25] qapi: More robust conditions for when labels are needed, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 16/25] sockets: Convert to new qapi union layout, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 13/25] qapi-visit: Convert to new qapi union layout, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 12/25] qapi: Start converting to new qapi union layout, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 21/25] tpm: Convert to new qapi union layout, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 25/25] qapi-schema: mark InetSocketAddress as mandatory again,
Markus Armbruster <=
- [Qemu-devel] [PULL 24/25] qapi: Simplify gen_struct_field(), Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 05/25] qapi: Reserve 'q_*' and 'has_*' member names, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 10/25] qapi: Unbox base members, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 06/25] vnc: Hoist allocation of VncBasicInfo to callers, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 09/25] qapi: Prefer typesafe upcasts to qapi base classes, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 20/25] memory: Convert to new qapi union layout, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 01/25] tests/qapi-schema: Test for reserved names, empty struct, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 17/25] net: Convert to new qapi union layout, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 15/25] block: Convert to new qapi union layout, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 22/25] qapi: Finish converting to new qapi union layout, Markus Armbruster, 2015/10/30