[PATCH] Do not fork daemon when dmeventd cannot be found.

[PATCH] Do not fork daemon when dmeventd cannot be found.

[Milan Broz]

Do not fork daemon when dmeventd cannot be found.

Otherwise it prints many errors about leaked pools.

(More elegant is properly release context, but this 
workaround is sufficient IMHO.)

Signed-off-by: Milan Broz <mbroz@redhat.com>
---
 daemons/dmeventd/libdevmapper-event.c |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/daemons/dmeventd/libdevmapper-event.c b/daemons/dmeventd/libdevmapper-event.c
index 2650cf0..dae76d3 100644
--- a/daemons/dmeventd/libdevmapper-event.c
+++ b/daemons/dmeventd/libdevmapper-event.c
@@ -425,6 +425,12 @@ static int _start_daemon(struct dm_event_fifos *fifos)
 
       start_server:
 	/* server is not running */
+
+	if (stat(DMEVENTD_PATH, &statbuf)) {
+		log_error("Unable to find dmeventd.");
+		return_0;
+	}
+
 	pid = fork();
 
 	if (pid < 0)

[PATCH] Do not fork daemon when dmeventd cannot be found.

[Petr Rockai]

Hi!

Milan Broz <mbroz@redhat.com> writes:
> diff --git a/daemons/dmeventd/libdevmapper-event.c b/daemons/dmeventd/libdevmapper-event.c
> index 2650cf0..dae76d3 100644
> --- a/daemons/dmeventd/libdevmapper-event.c
> +++ b/daemons/dmeventd/libdevmapper-event.c
> @@ -425,6 +425,12 @@ static int _start_daemon(struct dm_event_fifos *fifos)
>  
>        start_server:
>  	/* server is not running */
> +
> +	if (stat(DMEVENTD_PATH, &statbuf)) {
> +		log_error("Unable to find dmeventd.");
> +		return_0;
> +	}
Although not exactly spectacular, I guess this is fine. Just one thing, there's
an execvp later for DMEVENTD_PATH, so this may actually not be really a safe
check (DMEVENTD_PATH -> "dmeventd" should work without this check, but will
break with it).

Yours,
   Petr.

-- 
Peter Rockai | me()mornfall!net | prockai()redhat!com
 http://blog.mornfall.net | http://web.mornfall.net

"In My Egotistical Opinion, most people's C programs should be
 indented six feet downward and covered with dirt."
     -- Blair P. Houghton on the subject of C program indentation