From mboxrd@z Thu Jan  1 00:00:00 1970
From: Anthony Liguori <aliguori@us.ibm.com>
Subject: Re: [PATCH 2/3] Make it clear that	xs_get_domain_path()
	requires free()
Date: Tue, 01 Nov 2005 13:51:52 -0600
Message-ID: <4367C758.7020407@us.ibm.com>
References: <4367BDD3.9020201@us.ibm.com>
	<1130874644.4658.10.camel@w-stekloff.beaverton.ibm.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xensource.com>
In-Reply-To: <1130874644.4658.10.camel@w-stekloff.beaverton.ibm.com>
List-Unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xensource.com>
List-Help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-Subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
Sender: xen-devel-bounces@lists.xensource.com
Errors-To: xen-devel-bounces@lists.xensource.com
To: Daniel Stekloff <dsteklof@us.ibm.com>
Cc: xen-devel <xen-devel@lists.xensource.com>
List-Id: xen-devel@lists.xenproject.org

Daniel Stekloff wrote:

>Just out of curiosity, why have xenstore create a buffer that must be
>freed. Why not have the calling application create a buffer and manage
>it on its own?
>  
>
Because Rusty made it that way :-)

Regards,

Anthony Liguori

>
>On Tue, 2005-11-01 at 13:11 -0600, Anthony Liguori wrote:
>  
>
>>plain text document attachment (7574-xenconsole.diff)
>># HG changeset patch
>># User Anthony Liguori <anthony@codemonkey.ws>
>># Node ID a44fde5c4e15bdf2f8b42fa41439d6725a5363dc
>># Parent  6de1d7b66c8a52ac6420f35aac03fe219caf87a3
>>Make it clear that the result of xs_get_domain_path() must be free()'d.
>>
>>diff -r 6de1d7b66c8a -r a44fde5c4e15 tools/xenstore/xs.h
>>--- a/tools/xenstore/xs.h	Tue Nov  1 11:08:31 2005 -0500
>>+++ b/tools/xenstore/xs.h	Tue Nov  1 11:09:57 2005 -0500
>>@@ -136,7 +136,7 @@
>>  */
>> bool xs_release_domain(struct xs_handle *h, unsigned int domid);
>> 
>>-/* Query the home path of a domain.
>>+/* Query the home path of a domain.  Call free() after use.
>>  */
>> char *xs_get_domain_path(struct xs_handle *h, unsigned int domid);
>> 
>>_______________________________________________
>>Xen-devel mailing list
>>Xen-devel@lists.xensource.com
>>http://lists.xensource.com/xen-devel
>>    
>>
>
>
>  
>