kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-10-18 18:46:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix shutdown handling of sqlite3 astdb.

Browse Source 
If a db_sync was scheduled just before shutdown, the atexit code calling
db_sync would have no effect, causing the astdb commit thread to stay
alive. This caused the SIP/realtime_sipregs test to fail. (The fallback
kill would run the atexit code again and that would wreak havoc.) This
fixes that the atexit kill condition is picked up properly.

(closes issue ASTERISK-18883)
Reviewed by: Terry Wilson

Review: https://reviewboard.asterisk.org/r/1659
........

Merged revisions 350180 from http://svn.asterisk.org/svn/asterisk/branches/10


git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@350181 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
Walter Doekes
2012-01-09 19:37:23 +00:00
parent f9db1ac0ae
commit a2a3b3ee4b
1 changed files with 14 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
main/db.c
View File

@@ -891,6 +891,15 @@ static void *db_sync_thread(void *data)
ast_mutex_unlock(&dblock);
sleep(1);
ast_mutex_lock(&dblock);
/* Unfortunately, it is possible for signaling to happen
* when we're not waiting: in the bit when we're unlocked
* above. Do the do-exit check here again. (We could do
* it once, but that would impose a forced delay of 1
* second always.) */
if (doexit) {
ast_mutex_unlock(&dblock);
break;
}
}
return NULL;
@@ -898,8 +907,13 @@ static void *db_sync_thread(void *data)
static void astdb_atexit(void)
{
/* Set doexit to 1 to kill thread. db_sync must be called with
* mutex held. */
doexit = 1;
ast_mutex_lock(&dblock);
db_sync();
ast_mutex_unlock(&dblock);
pthread_join(syncthread, NULL);
ast_mutex_lock(&dblock);
sqlite3_close(astdb);