Fix the AV resulting from presence of undefined in the jsDocPropertyTags

sheetalkamat · sheetalkamat · commit f98369b71118 · 2016-10-24T18:10:05.000-07:00
Fixes microsoft#11703
diff --git a/src/compiler/parser.ts b/src/compiler/parser.ts
@@ -6639,12 +6639,16 @@ namespace ts {
                             return true;
                         case "prop":
                         case "property":
-                            if (!parentTag.jsDocPropertyTags) {
-                                parentTag.jsDocPropertyTags = <NodeArray<JSDocPropertyTag>>[];
-                            }
                             const propertyTag = parsePropertyTag(atToken, tagName);
-                            parentTag.jsDocPropertyTags.push(propertyTag);
-                            return true;
+                            if (propertyTag) {
+                                if (!parentTag.jsDocPropertyTags) {
+                                    parentTag.jsDocPropertyTags = <NodeArray<JSDocPropertyTag>>[];
+                                }
+                                parentTag.jsDocPropertyTags.push(propertyTag);
+                                return true;
+                            }
+                            // Error parsing property tag
+                            return false;
                     }
                     return false;
                 }
diff --git a/tests/cases/fourslash/incorrectJsDocObjectLiteralType.ts b/tests/cases/fourslash/incorrectJsDocObjectLiteralType.ts
@@ -0,0 +1,8 @@
+///<reference path="fourslash.ts" />
+
+//// /**/
+
+goTo.marker();
+verify.navigationItemsListCount(0, "foo", "exact");
+edit.insert("/**\n * @typedef {Object} foo\n * @property {any} [obj]\n */\nexport default function foo() {\n}");
+verify.navigationItemsListContains("foo", "function", "foo", "exact");
