From 24a00d9b0ba1ecab6e821c83c90ca6661c13ac38 Mon Sep 17 00:00:00 2001 From: Sho SHIMIZU Date: Tue, 5 May 2015 11:11:13 -0700 Subject: [PATCH] Catch Exceptions, not Throwable to remove blockers found in SonarQube Change-Id: I49ab731ce0c9950b55063e2afa4ce9226a7b6b4a --- .../java/org/onosproject/net/flow/impl/FlowRuleManager.java | 4 ++-- .../src/main/java/org/onlab/netty/NettyMessagingManager.java | 4 ++-- utils/nio/src/main/java/org/onlab/nio/SelectorLoop.java | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/core/net/src/main/java/org/onosproject/net/flow/impl/FlowRuleManager.java b/core/net/src/main/java/org/onosproject/net/flow/impl/FlowRuleManager.java index b49814f2fb..ebc17ca2d9 100644 --- a/core/net/src/main/java/org/onosproject/net/flow/impl/FlowRuleManager.java +++ b/core/net/src/main/java/org/onosproject/net/flow/impl/FlowRuleManager.java @@ -400,7 +400,7 @@ public class FlowRuleManager extraneousFlow(rule); } } - } catch (Throwable e) { + } catch (Exception e) { log.debug("Can't process added or extra rule {}", e.getMessage()); continue; } @@ -409,7 +409,7 @@ public class FlowRuleManager try { // there are rules in the store that aren't on the switch flowMissing(rule); - } catch (Throwable e) { + } catch (Exception e) { log.debug("Can't add missing flow rule {}", e.getMessage()); continue; } diff --git a/utils/netty/src/main/java/org/onlab/netty/NettyMessagingManager.java b/utils/netty/src/main/java/org/onlab/netty/NettyMessagingManager.java index bfc1c70263..f5504988f7 100644 --- a/utils/netty/src/main/java/org/onlab/netty/NettyMessagingManager.java +++ b/utils/netty/src/main/java/org/onlab/netty/NettyMessagingManager.java @@ -102,8 +102,8 @@ public class NettyMessagingManager implements MessagingService { serverChannelClass = EpollServerSocketChannel.class; clientChannelClass = EpollSocketChannel.class; return; - } catch (Throwable t) { - log.warn("Failed to initialize native (epoll) transport. Reason: {}. Proceeding with nio.", t.getMessage()); + } catch (Exception e) { + log.warn("Failed to initialize native (epoll) transport. Reason: {}. Proceeding with nio.", e.getMessage()); } clientGroup = new NioEventLoopGroup(); serverGroup = new NioEventLoopGroup(); diff --git a/utils/nio/src/main/java/org/onlab/nio/SelectorLoop.java b/utils/nio/src/main/java/org/onlab/nio/SelectorLoop.java index 605e0a92f3..95a9b61ea6 100644 --- a/utils/nio/src/main/java/org/onlab/nio/SelectorLoop.java +++ b/utils/nio/src/main/java/org/onlab/nio/SelectorLoop.java @@ -103,7 +103,7 @@ public abstract class SelectorLoop implements Runnable { state = State.STARTING; try { loop(); - } catch (Throwable e) { + } catch (Exception e) { error = e; log.error("Loop aborted", e); }