Fix NullPointerException #80

Handle file not found condition - skipping violation was not handle correctly
Bertk · Dec 18, 2013 · 9c6a633 · 9c6a633
1 parent 53fc223
commit 9c6a633
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/sonar-cxx-plugin/src/main/java/org/sonar/plugins/cxx/utils/CxxReportSensor.java b/sonar-cxx-plugin/src/main/java/org/sonar/plugins/cxx/utils/CxxReportSensor.java
@@ -166,16 +166,18 @@ protected void saveViolation(Project project, SensorContext context, String rule
               CxxUtils.LOG.warn("Skipping invalid line number: {}", line);
             }
           }
+          // file level violation
+          violation.setMessage(msg);
+          context.saveViolation(violation);        
         } else {
           CxxUtils.LOG.warn("Cannot find the file '{}', skipping violation '{}'", file, msg);
         }
       } else {
         // project level violation
         violation = Violation.create(rule, project);
+        violation.setMessage(msg);
+        context.saveViolation(violation);        
       }
-
-      violation.setMessage(msg);
-      context.saveViolation(violation);
     } else {
       CxxUtils.LOG.warn("Cannot find the rule {}, skipping violation", ruleId);
     }