Review

Detector:	MuDetect_180212_FixSeqOrder
Target:	project ' tbuktu-ntru ' version 8126929
Misuse:	misuse ' 473 '
Tags:

Misuse Details

Details about the known misuse from the MUBench dataset.

Description:	When a DataOutputStream instance wraps an underlying OutputStream instance, it is recommended to flush or close the DataOutputStream before operating on the OutputStream. Although in these cases this is not strictly necessary because the DataOutputStream's close and flush method has no effects. However, it is a good practice to call flush/close explicitly as mentioned for example here. This pull request adds a call to flush.
Fix Description:	(see diff)
Violation Types:	missing/call
In File:	net/sf/ntru/encrypt/EncryptionParameters.java
In Method:	writeTo(OutputStream)
Code with Misuse:	class EncryptionParameters { /** * Writes the parameter set to an output stream * @param os an output stream * @throws IOException */ public void writeTo(OutputStream os) throws IOException { DataOutputStream dos = new DataOutputStream(os); dos.writeInt(N); dos.writeInt(q); dos.writeInt(df); dos.writeInt(df1); dos.writeInt(df2); dos.writeInt(df3); dos.writeInt(db); dos.writeInt(dm0); dos.writeInt(maxM1); dos.writeInt(c); dos.writeInt(minCallsR); dos.writeInt(minCallsMask); dos.writeBoolean(hashSeed); dos.write(oid); dos.writeBoolean(sparse); dos.writeBoolean(fastFp); dos.write(polyType.ordinal()); dos.writeUTF(hashAlg); } }
Code with Pattern(s):	`class FlushStreamBeforeGetBytes { void pattern(OutputStream out) throws IOException { DataOutputStream dout = new DataOutputStream(out); dout.writeInt(0); dout.flush(); } }`

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
?	0	0.9090909090909091	overlap = 10.00 / 11.00	tbuktu-ntru/misuses/473/patterns-src/FlushStreamBeforeGetBytes.java#pattern(OutputStream)	1
?	1	0.9090909090909091	overlap = 10.00 / 11.00	tbuktu-ntru/misuses/473/patterns-src/FlushStreamBeforeGetBytes.java#pattern(OutputStream)	1
?	2	0.9090909090909091	overlap = 10.00 / 11.00	tbuktu-ntru/misuses/473/patterns-src/FlushStreamBeforeGetBytes.java#pattern(OutputStream)	1
?	3	0.9090909090909091	overlap = 10.00 / 11.00	tbuktu-ntru/misuses/473/patterns-src/FlushStreamBeforeGetBytes.java#pattern(OutputStream)	1
?	4	0.9090909090909091	overlap = 10.00 / 11.00	tbuktu-ntru/misuses/473/patterns-src/FlushStreamBeforeGetBytes.java#pattern(OutputStream)	1
?	5	0.9090909090909091	overlap = 10.00 / 11.00	tbuktu-ntru/misuses/473/patterns-src/FlushStreamBeforeGetBytes.java#pattern(OutputStream)	1
?	6	0.9090909090909091	overlap = 10.00 / 11.00	tbuktu-ntru/misuses/473/patterns-src/FlushStreamBeforeGetBytes.java#pattern(OutputStream)	1
?	7	0.9090909090909091	overlap = 10.00 / 11.00	tbuktu-ntru/misuses/473/patterns-src/FlushStreamBeforeGetBytes.java#pattern(OutputStream)	1
?	8	0.9090909090909091	overlap = 10.00 / 11.00	tbuktu-ntru/misuses/473/patterns-src/FlushStreamBeforeGetBytes.java#pattern(OutputStream)	1
?	9	0.9090909090909091	overlap = 10.00 / 11.00	tbuktu-ntru/misuses/473/patterns-src/FlushStreamBeforeGetBytes.java#pattern(OutputStream)	1
?	10	0.9090909090909091	overlap = 10.00 / 11.00	tbuktu-ntru/misuses/473/patterns-src/FlushStreamBeforeGetBytes.java#pattern(OutputStream)	1
?	11	0.9090909090909091	overlap = 10.00 / 11.00	tbuktu-ntru/misuses/473/patterns-src/FlushStreamBeforeGetBytes.java#pattern(OutputStream)	1