[Xen-devel] [PATCH] 'dom' error handled, by raising ENOMEM if failure occurs. 'xs_fd' leak handled in success path.

2015-10-27 Thread Lasya
Signed-off-by: Lasya Venneti --- tools/xenstore/init-xenstore-domain.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/tools/xenstore/init-xenstore-domain.c b/tools/xenstore/init-xenstore-domain.c index 0d12169..b413b09 100644 ---

Re: [Xen-devel] [PATCH] 'dom' error handled, by raising ENOMEM if failure occurs. 'xs_fd' leak handled in success path.

2015-10-27 Thread Dario Faggioli
On Wed, 2015-10-28 at 03:49 +0530, Lasya wrote: The description of the patch, that you previously put in a separate email, should live here. Also, this is the second version of a patch. That should be evident from the subject line (something like "[PATCH v2]"). And still about the subject line,

Re: [Xen-devel] [PATCH] 'dom' error handled, by raising ENOMEM if failure occurs. 'xs_fd' leak handled in success path.

2015-10-27 Thread George Dunlap
On 27/10/15 11:04, Lasya Venneti wrote: > On 27 October 2015 at 16:25, Dario Faggioli > wrote: > >> On Wed, 2015-10-28 at 03:49 +0530, Lasya wrote: >> >> The description of the patch, that you previously put in a separate >> email, should live here. >> > Oh, I had

Re: [Xen-devel] [PATCH] 'dom' error handled, by raising ENOMEM if failure occurs. 'xs_fd' leak handled in success path.

2015-10-27 Thread Lasya Venneti
On 27 October 2015 at 16:25, Dario Faggioli wrote: > On Wed, 2015-10-28 at 03:49 +0530, Lasya wrote: > > The description of the patch, that you previously put in a separate > email, should live here. > Oh, I had thought no cover letter was needed. Hence, I didn't put

Re: [Xen-devel] [PATCH] 'dom' error handled, by raising ENOMEM if failure occurs. 'xs_fd' leak handled in success path.

2015-10-27 Thread Lasya Venneti
On 27 October 2015 at 16:37, George Dunlap wrote: > On 27/10/15 11:04, Lasya Venneti wrote: > > On 27 October 2015 at 16:25, Dario Faggioli > > wrote: > > > >> On Wed, 2015-10-28 at 03:49 +0530, Lasya wrote: > >> > >> The description of the