From df3431f4447c77abff47fd1799a5d13c792e07f1 Mon Sep 17 00:00:00 2001
From: Jan Safranek <jsafrane@redhat.com>
Date: Wed, 24 Aug 2011 11:40:05 +0200
Subject: [PATCH] libcgroup: Fixed potential crash when setting a parameter
 value

fclose(NULL) dereferences its argument -> sigsegv.

Signed-off-by: Jan Safranek <jsafrane@redhat.com>
Acked-by: Dhaval Giani <dhaval.giani@gmail.com>
---
 src/api.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/api.c b/src/api.c
index e443ad50..71cec93d 100644
--- a/src/api.c
+++ b/src/api.c
@@ -1313,9 +1313,9 @@ static int cg_set_control_value(char *path, const char *val)
 					free(tasks_path);
 					return ECGROUPSUBSYSNOTMOUNTED;
 				}
+			} else {
+				fclose(control_file);
 			}
-
-			fclose(control_file);
 			free(tasks_path);
 			return ECGROUPNOTALLOWED;
 		}
-- 
2.47.2