Review

Detector:	MuDetect_180220_1_WithinMethodSupport4Real
Target:	project ' chensun ' version cf23b99
Misuse:	misuse ' jadet-1 '
Tags:

Misuse Details

Details about the known misuse from the MUBench dataset.

Description:	The `PreparedStatement` opened in line 635 is never closed. (Same as MuDetect finding 19)
Fix Description:
Violation Types:	missing/call
In File:	database/SocialNetworkDatabasePosts.java
In Method:	getPost(Connection, String, String, String, int)
Code with Misuse:	class SocialNetworkDatabasePosts { /** Gets a post from the designated board and region * with the given post number. * ASSUMES that the board, region, and post are all valid. / public static String getPost(Connection conn, String username, String boardName, String regionName, int postNum) { String getOriginalPost = ""; String getReplies = ""; String postAndReplies = ""; /No joining of results because of redundancy of data returned/ if (boardName.equals("freeforall")) { getOriginalPost = "SELECT FROM freeforall.posts " + "WHERE pid = ?"; getReplies = "SELECT * FROM freeforall.replies " + "WHERE pid = ? ORDER BY dateReplied ASC"; } else { getOriginalPost = "SELECT * FROM " + boardName + ".posts " + "WHERE pid = ? AND rname = ?"; getReplies = "SELECT * FROM " + boardName + ".replies " + "WHERE pid = ? AND rname = ? ORDER BY dateReplied ASC"; } PreparedStatement originalPost = null; ResultSet postResult = null; PreparedStatement replies = null; ResultSet repliesResult = null; boolean sqlex = false; try { originalPost = conn.prepareStatement(getOriginalPost); replies = conn.prepareStatement(getReplies); originalPost.setInt(1, postNum); replies.setInt(1, postNum); if (!boardName.equals("freeforall")) { originalPost.setString(2, regionName); replies.setString(2, regionName); } postResult = originalPost.executeQuery(); if (postResult.next()) { /Only expect one post result/ //Make sure the checksum is correct if (!Arrays.equals( Hash.generateChecksum((SharedKeyCrypto.decrypt(postResult.getString("content"))).getBytes("UTF8")), CryptoUtil.decode(postResult.getString("checksum")))) { postAndReplies += "print ----- Post# " + postNum + "[" + postResult.getString("postedBy") + "]----- " + postResult.getTimestamp("datePosted").toString() + ";print \t" + "Content could not be fetched -- Integrity Failure!" + ";"; } else { postAndReplies += "print ----- Post# " + postNum + "[" + postResult.getString("postedBy") + "]----- " + postResult.getTimestamp("datePosted").toString() + ";print \t" + SharedKeyCrypto.decrypt(postResult.getString("content")) + ";"; } repliesResult = replies.executeQuery(); while (repliesResult.next()) { //Print out all replies //for each reply, make sure the checksum is correct. if(!Arrays.equals( Hash.generateChecksum((SharedKeyCrypto.decrypt(repliesResult.getString("content"))).getBytes("UTF8")), CryptoUtil.decode(repliesResult.getString("checksum")))) { postAndReplies += "print ----- Reply[" + repliesResult.getString("repliedBy") + "] ----- " + repliesResult.getTimestamp("dateReplied").toString() + ";print \t" + "Content could not be fetched -- Integrity Failure!" + ";"; } else { postAndReplies += "print ----- Reply[" + repliesResult.getString("repliedBy") + "] ----- " + repliesResult.getTimestamp("dateReplied").toString() + ";print \t" + SharedKeyCrypto.decrypt(repliesResult.getString("content")) + ";"; } } } // if there's no postResult, the post DNE. } catch (SQLException e) { e.printStackTrace(); sqlex = true; } catch (UnsupportedEncodingException e) { // This should not happen. } if (postAndReplies.equals("") && !sqlex) { return "print Error: Post does not exist. Refresh. If the problem persists, contact an admin."; } else if (sqlex) { return "print Error: Database error while querying post and replies. Contact an admin."; } else return postAndReplies; } }

Potential Hits

Findings of the detector that identify an anomaly in the same file and method as the known misuse.

Hit	Rank	Confidence	Confidence String	Pattern Examples	Pattern Support	Pattern Violation	Target Environment Mapping
Yes	46	0.01036224312086381	(pattern support = 14 / 22)(pattern violations = 1 / 58)(overlap = 17.00 / 18.00)	chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#addUser(Connection, String, String, int, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#insertRegRequest(Connection, String, int, String, String) chensun/cf23b99/build/CS5430/src/database/SocialNetworkDatabasePosts.java#authorizedFFAPost(Connection, String, int, String) chensun/cf23b99/build/CS5430/src/database/SocialNetworkDatabasePosts.java#postExists(Connection, String, String, int) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getParticipantsOne(Connection, String, String)	14
No	47	0.01036224312086381	(pattern support = 14 / 22)(pattern violations = 1 / 58)(overlap = 17.00 / 18.00)	chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#addUser(Connection, String, String, int, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#insertRegRequest(Connection, String, int, String, String) chensun/cf23b99/build/CS5430/src/database/SocialNetworkDatabasePosts.java#authorizedFFAPost(Connection, String, int, String) chensun/cf23b99/build/CS5430/src/database/SocialNetworkDatabasePosts.java#postExists(Connection, String, String, int) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getParticipantsOne(Connection, String, String)	14
No	89	0.006641086186540732	(pattern support = 10 / 22)(pattern violations = 1 / 56)(overlap = 9.00 / 11.00)	chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getAdminsOfGroup(Connection, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getAdmins(Connection) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getFriends(Connection, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getParticipantsOne(Connection, String, String) chensun/cf23b99/build/CS5430/src/database/SocialNetworkDatabaseBoards.java#getBoardAdmins(Connection, String)	10
No	90	0.006641086186540732	(pattern support = 10 / 22)(pattern violations = 1 / 56)(overlap = 9.00 / 11.00)	chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getAdminsOfGroup(Connection, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getAdmins(Connection) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getFriends(Connection, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getParticipantsOne(Connection, String, String) chensun/cf23b99/build/CS5430/src/database/SocialNetworkDatabaseBoards.java#getBoardAdmins(Connection, String)	10
No	91	0.006641086186540732	(pattern support = 10 / 22)(pattern violations = 1 / 56)(overlap = 9.00 / 11.00)	chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getAdminsOfGroup(Connection, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getAdmins(Connection) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getFriends(Connection, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getParticipantsOne(Connection, String, String) chensun/cf23b99/build/CS5430/src/database/SocialNetworkDatabaseBoards.java#getBoardAdmins(Connection, String)	10
No	92	0.006641086186540732	(pattern support = 10 / 22)(pattern violations = 1 / 56)(overlap = 9.00 / 11.00)	chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getAdminsOfGroup(Connection, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getAdmins(Connection) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getFriends(Connection, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getParticipantsOne(Connection, String, String) chensun/cf23b99/build/CS5430/src/database/SocialNetworkDatabaseBoards.java#getBoardAdmins(Connection, String)	10
No	93	0.006641086186540732	(pattern support = 10 / 22)(pattern violations = 1 / 56)(overlap = 9.00 / 11.00)	chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getAdminsOfGroup(Connection, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getAdmins(Connection) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getFriends(Connection, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getParticipantsOne(Connection, String, String) chensun/cf23b99/build/CS5430/src/database/SocialNetworkDatabaseBoards.java#getBoardAdmins(Connection, String)	10
No	94	0.006641086186540732	(pattern support = 10 / 22)(pattern violations = 1 / 56)(overlap = 9.00 / 11.00)	chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getAdminsOfGroup(Connection, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getAdmins(Connection) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getFriends(Connection, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getParticipantsOne(Connection, String, String) chensun/cf23b99/build/CS5430/src/database/SocialNetworkDatabaseBoards.java#getBoardAdmins(Connection, String)	10
No	95	0.006641086186540732	(pattern support = 10 / 22)(pattern violations = 1 / 56)(overlap = 9.00 / 11.00)	chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getAdminsOfGroup(Connection, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getAdmins(Connection) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getFriends(Connection, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getParticipantsOne(Connection, String, String) chensun/cf23b99/build/CS5430/src/database/SocialNetworkDatabaseBoards.java#getBoardAdmins(Connection, String)	10
No	96	0.006641086186540732	(pattern support = 10 / 22)(pattern violations = 1 / 56)(overlap = 9.00 / 11.00)	chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getAdminsOfGroup(Connection, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getAdmins(Connection) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getFriends(Connection, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getParticipantsOne(Connection, String, String) chensun/cf23b99/build/CS5430/src/database/SocialNetworkDatabaseBoards.java#getBoardAdmins(Connection, String)	10
No	97	0.006641086186540732	(pattern support = 10 / 22)(pattern violations = 1 / 56)(overlap = 9.00 / 11.00)	chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getAdminsOfGroup(Connection, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getAdmins(Connection) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getFriends(Connection, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getParticipantsOne(Connection, String, String) chensun/cf23b99/build/CS5430/src/database/SocialNetworkDatabaseBoards.java#getBoardAdmins(Connection, String)	10
No	98	0.006641086186540732	(pattern support = 10 / 22)(pattern violations = 1 / 56)(overlap = 9.00 / 11.00)	chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getAdminsOfGroup(Connection, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getAdmins(Connection) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getFriends(Connection, String) chensun/cf23b99/build/CS5430/src/database/DatabaseAdmin.java#getParticipantsOne(Connection, String, String) chensun/cf23b99/build/CS5430/src/database/SocialNetworkDatabaseBoards.java#getBoardAdmins(Connection, String)	10

Reviewer Name:	sven
Comment: