Re: [PATCH v4] core: Support for Specifying Resource Limits

2011-03-15 Thread Daniel Wagner
Hi Grant, On 03/15/2011 05:24 PM, Daniel Wagner wrote: fatal: new file src/rlimits.c depends on old contents Patch failed at 0001 core: Support for Specifying Resource Limits When you have resolved this problem run "git am --resolved". If you would prefer to skip this patch, instead run "git am

Re: [PATCH v4] core: Support for Specifying Resource Limits

2011-03-15 Thread Daniel Wagner
> > fatal: new file src/rlimits.c depends on old contents > > Patch failed at 0001 core: Support for Specifying Resource Limits > > When you have resolved this problem run "git am --resolved". > > If you would prefer to skip this patch, instead run "git am --skip". > > To restore the original branc

Re: [PATCH v4] core: Support for Specifying Resource Limits

2011-03-15 Thread Grant Erickson
On 3/15/11 12:51 AM, Daniel Wagner wrote: > On Mon, Mar 14, 2011 at 05:44:24PM -0700, Grant Erickson wrote: >> Added support for optionally specifying, via a global, read-only >> configuration file, system resource limits for address space, core >> files, data segment size, file size, lockable memo

[PATCH v5] core: Support for Specifying Resource Limits

2011-03-15 Thread Grant Erickson
Added support for optionally specifying, via a global, read-only configuration file, system resource limits for address space, core files, data segment size, file size, lockable memory, open files, number of processes and stack size. This mirrors in function, almost identically, resource limit han

[PATCH v4] ofono: Free network_info

2011-03-15 Thread Daniel Wagner
From: Daniel Wagner The network_info data structure is not freed. Also the error path does not handle the freeing of the data structure correctly. Reported by: Henri Bragge --- plugins/ofono.c | 11 --- 1 files changed, 8 insertions(+), 3 deletions(-) diff --git a/plugins/ofono.c b/

[PATCH v3] ofono: Free network_info

2011-03-15 Thread Daniel Wagner
From: Daniel Wagner The network_info data structure is not freed. Also the error path does not handle the freeing of the data structure correctly. Reported by: Henri Bragge --- argh, v2 was a resend of v1, so another try... plugins/ofono.c | 11 +++ 1 files changed, 7 insertions(+)

[PATCH v2] ofono: Free network_info

2011-03-15 Thread Daniel Wagner
From: Daniel Wagner The network_info data structure is not freed. Also the error path does not handle the freeing of the data structure correctly. Reported by: Henri Bragge --- plugins/ofono.c |9 ++--- 1 files changed, 6 insertions(+), 3 deletions(-) diff --git a/plugins/ofono.c b/pl

Re: [PATCH] ofono: Free network_info

2011-03-15 Thread Daniel Wagner
> static char *get_ident(const char *path) > @@ -604,8 +606,10 @@ static int add_network(struct connman_device *device, > > connman_network_set_string(network, "Path", path); > hash_path = connman_network_get_string(network, "Path"); > - if (hash_path == NULL) > + if (hash_pa

[PATCH] ofono: Free network_info

2011-03-15 Thread Daniel Wagner
From: Daniel Wagner The network_info data structure is not freed. Also the error path does not handle the freeing of the data structure correctly. Reported by: Henri Bragge --- Hi Henry, could you give this one a run? Thanks. cheers, daniel plugins/ofono.c |9 ++--- 1 files changed,

Re: [PATCH v4] core: Support for Specifying Resource Limits

2011-03-15 Thread Daniel Wagner
Hi Grant, On Mon, Mar 14, 2011 at 05:44:24PM -0700, Grant Erickson wrote: > Added support for optionally specifying, via a global, read-only > configuration file, system resource limits for address space, core > files, data segment size, file size, lockable memory, open files, > number of processe

Re: [PATCH v3] Added Support for Optionally Specifying Resource Limits

2011-03-15 Thread Daniel Wagner
Good Morning Grant, > > On 03/14/2011 06:42 PM, Grant Erickson wrote: > >> > >> [ ... ] > >> > >> AC_ARG_WITH(dbusconfdir, AC_HELP_STRING([--with-dbusconfdir=PATH], > >> [path to D-Bus config directory]), [path_dbusconf=${withval}], > > > > This D-Bus change looks a bit odd. Is this a left ove