Xen devel - Jul 2013 - [PATCH] libxl_json: Fix backport of JSON_BOOL to 4.2.2

Commit 2b3072ed0cbeed8c0385f20e92ba0f1201db8a17 has the setting of obj->u.b
Commit 6a2aca9fdef0499e613715baf107f2296b9007cf does not.

This shows up by vnc-port and vnc-listen are missing in xenstore when they
should not be.

Signed-off-by: Don Slutz <dslutz@verizon.com>
---
 tools/libxl/libxl_json.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/tools/libxl/libxl_json.c b/tools/libxl/libxl_json.c
index 72b52e8..0810630 100644
--- a/tools/libxl/libxl_json.c
+++ b/tools/libxl/libxl_json.c
@@ -474,6 +474,7 @@ static int json_callback_boolean(void *opaque, int boolean)
 
     if ((obj = libxl__json_object_alloc(ctx->gc, JSON_BOOL)) == NULL)
         return 0;
+    obj->u.b = boolean;
 
     if (libxl__json_object_append_to(ctx->gc, obj, ctx->current) == -1) {
         libxl__json_object_free(ctx->gc, obj);
-- 
1.7.1

Looks like I had an issue with the mailer I used, and this did not get to anyone
at citrix.com. Resending via a different SMTP server.
    -Don Slutz

On 07/05/13 11:11, Don Slutz wrote:
> Commit 2b3072ed0cbeed8c0385f20e92ba0f1201db8a17 has the setting of
obj->u.b
> Commit 6a2aca9fdef0499e613715baf107f2296b9007cf does not.
>
> This shows up by vnc-port and vnc-listen are missing in xenstore when they
should not be.
>
> Signed-off-by: Don Slutz <dslutz@verizon.com>
> ---
>   tools/libxl/libxl_json.c |    1 +
>   1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/tools/libxl/libxl_json.c b/tools/libxl/libxl_json.c
> index 72b52e8..0810630 100644
> --- a/tools/libxl/libxl_json.c
> +++ b/tools/libxl/libxl_json.c
> @@ -474,6 +474,7 @@ static int json_callback_boolean(void *opaque, int
boolean)
>   
>       if ((obj = libxl__json_object_alloc(ctx->gc, JSON_BOOL)) == NULL)
>           return 0;
> +    obj->u.b = boolean;
>   
>       if (libxl__json_object_append_to(ctx->gc, obj, ctx->current) ==
-1) {
>           libxl__json_object_free(ctx->gc, obj);

Don Slutz <dslutz@verizon.com> wrote:
>Looks like I had an issue with the mailer I used, and this did not get
>to anyone at citrix.com. Resending via a different SMTP server.
>    -Don Slutz
>
>On 07/05/13 11:11, Don Slutz wrote:
>> Commit 2b3072ed0cbeed8c0385f20e92ba0f1201db8a17 has the setting of
>obj->u.b
>> Commit 6a2aca9fdef0499e613715baf107f2296b9007cf does not.
>>
>> This shows up by vnc-port and vnc-listen are missing in xenstore when
>they should not be.
>>
>> Signed-off-by: Don Slutz <dslutz@verizon.com>
>> ---
>>   tools/libxl/libxl_json.c |    1 +
>>   1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/tools/libxl/libxl_json.c b/tools/libxl/libxl_json.c
>> index 72b52e8..0810630 100644
>> --- a/tools/libxl/libxl_json.c
>> +++ b/tools/libxl/libxl_json.c
>> @@ -474,6 +474,7 @@ static int json_callback_boolean(void *opaque,
>int boolean)
>>   
>>       if ((obj = libxl__json_object_alloc(ctx->gc, JSON_BOOL))
=>NULL)
>>           return 0;
>> +    obj->u.b = boolean;
>>   
>>       if (libxl__json_object_append_to(ctx->gc, obj,
ctx->current) =>-1) {
>>           libxl__json_object_free(ctx->gc, obj);
>
>
>_______________________________________________
>Xen-devel mailing list
>Xen-devel@lists.xen.org
>http://lists.xen.org/xen-devel
Could you also include in the commit description not just the git commit id but
also also the title of said patches? Thanks
-- 
Sent from my Android phone. Please excuse my brevity.

On 07/05/13 16:29, Konrad Rzeszutek Wilk wrote:
> Don Slutz <dslutz@verizon.com> wrote:
>
>> Looks like I had an issue with the mailer I used, and this did not get
>> to anyone at citrix.com. Resending via a different SMTP server.
>>     -Don Slutz
>>
>> On 07/05/13 11:11, Don Slutz wrote:
>>> Commit 2b3072ed0cbeed8c0385f20e92ba0f1201db8a17 has the setting of
>> obj->u.b
>>> Commit 6a2aca9fdef0499e613715baf107f2296b9007cf does not.
>>>
>>> This shows up by vnc-port and vnc-listen are missing in xenstore
when
>> they should not be.
>>> Signed-off-by: Don Slutz <dslutz@verizon.com>
>>> ---
>>>    tools/libxl/libxl_json.c |    1 +
>>>    1 files changed, 1 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/tools/libxl/libxl_json.c b/tools/libxl/libxl_json.c
>>> index 72b52e8..0810630 100644
>>> --- a/tools/libxl/libxl_json.c
>>> +++ b/tools/libxl/libxl_json.c
>>> @@ -474,6 +474,7 @@ static int json_callback_boolean(void *opaque,
>> int boolean)
>>>    
>>>        if ((obj = libxl__json_object_alloc(ctx->gc, JSON_BOOL))
=>> NULL)
>>>            return 0;
>>> +    obj->u.b = boolean;
>>>    
>>>        if (libxl__json_object_append_to(ctx->gc, obj,
ctx->current) =>> -1) {
>>>            libxl__json_object_free(ctx->gc, obj);
>>
>> _______________________________________________
>> Xen-devel mailing list
>> Xen-devel@lists.xen.org
>> http://lists.xen.org/xen-devel
> Could you also include in the commit description not just the git commit id
but also also the title of said patches? Thanks
How does this look:

     Commit 2b3072ed0cbeed8c0385f20e92ba0f1201db8a17 has the setting of 
obj->u.b
     libxl_json: Replace JSON_TRUE/FALSE by JSON_BOOL.

     Commit 6a2aca9fdef0499e613715baf107f2296b9007cf does not.
     libxl_json: Replace JSON_TRUE/FALSE by JSON_BOOL.

     This shows up by vnc-port and vnc-listen are missing in xenstore 
when they should not be.

for the commit message?
    -Don Slutz

Don Slutz <dslutz@verizon.com> wrote:
>On 07/05/13 16:29, Konrad Rzeszutek Wilk wrote:
>> Don Slutz <dslutz@verizon.com> wrote:
>>
>>> Looks like I had an issue with the mailer I used, and this did not
>get
>>> to anyone at citrix.com. Resending via a different SMTP server.
>>>     -Don Slutz
>>>
>>> On 07/05/13 11:11, Don Slutz wrote:
>>>> Commit 2b3072ed0cbeed8c0385f20e92ba0f1201db8a17 has the setting
of
>>> obj->u.b
>>>> Commit 6a2aca9fdef0499e613715baf107f2296b9007cf does not.
>>>>
>>>> This shows up by vnc-port and vnc-listen are missing in
xenstore
>when
>>> they should not be.
>>>> Signed-off-by: Don Slutz <dslutz@verizon.com>
>>>> ---
>>>>    tools/libxl/libxl_json.c |    1 +
>>>>    1 files changed, 1 insertions(+), 0 deletions(-)
>>>>
>>>> diff --git a/tools/libxl/libxl_json.c
b/tools/libxl/libxl_json.c
>>>> index 72b52e8..0810630 100644
>>>> --- a/tools/libxl/libxl_json.c
>>>> +++ b/tools/libxl/libxl_json.c
>>>> @@ -474,6 +474,7 @@ static int json_callback_boolean(void
*opaque,
>>> int boolean)
>>>>    
>>>>        if ((obj = libxl__json_object_alloc(ctx->gc,
JSON_BOOL)) =>>> NULL)
>>>>            return 0;
>>>> +    obj->u.b = boolean;
>>>>    
>>>>        if (libxl__json_object_append_to(ctx->gc, obj,
ctx->current)
>=>>> -1) {
>>>>            libxl__json_object_free(ctx->gc, obj);
>>>
>>> _______________________________________________
>>> Xen-devel mailing list
>>> Xen-devel@lists.xen.org
>>> http://lists.xen.org/xen-devel
>> Could you also include in the commit description not just the git
>commit id but also also the title of said patches? Thanks
>How does this look:
>
>    Commit 2b3072ed0cbeed8c0385f20e92ba0f1201db8a17 has the setting of 
>obj->u.b
>     libxl_json: Replace JSON_TRUE/FALSE by JSON_BOOL.
>
>     Commit 6a2aca9fdef0499e613715baf107f2296b9007cf does not.
>     libxl_json: Replace JSON_TRUE/FALSE by JSON_BOOL.
>
>     This shows up by vnc-port and vnc-listen are missing in xenstore 
>when they should not be.
>
>for the commit message?
>    -Don Slutz
I don''t have my git tree in front of me but are you sure that both git
commits have the same title.  That would be pretty odd.

Anyhow I usually just include the title right after the commit as: git commit
xyz (''title of said patch'').
-- 
Sent from my Android phone. Please excuse my brevity.

Looks like I made an error backporting here, which is odd as I thought
this applied directly. I''ve checked the original hg patches I
was working from and there''s no obvious reason for the difference.
Sorry.

Whilst Don''s fix is right in that obj->u.b should be set, I note
that
4.3 does not check the error return from libxl__json_object_alloc
whereas 4.2 does. Given libxl__json_object_alloc itself may
SEGV if libxl__zalloc fails, this should not matter.

Acked-By from me below.

Alex


--On 5 July 2013 11:11:45 -0400 Don Slutz <dslutz@verizon.com> wrote:
> Commit 2b3072ed0cbeed8c0385f20e92ba0f1201db8a17 has the setting of
> obj->u.b Commit 6a2aca9fdef0499e613715baf107f2296b9007cf does not.
>
> This shows up by vnc-port and vnc-listen are missing in xenstore when
> they should not be.
>
> Signed-off-by: Don Slutz <dslutz@verizon.com>
Acked-By: Alex Bligh <alex@alex.org.uk>
> ---
>  tools/libxl/libxl_json.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/tools/libxl/libxl_json.c b/tools/libxl/libxl_json.c
> index 72b52e8..0810630 100644
> --- a/tools/libxl/libxl_json.c
> +++ b/tools/libxl/libxl_json.c
> @@ -474,6 +474,7 @@ static int json_callback_boolean(void *opaque, int
> boolean)
>      if ((obj = libxl__json_object_alloc(ctx->gc, JSON_BOOL)) == NULL)
>          return 0;
> +    obj->u.b = boolean;
>
>      if (libxl__json_object_append_to(ctx->gc, obj, ctx->current) ==
-1) {
>          libxl__json_object_free(ctx->gc, obj);
> --
> 1.7.1
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel
>
>


-- 
Alex Bligh

On 07/05/13 17:09, Konrad Rzeszutek Wilk wrote:
> Don Slutz <dslutz@verizon.com> wrote:
>
>> On 07/05/13 16:29, Konrad Rzeszutek Wilk wrote:
>>> Don Slutz <dslutz@verizon.com> wrote:
>>>
>>>> Looks like I had an issue with the mailer I used, and this did
not
>> get
>>>> to anyone at citrix.com. Resending via a different SMTP server.
>>>>      -Don Slutz
>>>>
>>>> On 07/05/13 11:11, Don Slutz wrote:
>>>>> Commit 2b3072ed0cbeed8c0385f20e92ba0f1201db8a17 has the
setting of
>>>> obj->u.b
>>>>> Commit 6a2aca9fdef0499e613715baf107f2296b9007cf does not.
>>>>>
>>>>> This shows up by vnc-port and vnc-listen are missing in
xenstore
>> when
>>>> they should not be.
>>>>> Signed-off-by: Don Slutz <dslutz@verizon.com>
>>>>> ---
>>>>>     tools/libxl/libxl_json.c |    1 +
>>>>>     1 files changed, 1 insertions(+), 0 deletions(-)
>>>>>
>>>>> diff --git a/tools/libxl/libxl_json.c
b/tools/libxl/libxl_json.c
>>>>> index 72b52e8..0810630 100644
>>>>> --- a/tools/libxl/libxl_json.c
>>>>> +++ b/tools/libxl/libxl_json.c
>>>>> @@ -474,6 +474,7 @@ static int json_callback_boolean(void
*opaque,
>>>> int boolean)
>>>>>     
>>>>>         if ((obj = libxl__json_object_alloc(ctx->gc,
JSON_BOOL)) =>>>> NULL)
>>>>>             return 0;
>>>>> +    obj->u.b = boolean;
>>>>>     
>>>>>         if (libxl__json_object_append_to(ctx->gc, obj,
ctx->current)
>> =>>>> -1) {
>>>>>             libxl__json_object_free(ctx->gc, obj);
>>>> _______________________________________________
>>>> Xen-devel mailing list
>>>> Xen-devel@lists.xen.org
>>>> http://lists.xen.org/xen-devel
>>> Could you also include in the commit description not just the git
>> commit id but also also the title of said patches? Thanks
>> How does this look:
>>
>>     Commit 2b3072ed0cbeed8c0385f20e92ba0f1201db8a17 has the setting of
>> obj->u.b
>>      libxl_json: Replace JSON_TRUE/FALSE by JSON_BOOL.
>>
>>      Commit 6a2aca9fdef0499e613715baf107f2296b9007cf does not.
>>      libxl_json: Replace JSON_TRUE/FALSE by JSON_BOOL.
>>
>>      This shows up by vnc-port and vnc-listen are missing in xenstore
>> when they should not be.
>>
>> for the commit message?
>>     -Don Slutz
> I don''t have my git tree in front of me but are you sure that both
git commits have the same title.  That would be pretty odd.
>
> Anyhow I usually just include the title right after the commit as: git
commit xyz (''title of said patch'').
I have attached a v2 of the commit message (including the Acked-By: Alex Bligh)
and the 2 git commit messages referred to.  I can top post the v2 if needed.
    -Don Slutz




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