Hi Tomasz, On Mon, Oct 17, 2011 at 04:49:03PM +0300, Tomasz Bursztyka wrote: > Hi, > > Le 12/10/2011 16:52, Jukka Rissanen a écrit : > > --- > > src/wispr.c | 6 +++++- > > 1 files changed, 5 insertions(+), 1 deletions(-) > > > > diff --git a/src/wispr.c b/src/wispr.c > > index 1f470f7..a0d491a 100644 > > --- a/src/wispr.c > > +++ b/src/wispr.c > > @@ -721,6 +721,8 @@ int __connman_wispr_start(struct connman_service > > *service, > > wp_context->service = service; > > wp_context->type = type; > > > > + connman_service_ref(service); > > + > It will reference service 2 times, if wispr_start() is called for both > ipv4/ipv6, and unref it only once. > > I have prepared a newer version of that fixes so don't commit that one. Good catch, and lucky me I didn't push Jukka's patch. That let me applied yours instead, thanks.
Cheers, Samuel. -- Intel Open Source Technology Centre http://oss.intel.com/ _______________________________________________ connman mailing list [email protected] http://lists.connman.net/listinfo/connman
