Daniel Lezcano <[EMAIL PROTECTED]> writes:

>> +
>> +    mutex_lock(&net_mutex);
>> +    err = setup_net(new_net);
>> +    if (err)
>> +            goto out_unlock;
>>
> Should we "net_free" in case of error ?

Oops.  Yes we should.
Thanks.

>> +    net_lock();
>> +    net_list_append(new_net);
>> +    net_unlock();
>> +
>> +    tsk->nsproxy->net_ns = new_net;
>> +
>> +out_unlock:
>> +    mutex_unlock(&net_mutex);
        net_free(new_net);
>> +out:
>> +    put_net(old_net);
>> +    return err;
>> +}
>> +
>>

Eric
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to