Merge pull request #208 from emopers/long_bad_76

wumpz · web-flow · commit cd6866c7276a · 2016-08-01T08:11:51.000+02:00
LongValue does not catch NumberFormatException
diff --git a/src/main/java/net/sf/jsqlparser/expression/LongValue.java b/src/main/java/net/sf/jsqlparser/expression/LongValue.java
@@ -34,8 +34,12 @@ public LongValue(final String value) {
 		if (val.charAt(0) == '+') {
 			val = val.substring(1);
 		}
-		this.value = Long.parseLong(val);
-        this.stringValue = val;
+		try {
+		        this.value = Long.parseLong(val);
+		} catch (NumberFormatException e) {
+		        throw new NumberFormatException("Passed value does not contain a parsable long value");
+		}
+		this.stringValue = val;
 	}
 	
 	public LongValue(long value) {

Original file line number	Diff line number	Diff line change
`@@ -34,8 +34,12 @@ public LongValue(final String value) {`
`34`	`34`	`if (val.charAt(0) == '+') {`
`35`	`35`	`val = val.substring(1);`
`36`	`36`	`}`
`37`		`- this.value = Long.parseLong(val);`
`38`		`- this.stringValue = val;`
	`37`	`+ try {`
	`38`	`+ this.value = Long.parseLong(val);`
	`39`	`+ } catch (NumberFormatException e) {`
	`40`	`+ throw new NumberFormatException("Passed value does not contain a parsable long value");`
	`41`	`+ }`
	`42`	`+ this.stringValue = val;`
`39`	`43`	`}`
`40`	`44`
`41`	`45`	`public LongValue(long value) {`