-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bug]: newsboat crashes while processing certain feed #7498
Comments
Still an issue? I was able to reproduce it, but now it works, so I guess that the issue was with one of the entries that have been removed from the feed now. Issue is caused by smileys, and does not occur when newsboat is compiled on device. Probably something is mis-configured when newsboat is cross-compiled, and/or something is weird with our libandroid-support. Adding this: <?xml version="1.0" encoding="UTF-8"?>
<feed xmlns="http://www.w3.org/2005/Atom" xml:lang="de">
<link rel="self" type="application/atom+xml" href="https://www.goingelectric.de/forum/feed/topics"/>
<title>Elektroauto Forum</title>
<subtitle>Elektroauto Forum auf GoingElectric.de</subtitle>
<link href="https://www.goingelectric.de/forum/index.php"/>
<updated>2021-09-08T19:53:14+02:00</updated>
<author><name><![CDATA[Elektroauto Forum]]></name></author>
<id>https://www.goingelectric.de/forum/feed/topics</id>
<entry>
<author><name><![CDATA[ ]]></name></author>
<updated>2021-09-08T11:25:52+02:00</updated>
<published>2021-09-08T11:25:52+02:00</published>
<id>https://www.goingelectric.de/forum/viewtopic.php?t=71495&p=1674610#p1674610</id>
<link href="https://www.goingelectric.de/forum/viewtopic.php?t=71495&p=1674610#p1674610"/>
<title type="html"><![CDATA[ID.4 - Allgemeine Themen • Lichtleiste hinten steht ab😒]]></title>
<category term="ID.4 - Allgemeine Themen" scheme="https://www.goingelectric.de/forum/viewforum.php?f=464" label="ID.4 - Allgemeine Themen"/>
<content type="html" xml:base="https://www.goingelectric.de/forum/viewtopic.php?t=71495&p=1674610#p1674610"><![CDATA[]]></content>
</entry>
</feed> to a feed.xml file, and then adding that file to .newsboat/urls (
|
Looks pretty identical indeed. I suppose you removed the patch after newsboat/newsboat@1e36d49. Bash, readline and a bunch of other software makes sure MB_LEN_MAX is at least 16, maybe newsboat should have a similar check (could open an issue in the newsboat repo to discuss it) |
Hi, Newsboat maintainer here 👋 I suspect this is fixed by newsboat/newsboat#1747. |
@Minoru Hi, yes, that commit fixes it as well (for the example feed above). Thanks for the info, I have replaced the patched used. |
Problem description
Feed:
https://www.goingelectric.de/forum/feed/topics
Error Message:
stack corruption detected (-fstack-protector)
What steps will reproduce the bug?
Just "reload".
What is the expected behavior?
No response
System information
termux-info:
The text was updated successfully, but these errors were encountered: