[OpenWrt-Devel] [PATCH] procd: service gets deleted when its last instance is freed

John Crispin john at phrozen.org
Fri Feb 24 04:53:59 EST 2017



On 24/02/2017 10:26, Alin Nastac wrote:
> Signed-off-by: Alin Nastac <alin.nastac at gmail.com>

Hi,

can you write a little more info as to why this is needed and what
scenario this fixes/changes ?

	John

> ---
>  service/service.c | 5 ++++-
>  service/service.h | 1 +
>  2 files changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/service/service.c b/service/service.c
> index 0584ee0..9675ba2 100644
> --- a/service/service.c
> +++ b/service/service.c
> @@ -140,6 +140,8 @@ service_update(struct service *s, struct blob_attr **tb, bool add)
>  			vlist_flush(&s->instances);
>  	}
>  
> +	s->deleted = false;
> +
>  	rc(s->name, "running");
>  
>  	return 0;
> @@ -149,6 +151,7 @@ static void
>  service_delete(struct service *s)
>  {
>  	vlist_flush_all(&s->instances);
> +	s->deleted = true;
>  	service_stopped(s);
>  }
>  
> @@ -602,7 +605,7 @@ service_start_early(char *name, char *cmdline)
>  
>  void service_stopped(struct service *s)
>  {
> -	if (avl_is_empty(&s->instances.avl)) {
> +	if (s->deleted && avl_is_empty(&s->instances.avl)) {
>  		service_event("service.stop", s->name, NULL);
>  		avl_delete(&services, &s->avl);
>  		trigger_del(s);
> diff --git a/service/service.h b/service/service.h
> index d4f0a83..cc629b1 100644
> --- a/service/service.h
> +++ b/service/service.h
> @@ -40,6 +40,7 @@ struct validate {
>  struct service {
>  	struct avl_node avl;
>  	const char *name;
> +	bool deleted;
>  
>  	struct blob_attr *trigger;
>  	struct vlist_tree instances;
> 
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel


More information about the openwrt-devel mailing list