php: fix memory leak in 72 functions returning allocated strings

Fixed functions:

- iconv_to_internal
- iconv_from_internal
- string_mask_to_regex
- string_format_size
- string_remove_color
- string_eval_expression
- string_eval_path_home
- list_new
- list_add
- list_search
- list_casesearch
- list_get
- list_next
- list_prev
- config_new
- config_new_section
- config_search_section
- config_new_option
- config_search_option
- config_get
- hook_command
- hook_completion
- hook_command_run
- hook_timer
- hook_fd
- hook_process
- hook_process_hashtable
- hook_connect
- hook_print
- hook_signal
- hook_hsignal
- hook_config
- hook_modifier
- hook_modifier_exec
- hook_info
- hook_info_hashtable
- hook_infolist
- hook_focus
- buffer_new
- buffer_search
- buffer_search_main
- current_buffer
- buffer_get_pointer
- buffer_string_replace_local_var
- current_window
- window_search_with_buffer
- window_get_pointer
- nicklist_add_group
- nicklist_search_group
- nicklist_add_nick
- nicklist_search_nick
- nicklist_group_get_pointer
- nicklist_nick_get_pointer
- bar_item_search
- bar_item_new
- bar_search
- bar_new
- infolist_new
- infolist_new_item
- infolist_new_var_integer
- infolist_new_var_string
- infolist_new_var_pointer
- infolist_new_var_time
- infolist_search_var
- infolist_get
- infolist_pointer
- hdata_get
- hdata_get_list
- hdata_move
- hdata_search
- hdata_pointer
- upgrade_new

2 files changed, 81 insertions, 72 deletions

diff --git a/ChangeLog.adoc b/ChangeLog.adoc
index ad8ebd19b..ec045e93b 100644
--- a/ChangeLog.adoc
+++ b/ChangeLog.adoc
@@ -33,6 +33,7 @@ Bug fixes::
   * irc: fix split of messages when server option "split_msg_max_length" is set to 0 (no split) (issue #1173)
   * irc: send whole IRC message including IRCv3 tags in the signals irc_in, irc_in2, irc_raw_in and irc_raw_in2 (issue #787)
   * irc: fix memory leak when receiving a message with IRCv3 tags
+  * php: fix memory leak in 72 functions returning allocated strings
   * xfer: set option TCP_NODELAY on socket when receiving a file via DCC (issue #1171)
 
 Documentation::
diff --git a/src/plugins/php/weechat-php-api.c b/src/plugins/php/weechat-php-api.c
index 19012db25..4ad86fea7 100644
--- a/src/plugins/php/weechat-php-api.c
+++ b/src/plugins/php/weechat-php-api.c
@@ -72,6 +72,14 @@
 #define API_RETURN_EMPTY RETURN_NULL()
 #define API_RETURN_STRING(__string)                                     \
     RETURN_STRING((__string) ? (__string) : "")
+#define API_RETURN_STRING_FREE(__string)                                \
+    if (__string)                                                       \
+    {                                                                   \
+        RETVAL_STRING(__string);                                        \
+        free (__string);                                                \
+        return;                                                         \
+    }                                                                   \
+    RETURN_STRING("");
 #define API_RETURN_INT(__int) RETURN_LONG(__int)
 #define API_RETURN_LONG(__long) RETURN_LONG(__long)
 #define weechat_php_get_function_name(__zfunc, __str)                   \
@@ -287,7 +295,7 @@ API_FUNC(iconv_to_internal)
     retval = weechat_iconv_to_internal ((const char *)charset,
                                         (const char *)string);
 
-    API_RETURN_STRING(retval);
+    API_RETURN_STRING_FREE(retval);
 }
 
 API_FUNC(iconv_from_internal)
@@ -305,7 +313,7 @@ API_FUNC(iconv_from_internal)
     retval = weechat_iconv_from_internal ((const char *)charset,
                                           (const char *)string);
 
-    API_RETURN_STRING(retval);
+    API_RETURN_STRING_FREE(retval);
 }
 
 API_FUNC(gettext)
@@ -440,7 +448,7 @@ API_FUNC(string_mask_to_regex)
     mask = ZSTR_VAL(z_mask);
     retval = weechat_string_mask_to_regex ((const char *)mask);
 
-    API_RETURN_STRING(retval);
+    API_RETURN_STRING_FREE(retval);
 }
 
 API_FUNC(string_format_size)
@@ -455,7 +463,7 @@ API_FUNC(string_format_size)
 
     retval = weechat_string_format_size ((unsigned long long)z_size);
 
-    API_RETURN_STRING(retval);
+    API_RETURN_STRING_FREE(retval);
 }
 
 API_FUNC(string_remove_color)
@@ -473,7 +481,7 @@ API_FUNC(string_remove_color)
     retval = weechat_string_remove_color ((const char *)string,
                                           (const char *)replacement);
 
-    API_RETURN_STRING(retval);
+    API_RETURN_STRING_FREE(retval);
 }
 
 API_FUNC(string_is_command_char)
@@ -542,7 +550,7 @@ API_FUNC(string_eval_expression)
                                              extra_vars,
                                              options);
 
-    API_RETURN_STRING(retval);
+    API_RETURN_STRING_FREE(retval);
 }
 
 API_FUNC(string_eval_path_home)
@@ -579,7 +587,7 @@ API_FUNC(string_eval_path_home)
                                             extra_vars,
                                             options);
 
-    API_RETURN_STRING(retval);
+    API_RETURN_STRING_FREE(retval);
 }
 
 API_FUNC(mkdir_home)
@@ -653,7 +661,7 @@ API_FUNC(list_new)
     retval = weechat_list_new ();
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(list_add)
@@ -680,7 +688,7 @@ API_FUNC(list_add)
                                user_data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(list_search)
@@ -700,7 +708,7 @@ API_FUNC(list_search)
     retval = weechat_list_search (weelist, (const char *)data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(list_search_pos)
@@ -739,7 +747,7 @@ API_FUNC(list_casesearch)
     retval = weechat_list_casesearch (weelist, (const char *)data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(list_casesearch_pos)
@@ -779,7 +787,7 @@ API_FUNC(list_get)
     retval = weechat_list_get (weelist, position);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(list_set)
@@ -813,7 +821,7 @@ API_FUNC(list_next)
     retval = weechat_list_next (item);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(list_prev)
@@ -830,7 +838,7 @@ API_FUNC(list_prev)
     retval = weechat_list_prev (item);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(list_string)
@@ -958,7 +966,7 @@ API_FUNC(config_new)
         (const char *)data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 static int
@@ -1148,7 +1156,7 @@ API_FUNC(config_new_section)
         (const char *)data_delete_option);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(config_search_section)
@@ -1170,7 +1178,7 @@ API_FUNC(config_search_section)
                                             (const char *)section_name);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 static int
@@ -1294,7 +1302,7 @@ API_FUNC(config_new_option)
         (const char *)data_delete);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(config_search_option)
@@ -1318,7 +1326,7 @@ API_FUNC(config_search_option)
                                            (const char *)option_name);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(config_string_to_boolean)
@@ -1763,7 +1771,7 @@ API_FUNC(config_get)
     retval = weechat_config_get ((const char *)option_name);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(config_get_plugin)
@@ -2081,7 +2089,7 @@ API_FUNC(hook_command)
                                              (const char *)data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 static int
@@ -2134,7 +2142,7 @@ API_FUNC(hook_completion)
                                                 (const char *)data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(hook_completion_get_string)
@@ -2227,7 +2235,7 @@ API_FUNC(hook_command_run)
                                                  (const char *)data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 static int
@@ -2275,7 +2283,7 @@ API_FUNC(hook_timer)
                                            (const char *)data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 static int
@@ -2325,7 +2333,7 @@ API_FUNC(hook_fd)
                                         (const char *)data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 static int
@@ -2375,7 +2383,7 @@ API_FUNC(hook_process)
                                              (const char *)data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 static int
@@ -2433,7 +2441,7 @@ API_FUNC(hook_process_hashtable)
                                                        (const char *)data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 static int
@@ -2505,7 +2513,7 @@ API_FUNC(hook_connect)
                                              (const char *)data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 static int
@@ -2569,7 +2577,7 @@ API_FUNC(hook_print)
                                            (const char *)data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 static int
@@ -2614,7 +2622,7 @@ API_FUNC(hook_signal)
                                             (const char *)data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(hook_signal_send)
@@ -2680,7 +2688,7 @@ API_FUNC(hook_hsignal)
                                              (const char *)data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(hook_hsignal_send)
@@ -2747,7 +2755,7 @@ API_FUNC(hook_config)
                                             (const char *)data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 static char *
@@ -2793,7 +2801,7 @@ API_FUNC(hook_modifier)
                                               (const char *)data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(hook_modifier_exec)
@@ -2814,7 +2822,7 @@ API_FUNC(hook_modifier_exec)
                                          (const char *)modifier_data,
                                          (const char *)string);
 
-    API_RETURN_STRING(retval);
+    API_RETURN_STRING_FREE(retval);
 }
 
 static const char *
@@ -2864,7 +2872,7 @@ API_FUNC(hook_info)
                                           (const char *)data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 struct t_hashtable *
@@ -2917,7 +2925,7 @@ API_FUNC(hook_info_hashtable)
                                                     data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 struct t_infolist *
@@ -2975,7 +2983,7 @@ API_FUNC(hook_infolist)
                                               data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 struct t_hashtable *
@@ -3016,7 +3024,7 @@ API_FUNC(hook_focus)
                                            data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(hook_set)
@@ -3136,7 +3144,7 @@ API_FUNC(buffer_new)
                                            (const char *)data_close);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(buffer_search)
@@ -3155,7 +3163,7 @@ API_FUNC(buffer_search)
     retval = weechat_buffer_search ((const char *)plugin, (const char *)name);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(buffer_search_main)
@@ -3169,7 +3177,7 @@ API_FUNC(buffer_search_main)
     retval = weechat_buffer_search_main ();
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(current_buffer)
@@ -3183,7 +3191,7 @@ API_FUNC(current_buffer)
     retval = weechat_current_buffer ();
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(buffer_clear)
@@ -3309,7 +3317,7 @@ API_FUNC(buffer_get_pointer)
     retval = weechat_buffer_get_pointer (buffer, (const char *)property);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(buffer_set)
@@ -3348,7 +3356,7 @@ API_FUNC(buffer_string_replace_local_var)
     retval = weechat_buffer_string_replace_local_var (buffer,
                                                       (const char *)string);
 
-    API_RETURN_STRING(retval);
+    API_RETURN_STRING_FREE(retval);
 }
 
 API_FUNC(buffer_match_list)
@@ -3381,7 +3389,7 @@ API_FUNC(current_window)
     retval = weechat_current_window ();
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(window_search_with_buffer)
@@ -3398,7 +3406,7 @@ API_FUNC(window_search_with_buffer)
     retval = weechat_window_search_with_buffer (buffer);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(window_get_integer)
@@ -3456,7 +3464,7 @@ API_FUNC(window_get_pointer)
     retval = weechat_window_get_pointer (window, (const char *)property);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(window_set_title)
@@ -3501,7 +3509,7 @@ API_FUNC(nicklist_add_group)
                                          visible);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(nicklist_search_group)
@@ -3526,7 +3534,7 @@ API_FUNC(nicklist_search_group)
                                             (const char *)name);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(nicklist_add_nick)
@@ -3563,7 +3571,7 @@ API_FUNC(nicklist_add_nick)
                                         visible);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(nicklist_search_nick)
@@ -3588,7 +3596,7 @@ API_FUNC(nicklist_search_nick)
                                            (const char *)name);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(nicklist_remove_group)
@@ -3712,7 +3720,7 @@ API_FUNC(nicklist_group_get_pointer)
                                                  (const char *)property);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(nicklist_group_set)
@@ -3810,7 +3818,7 @@ API_FUNC(nicklist_nick_get_pointer)
                                                 (const char *)property);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(nicklist_nick_set)
@@ -3852,7 +3860,7 @@ API_FUNC(bar_item_search)
     retval = weechat_bar_item_search ((const char *)name);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 static char *
@@ -3907,7 +3915,7 @@ API_FUNC(bar_item_new)
                                              (const char *)data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(bar_item_update)
@@ -3954,7 +3962,7 @@ API_FUNC(bar_search)
     retval = weechat_bar_search ((const char *)name);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(bar_new)
@@ -4010,7 +4018,7 @@ API_FUNC(bar_new)
         (const char *)items);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(bar_set)
@@ -4138,7 +4146,7 @@ API_FUNC(infolist_new)
     retval = weechat_infolist_new ();
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(infolist_new_item)
@@ -4155,7 +4163,7 @@ API_FUNC(infolist_new_item)
     retval = weechat_infolist_new_item (infolist);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(infolist_new_var_integer)
@@ -4180,7 +4188,7 @@ API_FUNC(infolist_new_var_integer)
                                                value);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(infolist_new_var_string)
@@ -4203,7 +4211,7 @@ API_FUNC(infolist_new_var_string)
                                               (const char *)value);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(infolist_new_var_pointer)
@@ -4227,7 +4235,7 @@ API_FUNC(infolist_new_var_pointer)
                                                pointer);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(infolist_new_var_time)
@@ -4250,7 +4258,7 @@ API_FUNC(infolist_new_var_time)
     retval = weechat_infolist_new_var_time (item, (const char *)name, time);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(infolist_search_var)
@@ -4270,7 +4278,7 @@ API_FUNC(infolist_search_var)
     retval = weechat_infolist_search_var (infolist, (const char *)name);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(infolist_get)
@@ -4294,7 +4302,7 @@ API_FUNC(infolist_get)
                                    (const char *)arguments);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(infolist_next)
@@ -4415,7 +4423,7 @@ API_FUNC(infolist_pointer)
     retval = weechat_infolist_pointer (infolist, (const char *)var);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(infolist_time)
@@ -4466,7 +4474,7 @@ API_FUNC(hdata_get)
     retval = weechat_hdata_get ((const char *)hdata_name);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(hdata_get_var_offset)
@@ -4589,7 +4597,7 @@ API_FUNC(hdata_get_list)
     retval = weechat_hdata_get_list (hdata, (const char *)name);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(hdata_check_pointer)
@@ -4632,7 +4640,7 @@ API_FUNC(hdata_move)
     retval = weechat_hdata_move (hdata, pointer, count);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(hdata_search)
@@ -4658,7 +4666,7 @@ API_FUNC(hdata_search)
     retval = weechat_hdata_search (hdata, pointer, (const char *)search, move);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(hdata_char)
@@ -4762,7 +4770,7 @@ API_FUNC(hdata_pointer)
     retval = weechat_hdata_pointer (hdata, pointer, (const char *)name);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(hdata_time)
@@ -4927,7 +4935,7 @@ API_FUNC(upgrade_new)
                                             (const char *)data);
     char *__retstr = API_PTR2STR(retval);
 
-    API_RETURN_STRING(__retstr);
+    API_RETURN_STRING_FREE(__retstr);
 }
 
 API_FUNC(upgrade_write_object)