From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mx2.suse.de ([195.135.220.15]) by bombadil.infradead.org with esmtps (Exim 4.85_2 #1 (Red Hat Linux)) id 1bXT58-0008DK-6G for kexec@lists.infradead.org; Wed, 10 Aug 2016 12:58:05 +0000 From: Martin Wilck Subject: [PATCH 3/3] close_dump_bitmap: simplify logic Date: Wed, 10 Aug 2016 14:56:58 +0200 Message-Id: <20160810125658.23366-4-mwilck@suse.de> In-Reply-To: <20160810125658.23366-1-mwilck@suse.de> References: <20160810125658.23366-1-mwilck@suse.de> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "kexec" Errors-To: kexec-bounces+dwmw2=infradead.org@lists.infradead.org To: ats-kumagai@wm.jp.nec.com Cc: Martin Wilck , ptesarik@suse.de, kexec@lists.infradead.org The boolean expression replicates the logic of open_dump_bitmap(). It's simpler and less error-prone to simply check if fd_bitmap is valid. Signed-off-by: Martin Wilck --- makedumpfile.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/makedumpfile.c b/makedumpfile.c index 43278f1..771ab7c 100644 --- a/makedumpfile.c +++ b/makedumpfile.c @@ -8579,8 +8579,7 @@ close_dump_file(void) void close_dump_bitmap(void) { - if (!info->working_dir && !info->flag_reassemble && !info->flag_refiltering - && !info->flag_sadump && !info->flag_mem_usage) + if (!info->fd_bitmap) return; if ((info->fd_bitmap = close(info->fd_bitmap)) < 0) -- 2.9.2 _______________________________________________ kexec mailing list kexec@lists.infradead.org http://lists.infradead.org/mailman/listinfo/kexec