[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/2] avoid asprintf (not available on mingw64)
From: |
Blue Swirl |
Subject: |
Re: [Qemu-devel] [PATCH 1/2] avoid asprintf (not available on mingw64) |
Date: |
Sun, 12 Aug 2012 09:54:33 +0000 |
On Fri, Aug 10, 2012 at 2:18 PM, Markus Armbruster <address@hidden> wrote:
> Gerd Hoffmann <address@hidden> writes:
>
>> Signed-off-by: Gerd Hoffmann <address@hidden>
>> ---
>> hw/msix.c | 5 ++---
>> 1 files changed, 2 insertions(+), 3 deletions(-)
>>
>> diff --git a/hw/msix.c b/hw/msix.c
>> index 800fc32..04345f2 100644
>> --- a/hw/msix.c
>> +++ b/hw/msix.c
>> @@ -307,9 +307,8 @@ int msix_init_exclusive_bar(PCIDevice *dev, unsigned
>> short nentries,
>> return -EINVAL;
>> }
>>
>> - if (asprintf(&name, "%s-msix", dev->name) == -1) {
>> - return -ENOMEM;
>> - }
>> + name = g_malloc(sizeof(dev->name) + 5);
>> + snprintf(name, sizeof(dev->name) + 5, "%s-msix", dev->name);
>>
>> memory_region_init(&dev->msix_exclusive_bar, name,
>> MSIX_EXCLUSIVE_BAR_SIZE);
>
> What about g_strdup_printf()?
+1
In either case, free() below needs to be changed to g_free().