Coverity points out this can be null, null-check it to be safe.
If it in indeed null, we must still continue on to verify the
parent loggers.
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
import java.util.Collection;
import java.util.Collections;
import java.util.Collection;
import java.util.Collections;
+import java.util.Enumeration;
import java.util.List;
import java.util.Set;
import java.util.TreeSet;
import java.util.List;
import java.util.Set;
import java.util.TreeSet;
private static boolean hasLttngAppenderAttached(Category logger) {
@SuppressWarnings("unchecked")
private static boolean hasLttngAppenderAttached(Category logger) {
@SuppressWarnings("unchecked")
- List<Appender> appenders = Collections.list(logger.getAllAppenders());
- for (Appender appender : appenders) {
- if (appender instanceof LttngLogAppender) {
- return true;
+ Enumeration<Appender> appenders = logger.getAllAppenders();
+ if (appenders != null) {
+ for (Appender appender : Collections.list(appenders)) {
+ if (appender instanceof LttngLogAppender) {
+ return true;
+ }