Re: [PATCH] gupnp-service-proxy: remove subscription timeout on subscribe

[Jens Georg <mail jensge org>]

Hi, thanks, looks fine.

One thing:

could you please send patches to Bugzilla (and if you like, announce
them here for broader discussion)?

I usually scan Bugzilla before releases and tend to forget things on the
mailinglist, especially now that I don't have that much time during the
day.

> We have seen crashes in subscription_expire() where user_data
> pointed to invalid memory. I do not see an obvious way how this
> can have happened, but it seems like a good idea to always
> remove the subscription timeout when sending a new subscription
> request.