From: Dhaval Giani <dhaval@retis.sssup.it>
Date: Fri, 12 Nov 2010 10:19:55 +0000 (+0100)
Subject: cgclear: Do not rmdir the mount point
X-Git-Tag: v0.37~25
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0cee8cfcbdc71a55f3995b44ebc103a7d1c1328e;p=thirdparty%2Flibcgroup.git

cgclear: Do not rmdir the mount point

This code is strictly not needed, and was originally only added only
because cgconfigparser can mkdir the mountpoint.

Doing it however caused a few bugs and had the possibility of
changing the system state. So, let's not do it.

Should also fix up https://bugzilla.redhat.com/show_bug.cgi?id=652583

Signed-off-by: Dhaval Giani <dhaval.giani@gmail.com>
Tested-by:  Ivana Hutarova Varekova <varekova@redhat.com>
Signed-off-by: Jan Safranek <jsafrane@redhat.com>
---

diff --git a/src/config.c b/src/config.c
index c38b69ec..513f95d8 100644
--- a/src/config.c
+++ b/src/config.c
@@ -752,12 +752,6 @@ static int cgroup_config_unload_controller(const struct cgroup_mount_point *moun
 		goto out_error;
 	}
 
-	error = rmdir(mount_info->path);
-	if (error) {
-		last_errno = errno;
-		ret = ECGOTHER;
-	}
-
 out_error:
 	if (cgroup)
 		cgroup_free(&cgroup);