Re: [Jack-Devel] Cleanup of clients in libjack2 causes crash

Le 16 avr. 2013 à 20:56, Bob van Loosen <[hidden]> a écrit :

> Hello,
> 
> I've written a program that can create multiple jack clients and connect them to jackd.
> I'm using a callback passed to jack_on_info_shutdown() to know when a jack client has exited,

?? 

jack_on_info_shutdown is called when the *server* quits….

> when that happens I deactivate and close the client, and I try to make a new one every second.
> This way I can restart jackd without having to restart my program.

> 
> So far so good, until my program started to segfault whenever I closed jackd.
> After some digging I found this code in JackLibGlobals.h:
> 
> if (!JackGlobals::fServerRunning && fClientCount > 0) {
> 
>    // Cleanup remaining clients
>    jack_error("Jack server was closed but clients are still allocated, cleanup...");
>    for (int i = 0; i < CLIENT_NUM; i++) {
>        JackClient* client = JackGlobals::fClientTable[i];
>        if (client) {
>            jack_error("Cleanup client ref = %d", i);
>            client->Close();
>            delete client;
>            JackGlobals::fClientTable[i] = NULL;
>        }
>    }

This code was supposed to close clients that the program incorrectly forgot to close. It is sometimes rather necessary… (you now libjack trying to do the job because developers don't use the API correctly..)
> 
> So what happens here is, jackd exits, I deactivate and close a client, I create a new one with jack_client_open(),
> libjack2 then deletes all the other clients, and my program is then left with several invalid jack_client_t* pointers.

Do you mean double "delete" yes?
> 
> What I suggest here instead, is to just call client->Close() and remove the client from JackGlobals::fClientTable
> (which JackClient::Close() already does btw), and then leave the dealloc up to the program itself.
> 
> Regards,


Please send me a test program that shows the problem.

Stéphane

1366139216.1839_0.ltw:2,a <3420C5DB-8170-4A0A-A86C-3F62E81FB33F at grame dot fr>