diff options
| author | Sébastien Helleu <flashcode@flashtux.org> | 2018-04-10 19:31:39 +0200 |
|---|---|---|
| committer | Sébastien Helleu <flashcode@flashtux.org> | 2018-04-10 19:31:39 +0200 |
| commit | c89035327cf2fc981699e6e077d3d9e19bd9ee75 (patch) | |
| tree | 75d427348f56083c5a5bf83723fd031cf2974747 /src | |
| parent | e7b2e3cb02e1db02a8c86737e1daf0e3a761cc92 (diff) | |
| download | weechat-c89035327cf2fc981699e6e077d3d9e19bd9ee75.zip | |
ruby: fix memory leak in 7 functions returning allocated strings
Fixed functions:
- list_new
- list_add
- list_search
- list_casesearch
- list_get
- list_next
- list_prev
Diffstat (limited to 'src')
| -rw-r--r-- | src/plugins/ruby/weechat-ruby-api.c | 21 |
1 files changed, 14 insertions, 7 deletions
diff --git a/src/plugins/ruby/weechat-ruby-api.c b/src/plugins/ruby/weechat-ruby-api.c index 2e04a0f10..6c14da64e 100644 --- a/src/plugins/ruby/weechat-ruby-api.c +++ b/src/plugins/ruby/weechat-ruby-api.c @@ -645,12 +645,13 @@ static VALUE weechat_ruby_api_list_new (VALUE class) { char *result; + VALUE return_value; API_INIT_FUNC(1, "list_new", API_RETURN_EMPTY); result = API_PTR2STR(weechat_list_new ()); - API_RETURN_STRING(result); + API_RETURN_STRING_FREE(result); } static VALUE @@ -658,6 +659,7 @@ weechat_ruby_api_list_add (VALUE class, VALUE weelist, VALUE data, VALUE where, VALUE user_data) { char *c_weelist, *c_data, *c_where, *c_user_data, *result; + VALUE return_value; API_INIT_FUNC(1, "list_add", API_RETURN_EMPTY); if (NIL_P (weelist) || NIL_P (data) || NIL_P (where) || NIL_P (user_data)) @@ -678,13 +680,14 @@ weechat_ruby_api_list_add (VALUE class, VALUE weelist, VALUE data, VALUE where, c_where, API_STR2PTR(c_user_data))); - API_RETURN_STRING(result); + API_RETURN_STRING_FREE(result); } static VALUE weechat_ruby_api_list_search (VALUE class, VALUE weelist, VALUE data) { char *c_weelist, *c_data, *result; + VALUE return_value; API_INIT_FUNC(1, "list_search", API_RETURN_EMPTY); if (NIL_P (weelist) || NIL_P (data)) @@ -699,7 +702,7 @@ weechat_ruby_api_list_search (VALUE class, VALUE weelist, VALUE data) result = API_PTR2STR(weechat_list_search (API_STR2PTR(c_weelist), c_data)); - API_RETURN_STRING(result); + API_RETURN_STRING_FREE(result); } static VALUE @@ -727,6 +730,7 @@ static VALUE weechat_ruby_api_list_casesearch (VALUE class, VALUE weelist, VALUE data) { char *c_weelist, *c_data, *result; + VALUE return_value; API_INIT_FUNC(1, "list_casesearch", API_RETURN_EMPTY); if (NIL_P (weelist) || NIL_P (data)) @@ -741,7 +745,7 @@ weechat_ruby_api_list_casesearch (VALUE class, VALUE weelist, VALUE data) result = API_PTR2STR(weechat_list_casesearch (API_STR2PTR(c_weelist), c_data)); - API_RETURN_STRING(result); + API_RETURN_STRING_FREE(result); } static VALUE @@ -770,6 +774,7 @@ weechat_ruby_api_list_get (VALUE class, VALUE weelist, VALUE position) { char *c_weelist, *result; int c_position; + VALUE return_value; API_INIT_FUNC(1, "list_get", API_RETURN_EMPTY); if (NIL_P (weelist) || NIL_P (position)) @@ -784,7 +789,7 @@ weechat_ruby_api_list_get (VALUE class, VALUE weelist, VALUE position) result = API_PTR2STR(weechat_list_get (API_STR2PTR(c_weelist), c_position)); - API_RETURN_STRING(result); + API_RETURN_STRING_FREE(result); } static VALUE @@ -812,6 +817,7 @@ static VALUE weechat_ruby_api_list_next (VALUE class, VALUE item) { char *c_item, *result; + VALUE return_value; API_INIT_FUNC(1, "list_next", API_RETURN_EMPTY); if (NIL_P (item)) @@ -823,13 +829,14 @@ weechat_ruby_api_list_next (VALUE class, VALUE item) result = API_PTR2STR(weechat_list_next (API_STR2PTR(c_item))); - API_RETURN_STRING(result); + API_RETURN_STRING_FREE(result); } static VALUE weechat_ruby_api_list_prev (VALUE class, VALUE item) { char *c_item, *result; + VALUE return_value; API_INIT_FUNC(1, "list_prev", API_RETURN_EMPTY); if (NIL_P (item)) @@ -841,7 +848,7 @@ weechat_ruby_api_list_prev (VALUE class, VALUE item) result = API_PTR2STR(weechat_list_prev (API_STR2PTR(c_item))); - API_RETURN_STRING(result); + API_RETURN_STRING_FREE(result); } static VALUE |