cgtop: initialize `ours' to NULL properly (#3139)
authorNaohiro Aota <naota@elisp.net>
Thu, 28 Apr 2016 15:41:50 +0000 (00:41 +0900)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Thu, 28 Apr 2016 15:41:50 +0000 (11:41 -0400)
commit80b2ab4bc00fe18ee4e9f1325bfc7cdd055c6466
tree8613ff120eac0340d38bfb0e2724115bcb6a7260
parent2b6c7113ad49d41c5fa40fef01b1fdcf23bca6f3
cgtop: initialize `ours' to NULL properly (#3139)

Running cgtop on a system, which lacks expecting stat file, results in a
segfault. For example, a system with blkio tree but without cfq io scheduler,
lacks "blkio.io_service_bytes".

When the targeting cgroup's file does not exist, process() returns 0 and
also does not modify `*ret' value (which is `*ours'). As a result,
callers of refresh_one() can have bogus pointer, which result in SEGV.

This patch just properly initialize the variable to NULL.
src/cgtop/cgtop.c