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

General protection fault ip #522

Closed
francoism90 opened this issue Nov 19, 2019 · 7 comments
Closed

General protection fault ip #522

francoism90 opened this issue Nov 19, 2019 · 7 comments
Labels

Comments

@francoism90
Copy link

traps: gerbera[242601] general protection fault ip:7ff6df6418c2 sp:7ff6c7ffde98 error:0 in libupnp.so.13.1.0[7ff6df62d000+1c000]

This happens after updating Gerbera to support the latest support libupnp. Note: I'm using the use ipv6 switch.

@whyman
Copy link
Member

whyman commented Nov 19, 2019

That is not nice... Can you get a traceback at all? Try running Gerbera under gdb and run bt when it crashes out, it looks like something is going wrong in libupnp.

Please post that along with an error log at debug level, thanks!

@marillat
Copy link
Contributor

marillat commented Nov 20, 2019

I see the same with upnpn 1.10 and gerbera 1.3.4
Of course the crash come from upnp

Thread 10 "gerbera" received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0x7fffe1ffb700 (LWP 56202)]
UpnpExtraHeaders_get_resp (p=p@entry=0x63696e6f73646572) at ../upnp/inc/TemplateSource.h:334
334	../upnp/inc/TemplateSource.h: Aucun fichier ou dossier de ce type.
(gdb) bt 
#0  0x00007ffff7f4c1d0 in UpnpExtraHeaders_get_resp (p=p@entry=0x63696e6f73646572) at ../upnp/inc/TemplateSource.h:334
#1  0x00007ffff7f42e74 in http_MakeMessage (buf=buf@entry=0x7fffe1ff9bc0, http_major_version=1, http_minor_version=1, fmt=0x7ffff7f556e1 "XcsCc", fmt@entry=0x7ffff7f556d6 "RNTGLDstcSEXcsCc")
    at src/genlib/net/http/httpreadwrite.c:1516
#2  0x00007ffff7f45f17 in process_request (RespInstr=0x7fffe1ff9c50, alias=0x7fffe1ff9c00, filename=0x7fffe1ff9be0, headers=0x7fffe1ff9bc0, rtype=<synthetic pointer>, req=<optimized out>)
    at src/genlib/net/http/webserver.c:1320
#3  0x00007ffff7f45f17 in web_server_callback (parser=0x7fffe1ffa310, req=<optimized out>, info=0x7fffe1ffa280) at src/genlib/net/http/webserver.c:1557
#4  0x00007ffff7f3bcdf in dispatch_request (hparser=0x7fffe1ffa310, info=0x7fffe1ffa280) at src/genlib/miniserver/miniserver.c:157
#5  0x00007ffff7f3bcdf in handle_request (args=0x7fffdc003c90) at src/genlib/miniserver/miniserver.c:230
#6  0x00007ffff7f33175 in WorkerThread (arg=<optimized out>) at src/threadutil/ThreadPool.c:576
#7  0x00007ffff7f75fb7 in start_thread () at /lib/x86_64-linux-gnu/libpthread.so.0
#8  0x00007ffff6feb2cf in clone () at /lib/x86_64-linux-gnu/libc.so.6

@francoism90
Copy link
Author

@whyman It doesn't crash when downgrading libupnp 1.8.5 to 1.8.4 and recompiling Gerbera.
I'll try to provide a crash dump later this week.

@KarlStraussberger
Copy link
Member

KarlStraussberger commented Nov 20, 2019

The issue can be solved in libpupnp by a fix https://github.com/mrjimenez/pupnp/pull/122

@whyman
Copy link
Member

whyman commented Nov 20, 2019

@KarlStraussberger Thanks for the patch!

I've replicated the issue with both 1.8.5 and 1.10.0 😞

But the above patch fixes the issue, so we just need to wait for the new versions

@whyman whyman added libupnp and removed needinfo labels Nov 20, 2019
@marillat
Copy link
Contributor

Fixed for me in linupnp 1.10.1
Thanks to @KarlStraussberger for the patch

@whyman
Copy link
Member

whyman commented Nov 21, 2019

Lets close this then, resolved with 1.8.6 and 1.10.1

@whyman whyman closed this as completed Nov 21, 2019
buildroot-auto-update pushed a commit to buildroot/buildroot that referenced this issue Nov 21, 2019
This version fixes a runtime issue which crashes gerbera:
gerbera/gerbera#522

The issue has been reported upstream:
https://github.com/mrjimenez/pupnp/pull/122

The fix for this issue is the only "useful" commit between 1.8.5 and
1.8.6:

$ git --no-pager log --format=oneline release-1.8.5..release-1.8.6
71a47673795e9228775959ea23a984ff6c4d0a43 (tag: release-1.8.6) Adjust the library numbers for release
436aae7b617a4cd7bc1e1411d6882780699eb2ee Put the 1.8.6 release on README.md
90069231d83d2f365b76e2b15d918dfb06209970 Update README.md
7d6158d2c88245f2da4354a8bd0bc359eb15fac6 Update Changelog and THANKS
463f1cc025b27af35b0b73a05ba379d0051bcedf Fix format string for ExtraHeaders
8516da470bf32fa1f5c6f59aac3508378d5a85be Homekeeping for the next release

Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants