Code Review
/
dmaap
/
datarouter.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Fix NetworkRoute Vulnerabilities
[dmaap/datarouter.git]
/
datarouter-prov
/
src
/
main
/
java
/
org
/
onap
/
dmaap
/
datarouter
/
provisioning
/
beans
/
NetworkRoute.java
diff --git
a/datarouter-prov/src/main/java/org/onap/dmaap/datarouter/provisioning/beans/NetworkRoute.java
b/datarouter-prov/src/main/java/org/onap/dmaap/datarouter/provisioning/beans/NetworkRoute.java
index
00eb6a2
..
bad6f53
100644
(file)
--- a/
datarouter-prov/src/main/java/org/onap/dmaap/datarouter/provisioning/beans/NetworkRoute.java
+++ b/
datarouter-prov/src/main/java/org/onap/dmaap/datarouter/provisioning/beans/NetworkRoute.java
@@
-73,7
+73,7
@@
public class NetworkRoute extends NodeClass implements Comparable<NetworkRoute>
}
\r
db.release(conn);
\r
} catch (SQLException e) {
\r
}
\r
db.release(conn);
\r
} catch (SQLException e) {
\r
-
e.printStackTrace(
);
\r
+
intlogger.error("SQLException " + e.getMessage()
);
\r
}
\r
return set;
\r
}
\r
}
\r
return set;
\r
}
\r
@@
-127,14
+127,14
@@
public class NetworkRoute extends NodeClass implements Comparable<NetworkRoute>
} catch (SQLException e) {
\r
rv = false;
\r
intlogger.warn("PROV0007 doDelete: " + e.getMessage());
\r
} catch (SQLException e) {
\r
rv = false;
\r
intlogger.warn("PROV0007 doDelete: " + e.getMessage());
\r
-
e.printStackTrace(
);
\r
+
intlogger.error("SQLException " + e.getMessage()
);
\r
} finally {
\r
try {
\r
if(ps!=null) {
\r
ps.close();
\r
}
\r
} catch (SQLException e) {
\r
} finally {
\r
try {
\r
if(ps!=null) {
\r
ps.close();
\r
}
\r
} catch (SQLException e) {
\r
-
e.printStackTrace(
);
\r
+
intlogger.error("SQLException " + e.getMessage()
);
\r
}
\r
}
\r
return rv;
\r
}
\r
}
\r
return rv;
\r
@@
-157,14
+157,14
@@
public class NetworkRoute extends NodeClass implements Comparable<NetworkRoute>
rv = true;
\r
} catch (SQLException e) {
\r
intlogger.warn("PROV0005 doInsert: " + e.getMessage());
\r
rv = true;
\r
} catch (SQLException e) {
\r
intlogger.warn("PROV0005 doInsert: " + e.getMessage());
\r
-
e.printStackTrace(
);
\r
+
intlogger.error("SQLException " + e.getMessage()
);
\r
} finally {
\r
try {
\r
if(ps!=null) {
\r
ps.close();
\r
}
\r
} catch (SQLException e) {
\r
} finally {
\r
try {
\r
if(ps!=null) {
\r
ps.close();
\r
}
\r
} catch (SQLException e) {
\r
-
e.printStackTrace(
);
\r
+
intlogger.error("SQLException " + e.getMessage()
);
\r
}
\r
}
\r
}
\r
}
\r
}
\r
}
\r
@@
-185,14
+185,14
@@
public class NetworkRoute extends NodeClass implements Comparable<NetworkRoute>
} catch (SQLException e) {
\r
rv = false;
\r
intlogger.warn("PROV0006 doUpdate: " + e.getMessage());
\r
} catch (SQLException e) {
\r
rv = false;
\r
intlogger.warn("PROV0006 doUpdate: " + e.getMessage());
\r
-
e.printStackTrace(
);
\r
+
intlogger.error("SQLException " + e.getMessage()
);
\r
} finally {
\r
try {
\r
if(ps!=null) {
\r
ps.close();
\r
}
\r
} catch (SQLException e) {
\r
} finally {
\r
try {
\r
if(ps!=null) {
\r
ps.close();
\r
}
\r
} catch (SQLException e) {
\r
-
e.printStackTrace(
);
\r
+
intlogger.error("SQLException " + e.getMessage()
);
\r
}
\r
}
\r
return rv;
\r
}
\r
}
\r
return rv;
\r