Open waooog opened 7 years ago
Hi,
Our tool found the following suspicious code in: src/org/swiftp/CmdLIST.java
src/org/swiftp/CmdLIST.java
In Line 38: public final static long MS_IN_SIX_MONTHS = 6 * 30 * 24 * 60 * 60 * 1000;
public final static long MS_IN_SIX_MONTHS = 6 * 30 * 24 * 60 * 60 * 1000;
Could this be as follow to avoid integer overflow?:
public final static long MS_IN_SIX_MONTHS = 6 * 30 * 24 * 60 * 60 * 1000L;
This might be a trivial thing but I wanted to report just in case.
Thanks!
Hi,
Our tool found the following suspicious code in:
src/org/swiftp/CmdLIST.java
In Line 38:
public final static long MS_IN_SIX_MONTHS = 6 * 30 * 24 * 60 * 60 * 1000;
Could this be as follow to avoid integer overflow?:
public final static long MS_IN_SIX_MONTHS = 6 * 30 * 24 * 60 * 60 * 1000L;
This might be a trivial thing but I wanted to report just in case.
Thanks!