[libvirt] [PATCH 05/10] vbox: Replace atoi with virStrToLong_i
Matthias Bolte
matthias.bolte at googlemail.com
Wed Mar 31 12:49:15 UTC 2010
2010/3/31 Eric Blake <eblake at redhat.com>:
> On 03/30/2010 10:20 AM, Matthias Bolte wrote:
>> ---
>> src/vbox/vbox_tmpl.c | 4 +++-
>> 1 files changed, 3 insertions(+), 1 deletions(-)
>>
>> diff --git a/src/vbox/vbox_tmpl.c b/src/vbox/vbox_tmpl.c
>> index 05b075f..07996d4 100644
>> --- a/src/vbox/vbox_tmpl.c
>> +++ b/src/vbox/vbox_tmpl.c
>> @@ -598,7 +598,9 @@ static int PRUnicharToInt(PRUnichar *strUtf16) {
>> if (!strUtf8)
>> return -1;
>>
>> - ret = atoi(strUtf8);
>> + if (virStrToLong_i(strUtf8, NULL, 10, &ret) < 0)
>> + return -1;
>> +
>> g_pVBoxGlobalData->pFuncs->pfnUtf8Free(strUtf8);
>
> Oops. Memory leak if strUtf8 was not valid.
>
Good catch.
It seems that I missed many details in this series :(
Matthias
More information about the libvir-list
mailing list