1
0
mirror of git://sourceware.org/git/lvm2.git synced 2024-12-22 17:35:59 +03:00

List _thread_registry missed mutex

Operation on _thread_registry needs to be covered by mutex.
Cosmetic move a die code after free for valgind short leak list.
This commit is contained in:
Zdenek Kabelac 2012-03-02 22:57:25 +00:00
parent d177da6b98
commit 00422596c1
3 changed files with 6 additions and 3 deletions

View File

@ -12,7 +12,6 @@ Version 2.02.94 -
Revert free of allocated segtype in init segment error path (2.02.89).
Test dm_hash_insert() failures in filter-persistent.c and fid_add_mda().
Ensure clvmd message is always \0 terminated after read.
Better detection of missing dmeventd fifo connection (2.02.93).
Add some close() and dev_close() error path backtraces.
For polling daemon reopen stdin,stdout,stderr to /dev/null.
Limit the max size of processed clvmd message to ~8KB.

View File

@ -1,8 +1,10 @@
Version 1.02.73 -
====================================
Test _thread_registry list with holding mutex in dmeventd.
Added dm_tree_node_set_callback() for preload and deactivation hooks.
Drop unsupported TRIM message for thin pool.
Improve logging for fifo startup in dmeventd.
Better detection of missing dmeventd fifo connection (1.02.71).
Add few pointer validation in dmsetup.
Support dm_task_get_driver_version() query without version string.
Log failure of pthread_join when cleaning unused threads in dmeventd.

View File

@ -1491,9 +1491,9 @@ static void _process_request(struct dm_event_fifos *fifos)
if (!_client_write(fifos, &msg))
stack;
if (die) raise(9);
dm_free(msg.data);
if (die) raise(9);
}
static void _process_initial_registrations(void)
@ -1987,11 +1987,13 @@ int main(int argc, char *argv[])
while (!_exit_now) {
_process_request(&fifos);
_cleanup_unused_threads();
_lock_mutex();
if (!dm_list_empty(&_thread_registry)
|| !dm_list_empty(&_thread_registry_unused))
_thread_registries_empty = 0;
else
_thread_registries_empty = 1;
_unlock_mutex();
}
_exit_dm_lib();