SetVariable

Hi all,
I found this code on the file variables.c and
in the function SetVariable I read:

if (strcmp(current->name, name) == 0)
{
free(current->value);
current->value = strdup(value);
return current->value ? true : false;
}

this mean that if there is no memory left on the
sistem we loose the old value,
if this is not the indeended behaviour may be is better do:

if (strcmp(current->name, name) == 0)
{
char * tmp_value = strdup(value);

if ( !tmp_value )
{
return false;
}

free(current->value);
current->value = tmp_value;

return true;
}

Regards
Gaetano Mendola