<div dir="ltr">Hi,<div><br></div><div>Please consider the following patch for your next release.<br><div><br></div><div><div>cgroup's mount_has_subsystem returns the first subsystem</div><div>found or NULL if no subsystem was found; however, the return</div>
<div>value is never initialized, so it's not guaranteed to return NULL.</div><div><br></div><div>Signed-off-by: Kyle Russell <<a href="mailto:bkylerussell@gmail.com">bkylerussell@gmail.com</a>></div><div>---</div>
<div> src/lxc/cgroup.c |    2 +-</div><div> 1 file changed, 1 insertion(+), 1 deletion(-)</div><div><br></div><div>diff --git a/src/lxc/cgroup.c b/src/lxc/cgroup.c</div><div>index b6c948b..f6243b8 100644</div><div>--- a/src/lxc/cgroup.c</div>
<div>+++ b/src/lxc/cgroup.c</div><div>@@ -59,7 +59,7 @@ enum {</div><div> static char *mount_has_subsystem(const struct mntent *mntent)</div><div> {</div><div> <span class="" style="white-space:pre"> </span>FILE *f;</div>
<div>-<span class="" style="white-space:pre">   </span>char *c, *ret;</div><div>+<span class="" style="white-space:pre">    </span>char *c, *ret = NULL;</div><div> <span class="" style="white-space:pre">     </span>char line[MAXPATHLEN];</div>
<div> </div><div> <span class="" style="white-space:pre">   </span>/* read the list of subsystems from the kernel */</div><div>-- </div><div>1.7.10.2.4.g36f8dc1</div></div></div></div>