[vlc-devel] [RFC] freetype and video-title-show crashes

Rafaël Carré funman at videolan.org
Fri Oct 5 11:41:12 CEST 2007


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Bernie Purcell a écrit :
> On Thu, 2007-10-04 at 22:55 +0200, Jean-Paul Saman wrote:
>> Maybe the module is shot down when the thread is still running. And that 
>> would definetly result in a segfault. I believe  you should, do a 
>> vlc_thread_ready(p_this) in the FontBuilder thread and a 
>> vlc_object_attach(). And you should definitely do a clean up of the 
>> thread in the error path.
>>
> I'm a bit new at this, but I'm not sure that's the problem. In the case
> of freetype.c the second thread is created on the same vlc_object as the
> first (we do it in a few other places in the code like decoder.c). As we
> don't have a parent/child relationship here (they are the same object)
> the use of vlc_object_attach doesn't seem to be wanted. Similarly,
> because the object itself is already in use I'm not convinced we should
> be using vlc_thread_ready in this case either - we don't do that in the
> decoder.c code for example.

It seems vlc_thread_ready() is only usefull when the object that created
the thread is waiting using vlc_object_wait()

> Is there any chance that the fix funman made to this module in r22421
> has already fixed the segfault issue?

No, but the two issues seem somehow related.
It looks like in libfontconfig, it tries to read config->fontDirs , but
that config->fontDirs has been previously destroyed.

I've got the debug symbols of libfontconfig, and will have a look at
this issue.

> Bitmap
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> http://mailman.videolan.org/listinfo/vlc-devel
> 


- --
Rafaël Carré
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFHBga4YWCeGMCv8Q8RAq7+AJ4w2/8XslPSlQC7ZUzU8JGBJHWqpwCfcG7d
mmCLnYRC2O9cCS9pdCGRaCg=
=mHXx
-----END PGP SIGNATURE-----



More information about the vlc-devel mailing list