Don't crash when encountering bare raises while checking inconsistent returns

Close #1773

3 files changed, 20 insertions, 1 deletions

diff --git a/ChangeLog b/ChangeLog
index 58205a4eb..1fdc4caa5 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -13,8 +13,11 @@ Release data: |TBA|
 
     * Use the proper node to get the name for redefined functions (#1792)
 
-    Close #1774
+      Close #1774
 
+    * Don't crash when encountering bare raises while checking inconsistent returns
+
+      Close #1773
 
 
 What's New in Pylint 1.8.1?
diff --git a/pylint/checkers/refactoring.py b/pylint/checkers/refactoring.py
index 4f72726af..6d3e4d491 100644
--- a/pylint/checkers/refactoring.py
+++ b/pylint/checkers/refactoring.py
@@ -52,6 +52,9 @@ def _is_node_return_ended(node):
         # a Raise statement doesn't need to end with a return statement
         # but if the exception raised is handled, then the handler has to
         # ends with a return statement
+        if not node.exc:
+            # Ignore bare raises
+            return True
         exc = utils.safe_infer(node.exc)
         if exc is None or exc is astroid.Uninferable:
             return False
diff --git a/pylint/test/functional/inconsistent_returns.py b/pylint/test/functional/inconsistent_returns.py
index 1b9bd447d..22a905465 100644
--- a/pylint/test/functional/inconsistent_returns.py
+++ b/pylint/test/functional/inconsistent_returns.py
@@ -138,3 +138,16 @@ def inconsistent_returns_in_nested_function():
             else:
                 return arg
     return not_consistent_returns_inner
+
+
+class BlargException(Exception):
+    pass
+
+
+def blarg(someval):
+    try:
+        if someval:
+            raise BlargException()
+        return 5
+    except BlargException:
+        raise