Skip to content

Commit 8343821

Browse files
committed
Fix NPE when playlist is empty
1 parent 0400ae0 commit 8343821

1 file changed

Lines changed: 8 additions & 2 deletions

File tree

extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/extractors/YoutubePlaylistExtractor.java

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,8 @@
66
import org.jsoup.Jsoup;
77
import org.jsoup.nodes.Document;
88
import org.jsoup.nodes.Element;
9-
import org.schabi.newpipe.extractor.*;
9+
import org.schabi.newpipe.extractor.Downloader;
10+
import org.schabi.newpipe.extractor.StreamingService;
1011
import org.schabi.newpipe.extractor.exceptions.ExtractionException;
1112
import org.schabi.newpipe.extractor.exceptions.ParsingException;
1213
import org.schabi.newpipe.extractor.linkhandler.LinkHandlerFactory;
@@ -19,6 +20,7 @@
1920
import org.schabi.newpipe.extractor.utils.Utils;
2021

2122
import javax.annotation.Nonnull;
23+
import javax.annotation.Nullable;
2224
import java.io.IOException;
2325

2426
@SuppressWarnings("WeakerAccess")
@@ -172,9 +174,13 @@ private String getNextPageUrlFrom(Document d) throws ParsingException {
172174
}
173175
}
174176

175-
private void collectStreamsFrom(StreamInfoItemsCollector collector, Element element) {
177+
private void collectStreamsFrom(@Nonnull StreamInfoItemsCollector collector, @Nullable Element element) {
176178
collector.reset();
177179

180+
if (element == null) {
181+
return;
182+
}
183+
178184
final LinkHandlerFactory streamLinkHandlerFactory = getService().getStreamLHFactory();
179185
for (final Element li : element.children()) {
180186
if(isDeletedItem(li)) {

0 commit comments

Comments
 (0)