chg: Explicitly bail on invalid lines in imported lists

[gigi.git] / util / club / wpia / gigi / util / HighFinancialValueFetcherUmbrella.java

diff --git a/util/club/wpia/gigi/util/HighFinancialValueFetcherUmbrella.java b/util/club/wpia/gigi/util/HighFinancialValueFetcherUmbrella.java
index 321c3df4b22cf4cd515e171ed88a915a5361a7d3..40662e3bf80eb21f8afde718f89fa6a89d50f579 100644 (file)
--- a/util/club/wpia/gigi/util/HighFinancialValueFetcherUmbrella.java
+++ b/util/club/wpia/gigi/util/HighFinancialValueFetcherUmbrella.java
@@ -15,8 +15,16 @@ public class HighFinancialValueFetcherUmbrella extends HighFinancialValueFetcher
     @Override
     public void handle(String line, PrintWriter fos) {
         String[] parts = line.split(",");
     @Override
     public void handle(String line, PrintWriter fos) {
         String[] parts = line.split(",");

+

         // Assert that the value before the "," is an integer
         // Assert that the value before the "," is an integer

-        Integer.parseInt(parts[0]);
+        try {
+            if (Integer.parseInt(parts[0]) < 1) {
+                throw new NumberFormatException("We expect a number greater then zero for the first column.");
+            }
+        } catch (NumberFormatException nfe) {
+            // Bail on lines with invalid first field
+            throw new Error("Invalid format of first column.", nfe);
+        }

 
         String registrablePart = PublicSuffixes.getInstance().getRegistrablePart(parts[1]);
         if (registrablePart != null && printed.add(registrablePart)) {
 
         String registrablePart = PublicSuffixes.getInstance().getRegistrablePart(parts[1]);
         if (registrablePart != null && printed.add(registrablePart)) {