channelstorage: Rename callbacks that conflict with DEBUG_FD_LEAKS.

DEBUG_FD_LEAKS replaces calls to "open" and "close" with functions that keep
track of file descriptors, even when those calls are actually callbacks
defined in structures like ast_channelstorage_instance->open and don't touch
file descriptors.  This causes compilation failures.  Those callbacks
have been renamed to "open_instance" and "close_instance" respectively.

Resolves: #1287

main/channelstorage.c

@@ -52,7 +52,7 @@ struct ast_channelstorage_instance *ast_channelstorage_open(
 {
 	struct ast_channelstorage_instance *storage_instance = NULL;
 
-	storage_instance = storage_driver->open(instance_name);
+	storage_instance = storage_driver->open_instance(instance_name);
 	if (!storage_instance) {
 		ast_log(LOG_ERROR, "Failed to open channel storage driver '%s'\n",
 			storage_driver->driver_name);
@@ -64,7 +64,7 @@ struct ast_channelstorage_instance *ast_channelstorage_open(
 
 void ast_channelstorage_close(struct ast_channelstorage_instance *storage_instance)
 {
-	CHANNELSTORAGE_API(storage_instance, close);
+	CHANNELSTORAGE_API(storage_instance, close_instance);
 };
 
 int channelstorage_exten_cb(void *obj, void *arg, void *data, int flags)

main/channelstorage.h

@@ -31,7 +31,7 @@ extern "C" {
 
 struct ast_channelstorage_driver {
 	const char *driver_name;
-	struct ast_channelstorage_instance* (*open)(const char *instance_name);
+	struct ast_channelstorage_instance* (*open_instance)(const char *instance_name);
 };
 
 int ast_channelstorage_register_driver(
@@ -45,7 +45,7 @@ struct ast_channelstorage_driver_pvt;
 struct ast_channelstorage_instance {
 	struct ast_channelstorage_driver_pvt *handle;
 	void *lock_handle;
-	void (*close)(struct ast_channelstorage_instance *driver);
+	void (*close_instance)(struct ast_channelstorage_instance *driver);
 	int (*insert)(struct ast_channelstorage_instance *driver, struct ast_channel *chan, int flags, int lock);
 	int (*remove)(struct ast_channelstorage_instance *driver, struct ast_channel *chan, int lock);
 	void (*rdlock)(struct ast_channelstorage_instance *driver);

main/channelstorage_ao2_legacy.c

@@ -305,7 +305,7 @@ static void close_instance(struct ast_channelstorage_instance *driver)
 
 static struct ast_channelstorage_instance channelstorage_instance = {
 	.handle = NULL,
-	.close = close_instance,
+	.close_instance = close_instance,
 	.insert = insert_channel,
 	.remove = delete_channel,
 	.rdlock = lock_driver,
@@ -386,7 +386,7 @@ static struct ast_channelstorage_instance* get_instance(const char *name)
 
 static struct ast_channelstorage_driver driver_type = {
 	.driver_name = "ao2_legacy",
-	.open = get_instance,
+	.open_instance = get_instance,
 };
 
 static void __attribute__((constructor)) __startup(void)

main/channelstorage_cpp_map_name_id.cc

@@ -387,7 +387,7 @@ static void close_instance(struct ast_channelstorage_instance *driver)
 static struct ast_channelstorage_instance channelstorage_instance = {
 	.handle = NULL,
 	.lock_handle = NULL,
-	.close = close_instance,
+	.close_instance = close_instance,
 	.insert = insert_channel,
 	.remove = delete_channel,
 	.rdlock = rdlock,
@@ -445,7 +445,7 @@ static struct ast_channelstorage_instance* get_instance(const char *name)
 
 static struct ast_channelstorage_driver driver_type = {
 	.driver_name = "cpp_map_name_id",
-	.open = get_instance,
+	.open_instance = get_instance,
 };
 
 static void __attribute__((constructor)) __startup(void)