Java: Update test.

aschackmull · aschackmull · commit 06e48ca19f3a · 2019-01-08T11:57:54.000+01:00
diff --git a/java/ql/test/query-tests/security/CWE-190/semmle/tests/IntMultToLong.expected b/java/ql/test/query-tests/security/CWE-190/semmle/tests/IntMultToLong.expected
@@ -1,4 +1,3 @@
 | Test.java:20:23:20:48 | ... * ... | Potential overflow in $@ before it is converted to long by use in an assignment context. | Test.java:20:23:20:48 | ... * ... | int multiplication |
 | Test.java:27:23:27:52 | ... + ... | Potential overflow in $@ before it is converted to long by use in an assignment context. | Test.java:27:23:27:48 | ... * ... | int multiplication |
 | Test.java:34:23:34:63 | ...?...:... | Potential overflow in $@ before it is converted to long by use in an assignment context. | Test.java:34:30:34:55 | ... * ... | int multiplication |
-| Test.java:41:25:41:49 | ... * ... | Potential overflow in $@ before it is converted to double by use in an assignment context. | Test.java:41:25:41:49 | ... * ... | long multiplication |
diff --git a/java/ql/test/query-tests/security/CWE-190/semmle/tests/Test.java b/java/ql/test/query-tests/security/CWE-190/semmle/tests/Test.java
@@ -37,7 +37,7 @@ public static void main(String[] args) {
 		{
 			long timeInSeconds = 10000000L;
 
-			// BAD: same problem, but with longs
+			// same problem, but with longs; not reported as the conversion to double is not sufficient indication of a large number
 			double timeInNanos = timeInSeconds * 10000000L;
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ public static void main(String[] args) {`
`37`	`37`	`{`
`38`	`38`	`long timeInSeconds = 10000000L;`
`39`	`39`
`40`		`- // BAD: same problem, but with longs`
	`40`	`+ // same problem, but with longs; not reported as the conversion to double is not sufficient indication of a large number`
`41`	`41`	`double timeInNanos = timeInSeconds * 10000000L;`
`42`	`42`	`}`
`43`	`43`