]> granicus.if.org Git - procps-ng/commit
top: address some potential libnuma display corruption
authorJim Warner <james.warner@comcast.net>
Sun, 3 Nov 2013 05:00:00 +0000 (00:00 -0500)
committerJaromir Capik <jcapik@redhat.com>
Mon, 4 Nov 2013 15:00:42 +0000 (16:00 +0100)
commit35dc6dcc49cc9cf8cff4300cb03a38dbe44c05db
tree805e80791f3fa31a8c3c8aad6905536c4551747f
parentaa0e4e7fe24e9760433374ef473a7b0f82a108b3
top: address some potential libnuma display corruption

There is a chance that the libnuma library may corrupt
top's display with some stderr warning messages in the
event something under /sys/devices/system/node/ cannot
be accessed. And, while 2 overridable 'weak' functions
are provided to alter such behavior, we can't use them
since top dynamically links to the library via dlopen.

This commit will redirect stderr to '/dev/null' during
just the first screen display cycle. Thus we can avoid
the corruption which would have remained visible until
the underlining screen row's data had finally changed.

Lastly, this patch should allow such a library warning
to actually appear when one finally exits our program.

[ i think the libnuma folks should consider changing ]
[ the error/warning interfaces to accommodate dlopen ]
[ rather than forcing something like the ugly kludge ]
[ we have employed or libnuma dependency on everyone ]

Reference(s):
https://bugzilla.redhat.com/show_bug.cgi?id=998678

Signed-off-by: Jim Warner <james.warner@comcast.net>
top/top.c