Re: [PATCH] Support default path for importing openvpn configuration file
- From: "Bin Li" <libin charles gmail com>
- To: networkmanager-list gnome org
- Subject: Re: [PATCH] Support default path for importing openvpn configuration file
- Date: Fri, 24 Oct 2008 23:41:51 +0800
On Fri, Oct 24, 2008 at 10:53 PM, Dan Williams <dcbw redhat com> wrote:
> On Fri, 2008-10-24 at 11:36 +0300, Tambet Ingo wrote:
>> It leaks 'default_path' variable in do_import(). I'd also use
>> g_build_filename() in handle_path_item() in case the path isn't
>> absolute.
>
> Yeah, other than that, looks good though. If you could post it again
> when this is cleaned up, then we can commit.
Tambet, do you commit your change? Do you need I change it like your said?
> Thanks!
> Dan
>
>
>
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]