Commit 9cca111d authored by Rémi Denis-Courmont's avatar Rémi Denis-Courmont

Work around cleanup handler bug in kfile plugin

Parental advisory: explicit hack.

If kfilemodule is loaded, bypass run cleanup handlers. Library really
should not use this C feature in any case (IMHO). And I find it
outright idiotic for a (Qt4) plugin to use cleanup handlers.
This is causing so many crash reports that it's not funny anymore:
https://bugs.launchpad.net/ubuntu/+source/vlc/+bug/408719
parent 6971cd48
......@@ -46,6 +46,8 @@ vlc_static_LDFLAGS = $(vlc_LDFLAGS) -no-install -static
if HAVE_WIN32
vlc_LDADD += -lwininet vlc_win32_rc.$(OBJEXT)
vlc_DEPENDENCIES += vlc_win32_rc.$(OBJEXT)
else
vlc_LDADD += -ldl
endif
.rc.in.rc: $(top_builddir)/config.status
......
......@@ -47,6 +47,7 @@ extern char *FromLocale (const char *);
#include <time.h>
#include <pthread.h>
#include <unistd.h>
#include <dlfcn.h>
/*****************************************************************************
* main: parse command line, start interface and spawn threads.
......@@ -172,5 +173,14 @@ int main( int i_argc, const char *ppsz_argv[] )
for (int i = 1; i < argc; i++)
LocaleFree (argv[i]);
#ifdef RTLD_NOLOAD
/* Avoid crash in KIO scheduler cleanup. */
/* This is ugly, but we get way too many crash reports due to this. */
if (dlopen ("libkfilemodule.so", RTLD_LAZY|RTLD_LOCAL|RTLD_NOLOAD) != NULL)
{
fprintf (stderr, "KFile plugin present. Unclean shutdown!\n");
_exit (0);
}
#endif
return 0;
}
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment