libre.sh/keycloak-scim
2
0
Fork 0
Code Issues 14 Pull requests Releases Packages Activity Actions 6

Fix lgtm.com alert: cast int to long before multiplication

Browse source 
The integer multiplication has the potential to overflow before the
result is being cast to the 'long' result.

Details:
https://lgtm.com/projects/g/keycloak/keycloak/snapshot/dist-7900299-1490802114895/files/saml-core/src/main/java/org/keycloak/saml/processing/core/saml/v2/util/XMLTimeUtil.java#V133
This commit is contained in:
Bas van Schaik 2017-04-28 14:54:47 +01:00
parent 2df1175315
commit ff6dbd6bde
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
saml-core/src/main/java/org/keycloak/saml/processing/core/saml/v2/util/XMLTimeUtil.java
View file

@ -130,7 +130,7 @@ public class XMLTimeUtil {
* @return * @return
*/ */
public static long inMilis(int valueInMins) { public static long inMilis(int valueInMins) {
return valueInMins * 60 * 1000; return (long) valueInMins * 60 * 1000;
} }
/** /**
@ -241,4 +241,4 @@ public class XMLTimeUtil {
} }
} }
} }
} }