Merge "res_stasis: trigger cleanup after update" into 13

13.32
George Joseph 5 years ago committed by Gerrit Code Review
commit a02f14ab9f

@ -1688,12 +1688,11 @@ static int __stasis_app_register(const char *app_name, stasis_app_cb handler, vo
if (app) { if (app) {
/* /*
* We need to unlock the apps_registry before calling app_update to * We need to unlock the apps_registry before calling app_update to
* prevent the possibility of a deadlock with the session. We'll still * prevent the possibility of a deadlock with the session.
* run the lazy cleanup first though.
*/ */
cleanup();
ao2_unlock(apps_registry); ao2_unlock(apps_registry);
app_update(app, handler, data); app_update(app, handler, data);
cleanup();
return 0; return 0;
} }
@ -1718,11 +1717,12 @@ static int __stasis_app_register(const char *app_name, stasis_app_cb handler, vo
} }
ao2_link_flags(apps_registry, app, OBJ_NOLOCK); ao2_link_flags(apps_registry, app, OBJ_NOLOCK);
ao2_unlock(apps_registry);
/* We lazily clean up the apps_registry, because it's good enough to /* We lazily clean up the apps_registry, because it's good enough to
* prevent memory leaks, and we're lazy. * prevent memory leaks, and we're lazy.
*/ */
cleanup(); cleanup();
ao2_unlock(apps_registry);
return 0; return 0;
} }

Loading…
Cancel
Save