Re: [PATCH 2/2] blkmapd: unlink pid file when error out

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 07/13/2016 10:51 PM, Kinglong Mee wrote:
> Fix some error out without unlink pid file.
> 
> Signed-off-by: Kinglong Mee <kinglongmee@xxxxxxxxx>
Committed... 

steved.

> ---
>  utils/blkmapd/device-discovery.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/utils/blkmapd/device-discovery.c b/utils/blkmapd/device-discovery.c
> index 50e2746..8eb3fd0 100644
> --- a/utils/blkmapd/device-discovery.c
> +++ b/utils/blkmapd/device-discovery.c
> @@ -486,13 +486,13 @@ int main(int argc, char **argv)
>  	signal(SIGHUP, SIG_IGN);
>  
>  	if (dflag) {
> -		bl_discover_devices();
> -		exit(0);
> +		ret = bl_discover_devices();
> +		goto out;
>  	}
>  
>  	if ((bl_watch_fd = inotify_init()) < 0) {
>  		BL_LOG_ERR("init inotify failed %s\n", strerror(errno));
> -		exit(1);
> +		goto out;
>  	}
>  
>  	/* open pipe file */
> @@ -513,7 +513,7 @@ int main(int argc, char **argv)
>  			BL_LOG_ERR("inquiry process return %d\n", ret);
>  		}
>  	}
> -
> +out:
>  	if (pidfd >= 0) {
>  		close(pidfd);
>  		unlink(PID_FILE);
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux