Change xcgroup_ns_destroy() and xcgroup_destroy to void.

Nothing checks their return code, and it was always success.

src/slurmd/common/xcgroup.c

@@ -129,19 +129,13 @@ clean:
 
 /*
  * destroy a cgroup namespace
- *
- * returned values:
- *  - XCGROUP_ERROR
- *  - XCGROUP_SUCCESS
  */
-int xcgroup_ns_destroy(xcgroup_ns_t* cgns)
+void xcgroup_ns_destroy(xcgroup_ns_t* cgns)
 {
 	xfree(cgns->mnt_point);
 	xfree(cgns->mnt_args);
 	xfree(cgns->subsystems);
 	xfree(cgns->notify_prog);
-
-	return XCGROUP_SUCCESS;
 }
 
 /*
@@ -399,14 +393,13 @@ int xcgroup_create(xcgroup_ns_t* cgns, xcgroup_t* cg,
 	return XCGROUP_SUCCESS;
 }
 
-int xcgroup_destroy(xcgroup_t* cg)
+void xcgroup_destroy(xcgroup_t* cg)
 {
 	cg->ns = NULL;
 	xfree(cg->name);
 	xfree(cg->path);
 	cg->uid = -1;
 	cg->gid = -1;
-	return XCGROUP_SUCCESS;
 }
 
 int xcgroup_lock(xcgroup_t* cg)

src/slurmd/common/xcgroup.h

@@ -93,12 +93,8 @@ int xcgroup_ns_create(slurm_cgroup_conf_t *conf,
 
 /*
  * destroy a cgroup namespace
- *
- * returned values:
- *  - XCGROUP_ERROR
- *  - XCGROUP_SUCCESS
  */
-int xcgroup_ns_destroy(xcgroup_ns_t* cgns);
+void xcgroup_ns_destroy(xcgroup_ns_t* cgns);
 
 /*
  * mount a cgroup namespace
@@ -161,12 +157,8 @@ int xcgroup_create(xcgroup_ns_t* cgns, xcgroup_t* cg,
 
 /*
  * destroy a cgroup internal structure
- *
- * returned values:
- *  - XCGROUP_ERROR
- *  - XCGROUP_SUCCESS
  */
-int xcgroup_destroy(xcgroup_t* cg);
+void xcgroup_destroy(xcgroup_t* cg);
 
 /*
  * lock a cgroup (must have been instanciated)