Skip to content
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

Newsboat stack corruption when opening article containing emojis #4334

Closed
ghost opened this issue Sep 24, 2019 · 5 comments
Closed

Newsboat stack corruption when opening article containing emojis #4334

ghost opened this issue Sep 24, 2019 · 5 comments
Labels
bug report Something is not working properly.

Comments

@ghost
Copy link

ghost commented Sep 24, 2019

Problem description
Newsboat crashes with the message "stack corruption detected" when opening an article containing emojis.
Video: https://i.imgur.com/1fehXWJ.mp4

Steps to reproduce
Open an article in Newsboat containing emojis. (Note: I am not sure if the title of the article matters, in this case both the title and the content contain emojis.)

Feed used in example: https://www.twitrss.me/mobile_twitter_to_rss/?user=nitrome

Expected behavior
It should open the article like it does on my Arch Linux PC. Keep in mind as of writing Arch's Newsboat package version is version 2.17, while on Termux it's 2.16.1.

(Note: I am connected via ssh in the video, but the same issue occurs in Termux)

Additional information

$ termux-info
Updatable packages:
All packages up to date
Subscribed repositories:
https://dl.bintray.com/grimler/science-packages-24 science/stable
https://dl.bintray.com/grimler/game-packages-24 games/stable
https://dl.bintray.com/termux/termux-packages-24 stable/main
System information:
Linux localhost 3.10.49-14895767 #1 SMP PREEMPT Tue Jun 11 20:25:44 KST 2019 armv7l Android
Termux-packages arch:
arm
Android version:
7.1.1
Device manufacturer:
samsung
Device model:
SM-A5100
@ghost
Copy link

ghost commented Sep 24, 2019

Keep in mind as of writing Arch's Newsboat package version is version 2.15,

Perhaps 2.17, isn't it ?

@ghost ghost added the bug report Something is not working properly. label Sep 24, 2019
@ghost
Copy link
Author

ghost commented Sep 24, 2019

Yeah, updated recently I see. It's been out of date for some time :P

I still can't reproduce the issue on Arch with 2.17, so going to quickly edit that.

ghost pushed a commit that referenced this issue Sep 24, 2019
@ghost
Copy link

ghost commented Sep 24, 2019

I have added a fix - bc9082e. Error seems Termux-specific and happens on ARM devices only.

@ghost
Copy link

ghost commented Sep 24, 2019

Try pkg upgrade and check if error is gone.

@ghost
Copy link
Author

ghost commented Sep 26, 2019

It works! Good job on fixing it.

@ghost ghost closed this as completed Sep 26, 2019
ghost pushed a commit that referenced this issue Sep 28, 2019
amuramatsu pushed a commit to amuramatsu/termux-packages that referenced this issue Dec 7, 2019
ghost pushed a commit that referenced this issue Sep 8, 2021
Issue #7498

Android's MB_LEN_MAX is too small (4). On Linux distributions it is
usually 16, so using this value which resolved the older issue:
#4334
@ghost ghost locked and limited conversation to collaborators Oct 9, 2021
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug report Something is not working properly.
Projects
None yet
Development

No branches or pull requests

0 participants