conf: Remove console stream callback only when freeing console helper

Commit ba226d334a tried to fix crash of
the daemon when a domain with an open console was destroyed. The fix was
wrong as it tried to remove the callback also when the stream was
aborted, where at that point the fd stream driver was already freed and
removed.

This patch clears the callbacks with a helper right before the hash is
freed, so that it doesn't interfere with other codepaths where the
stream object is freed.

(cherry picked from commit 45edefc7a7)
This commit is contained in:
Peter Krempa 2012-08-03 11:20:29 +02:00 committed by Guido Günther
parent b01c3ae891
commit 1baa2fabad

View File

@ -222,9 +222,6 @@ static void virConsoleHashEntryFree(void *data,
const char *pty = name;
virStreamPtr st = data;
/* remove callback from stream */
virFDStreamSetInternalCloseCb(st, NULL, NULL, NULL);
/* free stream reference */
virStreamFree(st);
@ -292,6 +289,18 @@ error:
return NULL;
}
/**
* Helper to clear stream callbacks when freeing the hash
*/
static void virConsoleFreeClearCallbacks(void *payload,
const void *name ATTRIBUTE_UNUSED,
void *data ATTRIBUTE_UNUSED)
{
virStreamPtr st = payload;
virFDStreamSetInternalCloseCb(st, NULL, NULL, NULL);
}
/**
* Free structures for handling open console streams.
*
@ -303,6 +312,7 @@ void virConsoleFree(virConsolesPtr cons)
return;
virMutexLock(&cons->lock);
virHashForEach(cons->hash, virConsoleFreeClearCallbacks, NULL);
virHashFree(cons->hash);
virMutexUnlock(&cons->lock);
virMutexDestroy(&cons->lock);