ext/dba: switch dba_handlers full_info path to zend_string API

Author: arshidkv12

ext/dba/dba_cdb.c

@@ -318,14 +318,13 @@ DBA_SYNC_FUNC(cdb)
 DBA_INFO_FUNC(cdb)
 {
 #ifdef DBA_CDB_BUILTIN
-	char* version_str = NULL;
+	const char* version_str = NULL;
 	if (!strcmp(hnd->name, "cdb")) {
 		version_str = cdb_version();
-		return zend_string_init(cdb_version, strlen(cdb_version), false);
 	} else {
 		version_str = cdb_make_version();
-		return zend_string_init(cdb_version, strlen(cdb_version), false);
 	}
+	return zend_string_init(cdb_version, strlen(cdb_version), false);
 #else
 	return zend_string_init("External", strlen("External"), false);
 #endif

ext/dba/dba_gdbm.c

@@ -189,7 +189,7 @@ DBA_SYNC_FUNC(gdbm)
 
 DBA_INFO_FUNC(gdbm)
 {
-	return estrdup(gdbm_version);
+	return zend_string_init(gdbm_version, strlen(gdbm_version), false);
 }
 
 #endif

ext/dba/dba_tcadb.c

@@ -193,7 +193,7 @@ DBA_SYNC_FUNC(tcadb)
 
 DBA_INFO_FUNC(tcadb)
 {
-	return zend_string_init(tcversion, strlen(tcversion), false);;
+	return zend_string_init(tcversion, strlen(tcversion), false);
 }
 
 #endif