From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:34484)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <quintela@redhat.com>) id 1WwVXU-0001jf-7B
	for qemu-devel@nongnu.org; Mon, 16 Jun 2014 07:57:34 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <quintela@redhat.com>) id 1WwVXL-00059r-Vm
	for qemu-devel@nongnu.org; Mon, 16 Jun 2014 07:57:28 -0400
From: Juan Quintela <quintela@redhat.com>
In-Reply-To: <53996F09.8030200@linux.vnet.ibm.com> (Michael R. Hines's message
	of "Thu, 12 Jun 2014 17:12:41 +0800")
References: <1392690846-30926-1-git-send-email-mrhines@linux.vnet.ibm.com>
	<53996F09.8030200@linux.vnet.ibm.com>
Date: Mon, 16 Jun 2014 13:57:08 +0200
Message-ID: <87ioo1ysmj.fsf@troll.troll>
MIME-Version: 1.0
Content-Type: text/plain
Subject: Re: [Qemu-devel] [PATCH] rdma: bug fixes
Reply-To: quintela@redhat.com
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: "Michael R. Hines" <mrhines@linux.vnet.ibm.com>
Cc: "Michael R. Hines" <mrhines@us.ibm.com>, qemu-stable@nongnu.org, hinesmr@cn.ibm.com, qemu-devel@nongnu.org, mdroth@linux.vnet.ibm.com

"Michael R. Hines" <mrhines@linux.vnet.ibm.com> wrote:
> On 02/18/2014 10:34 AM, mrhines@linux.vnet.ibm.com wrote:
>> From: "Michael R. Hines" <mrhines@us.ibm.com>


Applied, would go in next pull request.

>> @@ -2512,8 +2510,10 @@ static void *qemu_rdma_data_init(const char *host_port, Error **errp)
>>           } else {
>>               ERROR(errp, "bad RDMA migration address '%s'", host_port);
>>               g_free(rdma);
>> -            return NULL;
>> +            rdma = NULL;
>>           }
>> +
>> +        qapi_free_InetSocketAddress(addr);
>>       }
>>
>>       return rdma;
>
> Juan? Ping again?

I obviously missed this one, sorry :-(

Later, Juan.)