[libvirt] [PATCH] libxl: fix leaking of allocated migration ports

Jim Fehlig jfehlig at suse.com
Tue Oct 25 02:20:27 UTC 2016


On 10/21/2016 04:41 AM, Pavel Hrdina wrote:
> On Fri, Oct 14, 2016 at 12:19:19PM -0600, Jim Fehlig wrote:
>> Although the migration port is immediately released in the
>> finish phase of migration, it was never set in the domain
>> private object when allocated in the prepare phase. So
>> libxlDomainMigrationFinish() always released a 0-initialized
>> migrationPort, leaking any allocated port. After enough
>> migrations to exhaust the migration port pool, migration would
>> fail with
>>
>> error: internal error: Unable to find an unused port in range
>>        'migration' (49152-49216)
>>
>> Fix it by setting libxlDomainObjPrivate->migrationPort to the
>> port allocated in the prepare phase. While at it, also fix
>> leaking an allocated port if the prepare phase fails.
>>
>> Signed-off-by: Jim Fehlig <jfehlig at suse.com>
>> ---
> ACK

Thanks. Pushed now.

Regards,
Jim




More information about the libvir-list mailing list