Skip illegally-nested items

danmactough · danmactough · commit 0b77cfcdb1ab · 2018-01-27T22:17:12.000-05:00
diff --git a/lib/feedparser/index.js b/lib/feedparser/index.js
@@ -213,6 +213,7 @@ FeedParser.prototype.handleCloseTag = function (el){
     , stdEl
     , item
     , baseurl
+    , isIllegallyNested = false
     ;
   var n = this.stack.shift();
   el = el.split(':');
@@ -297,6 +298,17 @@ FeedParser.prototype.handleCloseTag = function (el){
       (node['#local'] === 'item' && (node['#prefix'] === '' || node['#type'] === 'rdf')) ||
       (node['#local'] == 'entry' && (node['#prefix'] === '' || node['#type'] === 'atom'))) { // We have an article!
 
+    isIllegallyNested = (
+      ( node['#name'] === 'item' && this.stack[0]['#name'] === 'item' ) ||
+      ( node['#name'] === 'entry' && this.stack[0]['#name'] === 'entry' ) ||
+      ( (node['#local'] === 'item' && (node['#prefix'] === '' || node['#type'] === 'rdf')) && this.stack[0]['#name'] === 'item' ) ||
+      ( (node['#local'] == 'entry' && (node['#prefix'] === '' || node['#type'] === 'atom')) && this.stack[0]['#name'] === 'entry' )
+    );
+
+    if (isIllegallyNested) {
+      return;
+    }
+
     if (!this.meta.title) { // We haven't yet parsed all the metadata
       _.assign(this.meta, this.handleMeta(this.stack[0], this.meta['#type'], this.options));
       if (!this._emitted_meta) {
