[libvirt] [PATCH 10/10] Replace sscanf in PCI device address parsing

Matthias Bolte matthias.bolte at googlemail.com
Wed Mar 31 17:01:40 UTC 2010


2010/3/31 Eric Blake <eblake at redhat.com>:
> On 03/30/2010 10:20 AM, Matthias Bolte wrote:
>> This also fixes a problem with MinGW's GCC on Windows.
>> GCC complained bout the L modifier being unknown.

>>              VIR_WARN("Unusual entry in " PCI_SYSFS "devices: %s", entry->d_name);
>>              continue;
>>          }
>> @@ -914,10 +925,9 @@ pciWaitForDeviceCleanup(pciDevice *dev, const char *matcher)
>>      FILE *fp;
>>      char line[160];
>>      unsigned long long start, end;
>> -    int consumed;
>>      char *rest;
>> -    unsigned long long domain;
>> -    int bus, slot, function;
>> +    char *tmp;
>> +    unsigned int domain, bus, slot, function;
>
> Why the change in the size of domain?

domain is parsed and stored everywhere else as unsigned int. Do the
same here, there's no reason for domain to be handled as 64bit

Matthias




More information about the libvir-list mailing list